Auto merge of #30834 - reem:rwlock-read-guard-map, r=alexcrichton

This is very useful when the RwLock is synchronizing access to a data
structure and you would like to return or store guards which contain
references to data inside the data structure instead of the data structure
itself.

Auto merge of #31056 - kamalmarhubi:std-process-nul-chars, r=alexcrichton

This reports an error at the point of calling `Command::spawn()` or one of
its equivalents.

Fixes #30858
Fixes #30862

std: Properly handle interior NULs in std::process

This reports an error at the point of calling `Command::spawn()` or one of
its equivalents.

Fixes https://github.com/rust-lang/rust/issues/30858
Fixes https://github.com/rust-lang/rust/issues/30862

Auto merge of #31371 - apasel422:docs, r=alexcrichton

r? @steveklabnik

Auto merge of #31385 - oli-obk:doc/mir, r=nagisa

I didn't change any content, just added another slash so we can see those comments in the docs

r? @steveklabnik

upgrade comments on MIR structures and functions to doc comments

Auto merge of #31375 - nagisa:path-docs, r=alexcrichton

Auto merge of #31338 - dirk:dirk/add-name-bindings-for-bad-imports, r=nrc

WIP implementation of #31209.

The goal is to insert fake/dummy definitions for names that we failed to import so that later resolver stages won't complain about them.

Auto merge of #31263 - dhuseby:fixing_bsd_builds, r=alexcrichton

Something went haywire with github last night and the old PR https://github.com/rust-lang/rust/pull/31230 got closed somehow.  This new PR is to replace the old one.  This incorporates all of the feedback from the other PR.

@alexcrichton I incorporated the suggestion from @semarie and the result is cleaner and clearer.  I think this is ready to go.

trying again at fixing stackp initialization

simplifying get_stack

refactoring get_stack to be cleaner

unifying name_bytes now that the two blocks are the same

Fixes #31229

Spelling fix in `middle::def::Def`

Add fake import resolutions & targets for names in bad imports

Auto merge of #31319 - alexcrichton:msvc-backtraces, r=michaelwoerister

This mirrors the behavior of `clang-cl.exe` by adding a `CodeView` global
variable when emitting debug information. This should in turn help stack traces
that are generated when code is compiled with debuginfo enabled.

Closes #28133

Auto merge of #31370 - Manishearth:rollup, r=Manishearth

- Successful merges: #27499, #31220, #31329, #31332, #31347, #31351, #31352, #31366
- Failed merges:

Add issue number to guard map methods.

Improve docs for Path::methods

Auto merge of #31361 - alexcrichton:revert-mk-changes, r=brson

This reverts commit d03712977d7c913044f2b863269c4491d7fa7c36.

Correct `linked_list::IntoIter` doc comment

Rollup merge of #31366 - paulsmith:patch-1, r=steveklabnik

The context of the link is `Result` but it points to the docs on `Option`'s `expect`.

Rollup merge of #31352 - steveklabnik:gh31154, r=nikomatsakis

Fixes #31154

Rollup merge of #31351 - steveklabnik:gh31318, r=alexcrichton

This is a behavior that some find confusing, so it deserves its own example.

Fixes #31318

I think this wording might be a bit strange, but I couldn't come up with anything better. Feedback very welcome.

Rollup merge of #31347 - GuillaumeGomez:fix_E0118, r=Manishearth

r? @eddyb

Rollup merge of #31332 - durka:errorck-fileline, r=brson

Small modification to #31288. Shows full path and line number for unused error codes.

Rollup merge of #31329 - quodlibetor:no-const-doc-in-stable, r=alexcrichton

Fixes #31098

AFAICT this is the only place where rustdoc explicitly checks if we are on stable before emitting content, so I can't tell if this is the sane way to handle this, or if anything else should be done to make sure that nobody forgets to remove this check when `const` is stabilized.

Rollup merge of #31220 - steveklabnik:gh30632, r=nikomatsakis

Fixes #30632

I'm not sure if this explanation is good enough. If it is, I will add it to filter as well.

Rollup merge of #27499 - barosl:macro-doc-raw-str-hashes, r=nikomatsakis

Any documentation comments that contain raw-string-looking sequences may pretty-print invalid code when expanding them, as the current logic always uses the `r"literal"` form, without appending any `#`s.

This commit calculates the minimum number of `#`s required to wrap a comment correctly and appends `#`s appropriately.

Fixes #27489.

Auto merge of #31312 - alexcrichton:no-le-in-powerpc64le, r=alexcrichton

Currently the `mipsel-unknown-linux-gnu` target doesn't actually set the
`target_arch` value to `mipsel` but it rather uses `mips`. Alternatively the
`powerpc64le` target does indeed set the `target_arch` as `powerpc64le`,
causing a bit of inconsistency between theset two.

As these are just the same instance of one instruction set, let's use
`target_endian` to switch between them and only set the `target_arch` as one
value. This should cut down on the number of `#[cfg]` annotations necessary and
all around be a little more ergonomic.

Add note about temporaries

Fix reference to `expect`

