Link compiler test documentation to rustc-guide

Update the compiler test documentation to point to the relevant
rustc-guide page.

Auto merge of #53131 - davidtwco:issue-52663-thread-local-static, r=nikomatsakis

NLL says something "does not live long enough" when talking about a (thread-local) static

Part of #52663.

r? @nikomatsakis

Auto merge of #53124 - davidtwco:issue-52742, r=nikomatsakis

region error messages involving impls are confusing

Part of #52742.

r? @nikomatsakis

Auto merge of #53073 - Mark-Simulacrum:data-structures, r=pnkfelix

Cleanup to librustc::session and related code

No functional changes, just some cleanup.

This also creates the `rustc_fs_util` crate, but I can remove that change if desired. It felt a little odd to force crates to depend on librustc for some fs utilities; and also seemed good to generally keep the size of librustc lower (for compile times); fs_util will compile in parallel with essentially the first crate since it has no dependencies beyond std.

Auto merge of #52788 - LukasKalbertodt:improve-index-mut-error, r=estebank

Add help message for missing `IndexMut` impl

Code:
```rust
let mut map = HashMap::new();
map.insert("peter", 23);
map["peter"] = 27;
```

Before:
```
error[E0594]: cannot assign to immutable indexed content
 --> src/main.rs:7:5
  |
7 |     map["peter"] = 27;
  |     ^^^^^^^^^^^^^^^^^ cannot borrow as mutable
```

With this change (just the `help` was added):
```
error[E0594]: cannot assign to immutable indexed content
 --> index-error.rs:7:5
  |
7 |     map["peter"] = 27;
  |     ^^^^^^^^^^^^^^^^^ cannot borrow as mutable
  |
  = help: trait `IndexMut` is required to modify indexed content, but it is not implemented for std::collections::HashMap<&str, i32>
```

---

Yesterday I did some pair programming with a Rust-beginner. We created a type and implemented `Index` for it. Trying to modify the value returned by the index operation returns in a rather vague error that was not very clear for the Rust beginner. So I tried to improve the situation.

## Notes/questions for reviewers:
- Is the formulation OK like that? I'm fine with changing it.
- Can we be absolutely sure that `IndexMut` is actually not implemented in the case my `help` message is added? I'm fairly sure myself, but there could be some cases I didn't think of. Also, I don't know the compiler very well, so I don't know what exactly certain enum variants are used for.
  - It would be nice to test if `IndexMut` is in fact not implemented for the type, but I couldn't figure out how to check that. If you think that additional check would be beneficial, could you tell me how to check if a trait is implemented?
- Do you think I should change the error message instead of only adding an additional help message?

Auto merge of #53216 - kennytm:rollup, r=kennytm

Rollup of 15 pull requests

Successful merges:

 - #52773 (Avoid unnecessary pattern matching against Option and Result)
 - #53082 (Fix doc link (again))
 - #53094 (Automatically expand section if url id point to one of its component)
 - #53106 (atomic ordering docs)
 - #53110 (Account for --remap-path-prefix in save-analysis)
 - #53116 (NetBSD: fix signedess of char)
 - #53179 (Whitelist wasm32 simd128 target feature)
 - #53183 (Suggest comma when missing in macro call)
 - #53207 (Add individual docs for rotate_{left, right})
 - #53211 ([nll] enable feature(nll) on various crates for bootstrap)
 - #53214 ([nll] enable feature(nll) on various crates for bootstrap: part 2)
 - #53215 (Slightly refactor syntax_ext/format)
 - #53217 (inline some short functions)
 - #53219 ([nll] enable feature(nll) on various crates for bootstrap: part 3)
 - #53222 (A few cleanups for rustc_target)

Rollup merge of #53207 - llogiq:num-rotate-docs, r=QuietMisdreavus

Add individual docs for rotate_{left, right}

Rollup merge of #53222 - ljedrz:cleanup_rustc_target, r=Mark-Simulacrum

A few cleanups for rustc_target

- remove redundant struct field names
- shorten a self-assignment
- prefer `unwrap_or_else` in case of function calls
- collapse an `if`
- collapse a double `map()`
- match on dereferenced objects
- consume `self` if it implements `Copy`