The context of the link is `Result` but it points to the docs on `Option`'s `expect`.

Auto merge of #31254 - tmiasko:macro-pretty-print-fix, r=sfackler

Pretty printing of macro with braces but without terminated semicolon
removed more boxes from stack than it put there, resulting in panic.
This fixes the issue #30731.

Auto merge of #30991 - rthomas:master, r=Gankro

Using the test @bluss suggested in #30983

Auto merge of #31359 - steveklabnik:rollup, r=steveklabnik

- Successful merges: #30971, #31202, #31247, #31270, #31281, #31327, #31339, #31340, #31342, #31344, #31345, #31346, #31348
- Failed merges:

Revert "mk: fix some undefined variable warnings"

This reverts commit d03712977d7c913044f2b863269c4491d7fa7c36.

Ensure capacity returned of HashMap is max(capacity, length).

r? @Gankro

Rollup merge of #31348 - alexcrichton:shuffle-tiers, r=steveklabnik

Some other shufflings as well:

* Three powerpc triples for Linux have been added recently
* An armv7 linux triple was added recently
* The 64-bit Solaris triple is now mentioned in tier 3

We are currently now also building nightlies for iOS, powerpc triples, and
armv7, but there hasn't been much vetting of the triples themselves so I've left
them in tier 3 for now.

Rollup merge of #31346 - alopatindev:fixes, r=aturon

Space character is missed after `let some_usize`

Rollup merge of #31345 - kamalmarhubi:book-docs-special-section-errors, r=steveklabnik

This matches the usage in the standard library's documentation.

Rollup merge of #31344 - steveklabnik:gh31334, r=alexcrichton

Fixes #31334

This is just a quicker fix for this issue; since I'm working on the next draft of the book, I don't want to put a huuuge amount of work into improving it here.

Rollup merge of #31342 - reeze:patch-1, r=steveklabnik

Rollup merge of #31340 - pra85:patch-1, r=alexcrichton

Spelling mistake -
`familliar` > `familiar`

Rollup merge of #31339 - rthomas:doc, r=alexcrichton

When trying to run a specific test, I found the contributing docs a bit confusing and through a bit of googling found out that TESTNAME takes the fully qual'd name of the test.

I'm unsure if this can also take the source file, but I was unable to get that to work.

Rollup merge of #31327 - dirk:dirk/process-child-safety-docs, r=alexcrichton

`Drop` is not implemented for `Child`, so if it goes out of scope in Rust-land and gets deallocated the child process will continue to exist and execute. If users want a guarantee that the process has finished running and exited they must manually use `kill`, `wait`, or `wait_with_output`.

Fixes #31289.

r? @steveklabnik

Rollup merge of #31281 - oli-obk:patch-2, r=nikomatsakis

Rollup merge of #31270 - ruud-v-a:improve-e0507, r=steveklabnik

E0507 can occur when you try to move out of a member of a mutably borrowed struct, in which case `mem::replace` can help. Mentioning that here hopefully saves future users a trip to Google.

Rollup merge of #31247 - tshepang:redundant-bindings, r=steveklabnik

Rollup merge of #31202 - steveklabnik:gh30459, r=alexcrichton

Fixes #30459

Fun fact: i wanted to write "Arabic" and "Hebrew" in Arabic and Hebrew, but vim kept doing the copy/paste in the wrong direction.

Rollup merge of #30971 - SDX2000:docfixes, r=steveklabnik

Updated documentation to clarify the difference between `and_then` and `map`. This also explains why we need `and_then` in addition to `map`. Please look at the diff for more information.

r?  @alexcrichton

Add doctests for directionality

Thanks @nodakai

Remove "powerpc64le" and "mipsel" target_arch

Currently the `mipsel-unknown-linux-gnu` target doesn't actually set the
`target_arch` value to `mipsel` but it rather uses `mips`. Alternatively the
`powerpc64le` target does indeed set the `target_arch` as `powerpc64le`,
causing a bit of inconsistency between theset two.

As these are just the same instance of one instruction set, let's use
`target_endian` to switch between them and only set the `target_arch` as one
value. This should cut down on the number of `#[cfg]` annotations necessary and
all around be a little more ergonomic.

Auto merge of #31314 - alexcrichton:less-warnings, r=brson

Help cleans up our build a bit and stays in line with the rest of our crates
denying warnings traditionally.

docs: Standardize on 'Errors' header in std docs

Auto merge of #31279 - DanielJCampbell:MacroReferencing, r=nrc

r? @nrc

Update TESTNAME description

Update TESTNAME matching description

Merge branch 'master' of https://github.com/rust-lang/rust into doc

Explain behavior of _

Fixes #31154

Further explain take_while

This is a behavior that some find confusing, so it deserves its own example.

Fixes #31318

Auto merge of #31250 - nrc:more-aborts, r=@nikomatsakis

With this PR we can save-analysis on code with errors, essential foundation work for IDE support.

doc: Move 32-bit MSVC to a tier 1 platform

Some other shufflings as well:

* Three powerpc triples for Linux have been added recently
* An armv7 linux triple was added recently
* The 64-bit Solaris triple is now mentioned in tier 3