Rollup merge of #53215 - ljedrz:refactor_format, r=estebank

Slightly refactor syntax_ext/format

expand_preparsed_format_args:
- move a potential error `return` earlier in the processing
- pre-allocate some of the required space for `cx.pieces` and `cx.str_pieces`
- create `cx`-independent objects before `cx`
- build `pieces` and `errs` using `collect` instead of a `push` loop

describe_num_args:
- return `Cow<str>` instead of `String`

Rollup merge of #53219 - memoryruins:nll_bootstrap_3, r=nikomatsakis

[nll] enable feature(nll) on various crates for bootstrap: part 3

#53172

Rollup merge of #53214 - memoryruins:nll_bootstrap_2, r=nikomatsakis

[nll] enable feature(nll) on various crates for bootstrap: part 2

#53172

Rollup merge of #53211 - memoryruins:nll_bootstrap, r=nikomatsakis

[nll] enable feature(nll) on various crates for bootstrap

#53172

Rollup merge of #53082 - felixrabe:fix-doc-link-again, r=GuillaumeGomez

Fix doc link (again)

Similar to #52404. The link for comparison:

-   https://doc.rust-lang.org/book/second-edition/ch19-04-advanced-types.html#dynamically-sized-types-and-sized (broken)

-   https://doc.rust-lang.org/book/second-edition/ch19-04-advanced-types.html#dynamically-sized-types-and-the-sized-trait (correct, stable 2nd ed)

-   https://doc.rust-lang.org/nightly/book/second-edition/ch19-04-advanced-types.html#dynamically-sized-types-and-the-sized-trait (correct, nightly 2nd ed)

-   https://doc.rust-lang.org/nightly/book/2018-edition/ch19-04-advanced-types.html#dynamically-sized-types-and-the-sized-trait (correct, nightly 2018 ed)

This commit is the result of (first) searching via ripgrep (0.8.1 -SIMD -AVX):

    rg -l dynamically-sized-types-and-sized

and then replacing all relevant occurrences via:

    find src/{libcore,test/ui} -type f -print0 | xargs -0 sed -i.bak \
      s/dynamically-sized-types-and-sized/dynamically-sized-types-and-the-sized-trait/g
    find src/{libcore,test/ui} -type f -name '*.bak' -print0 | xargs -0 rm

(Note: Tested on on macOS 10.13 (BSD). `sed -i.bak` should work on Linux
(GNU sed) as well, but not tested.)

Rollup merge of #53217 - strake:inline, r=nagisa

inline some short functions

I found these were outline in binaries i link. I think they ought to be inline, considering their size.

Rollup merge of #53106 - RalfJung:ordering, r=stjepang

atomic ordering docs

Discussion in https://github.com/rust-lang/rfcs/pull/2503 revealed that this could be improved. I hope this helps.

Rollup merge of #53183 - estebank:println-comma, r=oli-obk

Suggest comma when missing in macro call

When missing a comma in a macro call, suggest it, regardless of
position. When a macro call doesn't match any of the patterns, check
if the call's token stream could be missing a comma between two idents,
and if so, create a new token stream containing the comma and try to
match against the macro patterns. If successful, emit the suggestion.

This works on arbitrary macros, with no need of special support from
the macro writers.

```
error: no rules expected the token `d`
  --> $DIR/missing-comma.rs:26:18
   |
LL |     foo!(a, b, c d, e);
   |                 -^
   |                 |
   |                 help: missing comma here
```
Follow up to #52397.

Rollup merge of #53179 - gnzlbg:patch-3, r=alexcrichton

Whitelist wasm32 simd128 target feature

r? @alexcrichton

Move span_bug and bug helper functions to util

Cache ignored attributes inside ICH entirely

Move Fingerprint to data structures

Reuse Hash impls for session data structures

Remove dependency on rustc::ty from code_stats

Move path2cstr to rustc_fs_util

Move rustc::util::fs into separate (new) crate

Move SVH structure to data structures

Add individual docs for rotate_{left, right}

A few cleanups for rustc_target

Auto merge of #53031 - michaelwoerister:cross-lto, r=alexcrichton