We are currently now also building nightlies for iOS, powerpc triples, and
armv7, but there hasn't been much vetting of the triples themselves so I've left
them in tier 3 for now.

Discuss pitfalls of stateful closures with Map

Fixes #30632

doc: bindings not needed for this example

Reviewer requested changes and test fixes

Auto merge of #30367 - tamird:fix-makefile-bugs, r=alexcrichton

Some of this is scary stuff. Probably time to lint against this.

Found with `make --warn-undefined-variables`.

r? @alexcrichton

Fix a documentation typo

Decide to hide constness only in fn/method renders

ConstnessSpace has no knowledge of the type of item it's modifying, so
hide the constness a level up.

book: Change "Failures" to "Errors" in doc special sections chapter

This matches the usage in the standard library's documentation.

make this example more obvious

Fixes #31334

Auto merge of #30901 - mackwic:doc-core-convert, r=steveklabnik

Also add a note about the necessary simplicity of the conversion.
Related issue: #29349

r? @steveklabnik

Auto merge of #31317 - jseyfried:remove_external_module_children, r=nrc

This PR refactors away `Module`'s `external_module_children` and instead puts `extern crate` declarations in `children` like other items, simplifying duplicate checking and name resolution.

This PR also allows values to share a name with extern crates, which are only defined in the type namespace. Other than that, it is a pure refactoring.

r? @nrc

Auto merge of #31303 - alexcrichton:mips-warnings, r=aturon

Currently any compilation to MIPS spits out the warning:

    'generic' is not a recognized processor for this target (ignoring processor)

Doesn't make for a great user experience! We don't encounter this in the normal
bootstrap because the cpu/feature set are set by the makefiles. Instead let's
just propagate these to the defaults for the entire target all the time (still
overridable from the command line) and prevent warnings from being emitted by
default.

Comment fix

Auto merge of #31277 - DanielJCampbell:SpanEquality, r=nrc

r? @nrc

mk: fix some undefined variable warnings

Some of this is scary stuff. Probably time to lint against this.

Found with `make --warn-undefined-variables`.

Auto merge of #31232 - stepancheg:enum-univariant, r=nrc

```
enum Univariant {
    X = 17
}
```

Fixes #10292

Fix typo in doc/book/getting-started.md

Spelling mistake -
`familliar` > `familiar`

Implemented macro referencing for save analysis

Update docs about how to run specific tests

Auto merge of #30866 - jseyfried:fix_shadowed_use_visibility, r=nrc

This reverts PR #30324, fixing bug #30159 in which a public a glob import makes public any preceding imports that share a name with an item in the module being glob imported from.

For example,
```rust
pub fn f() {}
pub mod foo {
    fn f() {}
}

mod bar {
    use f;
    use f as g;
    pub use foo::*; // This makes the first import public but does not affect the second import.
}
```

This is a [breaking-change].

Minor corrections in docs for `std::process::Child`

Auto merge of #31331 - petevine:master, r=alexcrichton

The target was meant as a modern generic `armv7` option, therefore a few changes were necessary:

- gcc's `-march=armv7` was causing compilation failures on modern linux systems
- rust codegen defaulted to `cortex-a7` causing illegal instruction crashes on previous `armv7-a` processors (e.g, cortex-a5, cortex-a8)

show location of unused error codes

Fix the armv7 linux target

Spans now derive equality

Don't show `const` in docs when it's not available

Fixes #31098

Safety docs about `std::process::Child` going out of scope

There is no `Drop` implemented for `Child`, so if it goes out
of scope in Rust-land and gets deallocated, the child process
will continue to exist and execute. If users want a guarantee
that the process has finished running and exited they must
manually use `kill`, `wait`, or `wait_with_output`.

Fixes #31289.

Some changes to save-analysis to cope with errors

Try to run compiler callbacks when we error out

Replace some aborts with Results

Fixes #31207

by removing abort_if_new_errors

Auto merge of #31298 - japaric:mips-musl, r=alexcrichton

This target covers MIPS devices that run the trunk version of OpenWRT.

The x86_64-unknown-linux-musl target always links statically to C libraries. For
the mips(el)-unknown-linux-musl target, we opt for dynamic linking (like most of
other targets do) to keep binary size down.

As for the C compiler flags used in the build system, we use the same flags used
for the mips(el)-unknown-linux-gnu target.

r? @alexcrichton

update libc submodule

fixes failed test (std::os::raw::tests::unix) in x86_64-musl target

 Doc:std::convert: disambiguate traits and keywords

trans: Inform LLVM we want CodeView on MSVC

This mirrors the behavior of `clang-cl.exe` by adding a `CodeView` global
variable when emitting debug information. This should in turn help stack traces
that are generated when code is compiled with debuginfo enabled.

Closes #28133

Fix E0118

Refactor away the field Module::external_module_children in resolve

Revert PR #30324, fixing a bug in which a public glob import can make preceding imports public (fixes #30159).

test: Deny warnings in {core,collections}test

Help cleans up our build a bit and stays in line with the rest of our crates
denying warnings traditionally.