Apply some fixes to cross-language LTO (especially when targeting MSVC)

This PR contains a few fixes that were needed in order to get Firefox compiling with Rust/C++ cross-language ThinLTO on Windows. The commits are self-contained and should be self-explanatory.

r? @alexcrichton

[nll] libunwind: enable feature(nll) for bootstrap

[nll] libterm: enable feature(nll) for bootstrap

[nll] libsyntax_pos: enable feature(nll) for bootstrap

[nll] librustc_target: enable feature(nll) for bootstrap

[nll] librustc_incremental: enable feature(nll) for bootstrap

[nll] librustc_driver: enable feature(nll) for bootstrap

[nll] librustc_cratesio_shim: enable feature(nll) for bootstrap

[nll] librustc_apfloat: enable feature(nll) for bootstrap

[nll] libpanic_unwind: enable feature(nll) for bootstrap

[nll] libpanic_abort: enable feature(nll) for bootstrap

[nll] libfmt_macros: enable feature(nll) for bootstrap

Updated tests after rebase.

inline some short functions

Rollup merge of #53116 - jakllsch:netbsd-unsigned-char, r=alexcrichton

NetBSD: fix signedess of char

Rollup merge of #53110 - Xanewok:save-analysis-remap-path, r=nrc

Account for --remap-path-prefix in save-analysis

Fixes #52549.

Didn't add a test since save-analysis is still unstable, only tested this locally. Should I add a test for that? If so, is run-make-fulldeps an appropriate format?

Session is already created with remapped working directory, so use that instead of the actual cwd.
This was the only place affected, since the rest of the paths in save-analysis are directly derived from files in spans from `sess.codemap()`, which already creates remapped ones.

r? @nrc

Rollup merge of #53094 - GuillaumeGomez:automatic-expand, r=nrc

Automatically expand section if url id point to one of its component

Fixes #52517.

r? @nrc

Rollup merge of #52773 - ljedrz:unncecessary_patterns, r=nikomatsakis

Avoid unnecessary pattern matching against Option and Result

Fallback to 'has type' error messages rather than 'lifetime appears in type'.

Correctly identify named early bound regions.

[nll] librustc_tsan: enable feature(nll) for bootstrap

[nll] librustc_traits: enable feature(nll) for bootstrap

[nll] librustc_save_analysis: enable feature(nll) for bootstrap

[nll] librustc_resolve: enable feature(nll) for bootstrap

[nll] librustc_privacy: enable feature(nll) for bootstrap

[nll] librustc_plugin: enable feature(nll) for bootstrap

[nll] librustc_passes: enable feature(nll) for bootstrap

[nll] librustc_errors: enable feature(nll) for bootstrap

[nll] librustc_asan: enable feature(nll) for bootstrap

[nll] libarena: enable feature(nll) for bootstrap

Use Cow<str> in describe_num_args

Relax the target-cpu-on-function codegen test so it just checks for presence of the attribute.

Auto merge of #53108 - RalfJung:mutex, r=alexcrichton

clarify partially initialized Mutex issues

Using a `sys_common::mutex::Mutex` without calling `init` is dangerous, and yet there are some places that do this. I tried to find all of them and add an appropriate comment about reentrancy.

I found two places where (I think) reentrancy can actually occur, and was not able to come up with an argument for why this is okay. Someone who knows `io::lazy` and/or `sys_common::at_exit_imp` should have a careful look at this.

Refactor expand_preparsed_format_args

liballoc: enable feature(nll) for bootstrap

liballoc_jemalloc: enable feature(nll) for bootstrap

liballoc_system: enable feature(nll) for bootstrap

libgraphviz: enable feature(nll) for bootstrap

librustc_borrowck: enable feature(nll) for bootstrap

librustc_lint: enable feature(nll) for bootstrap

librustc_metadata: enable feature(nll) for bootstrap

Auto merge of #53100 - VPashkov:issue-52456-improper_ctypes, r=eddyb

Fix improper_ctypes lint for individual foreign items

Fixes #52456.

r? @eddyb

Auto merge of #53186 - mikhail-m1:master, r=nikomatsakis

Fixes #53119.

Fixes #53119.

I minimized sample little bit more, but I checked the sample from issue too.
r? @nikomatsakis

update target-feature-gate.stderr output

Auto merge of #53163 - oli-obk:const_prop_ice, r=nikomatsakis

Remove an overly pedantic and wrong assertion

fixes #53157
fixes #53087

add comment

missed one

avoid using the word 'initialized' to talk about that non-reentrant-capable state of the mutex

Auto merge of #53192 - oli-obk:clippyup, r=kennytm

Update clippy submodule

r? @kennytm

fixes breakage from https://github.com/rust-lang/rust/pull/53053#issuecomment-411364171

Auto merge of #53088 - matthewjasper:closure-region-spans, r=nikomatsakis

[NLL] Use span of the closure args in free region errors

Also adds a note when one of the free regions is BrEnv.
In a future PR I'll change these messages to say "return requires", which should improve them a bit more.

r? @nikomatsakis

Address review comments for #53031 and fix some merge fallout.

Update clippy submodule

Auto merge of #52872 - faern:use-modern-alignment-libc, r=TimNN

Make IpvXAddr::new const fns and the well known addresses associated constants

Implements/fixes https://github.com/rust-lang/rust/issues/44582

I just got a PR towards libc (https://github.com/rust-lang/libc/pull/1044) merged. With the new feature added in that PR it is now possible to create `in6_addr` instances as consts. This enables us to finally make the constructors of the IP structs const fns and to make the localhost/unspecified addresses associated constants, as agreed in the above mentioned tracking issue.

I also added a BROADCAST constant. Personally this is the well known address I tend to need the most often.

Use repr(align(x)) for redox in6_addr

Update libc to 0.2.43 in Cargo.lock

Simplify Ipv6Addr::from(octets) to not use unsafe

Simplify is_broadcast

Add Ipv4Addr BROADCAST assoc const

Move IPs to assoc consts

Make Ipv{4,6}Addr::new const fns

Update libc and activate align feature

Auto merge of #53053 - petrochenkov:custattr, r=alexcrichton

resolve:  Support custom attributes when macro modularization is enabled

Basically, if resolution of a single-segment attribute is a determined error, then we interpret it as a custom attribute.

Since custom attributes are integrated into general macro resolution, `feature(custom_attribute)` now requires and implicitly enables macro modularization (`feature(use_extern_macros)`).
Actually, a few other "advanced" macro features now implicitly enable macro modularization too (and one bug was found and fixed in process of enabling it).

The first two commits are preliminary cleanups/refactorings.

Fixes #53119.

Auto merge of #53141 - nrc:update, r=nrc

Update RLS, Rustfmt, and Clippy

Fixes bustage due to https://github.com/rust-lang/rust/pull/52332

r? @kennytm

Prevent duplication of rust-ap-syntax

author: alexcrichton

Update RLS and Rustfmt

Suggest comma when missing in macro call

When missing a comma in a macro call, suggest it, regardless of
position. When a macro call doesn't match any of the patterns, check
if the call's token stream could be missing a comma between two idents,
and if so, create a new token stream containing the comma and try to
match against the macro patterns. If successful, emit the suggestion.

Auto merge of #52993 - alexcrichton:fix-some-vis, r=michaelwoerister

rustc: Tweak visibility of some lang items

This commit tweaks the linker-level visibility of some lang items that rustc
uses and defines. Notably this means that `#[panic_implementation]` and
`#[alloc_error_handler]` functions are never marked as `internal`. It's up to
the linker to eliminate these, not rustc.

Additionally `#[global_allocator]` generated symbols are no longer forced to
`Default` visibility (fully exported), but rather they're relaxed to `Hidden`
visibility). This symbols are *not* needed across DLL boundaries, only as a
local implementation detail of the compiler-injected allocator symbols, so
`Hidden` should suffice.

Closes #51342
Closes #52795

add feature-gate test

add wasm_target_feature feature gate

Add wasm32 simd128 target feature

Add and update tests for `IndexMut` help message

Auto merge of #52397 - estebank:println-comma, r=oli-obk

Suggest comma when writing `println!("{}" a);`

Fix #49370.