Remove incorrect space

Remove unnecessary comment

Turns out I didn't read the original comment correctly and had some
confusion about how MIR uses temporaries.

Test patterns in tuples

Coalesce var-is-not-Copy notes per move

Dedup suggestions

Remove unnecessary underscore

Change TODOs to FIXMEs

Make move errors more consistent with typeck errors

Make move error suggestions clearer

Add backquotes around variable name

Fix move errors for index expressions

The suggestion logic gave up too early, which kept it from suggesting
borrowing index expressions.

Test move errors for index expressions

Accept whitespace between `&` and `mut`

Add more TODOs

Suggest match ergonomics, not `ref`/`ref mut`

Fix some random stuff

Removed `ignore-test-compare-mode-nll` from unboxed-closures tests
by strengthening the tests (by adding no-op references to the
closures doing the borrows after the conflicting borrows, thus
forcing the lifetimes to resemble lexical scopes even under NLL).

Removed `ignore-test-compare-mode-nll` from hashmap-lifetimes.rs
by strengthening the test there.

Removed `ignore-test-compare-mode-nll` from borrowck-closures-two-mut-fail.rs
by strengthening the tests there.

Removed `ignore-test-compare-mode-nll` from borrowck-closures-unique.rs
by strengthening the tests there.

In almost all cases the strengthening amount to just encoding a use
that models the original lexical lifetime. A more invasive revision
was made in one case where it seems the actual issue is MIR-borrowck's
greater "knowledge" of unreachable code in the control flow...

Do not emit "incorrect close delimiter" twice in the same place

Tweak unclosed delimiter parser error

Do not suggest conversion method that is already there

Auto merge of #53400 - GuillaumeGomez:rollup, r=GuillaumeGomez

Rollup of 8 pull requests

Successful merges:

 - #52453 (improve diagnostics for tests with custom return values)
 - #53271 (use ? to simplify `TransitiveRelation.maybe_map`)
 - #53279 (Extend documentation of `rustc_on_unimplemented`)
 - #53342 (fix error for unsized packed struct field)
 - #53344 (Add doc examples for std::alloc::{alloc,alloc_zeroed}.)
 - #53368 (Ignore test that fails on stage1)
 - #53388 (Fix links' color)
 - #53396 (Fix since of Iterator::flatten to be a proper semver)

Failed merges:

r? @ghost

Test VecDeque append not dropping twice

Rollup merge of #53396 - matklad:fix-flatten-since, r=varkor

Fix since of Iterator::flatten to be a proper semver

r? @kennytm

Rollup merge of #53388 - GuillaumeGomez:fix-links-color, r=frewsxcv

Fix links' color

Fixes #53375.

Problem was coming from the minifier.

r? @frewsxcv

Rollup merge of #53368 - matthewjasper:only-stage-2, r=varkor

Ignore test that fails on stage1

The error code is not emitted on stage1, so ignore this test there.

Rollup merge of #53344 - frewsxcv:frewsxcv-doc-ptr, r=ollie27

Add doc examples for std::alloc::{alloc,alloc_zeroed}.

None

Rollup merge of #53342 - RalfJung:unsized-packed, r=cramertj

fix error for unsized packed struct field

It was really confusing to be told "only the last field of a struct may have a dynamically sized type" when only the last field *was* unsized.

Rollup merge of #53279 - estebank:on-unimplemented-doc, r=GuillaumeGomez

Extend documentation of `rustc_on_unimplemented`

Rollup merge of #53271 - llogiq:simplify-maybe-map, r=joshtriplett

use ? to simplify `TransitiveRelation.maybe_map`

I think this looks much clearer than the original.

Rollup merge of #52453 - srijs:fix-52436, r=TimNN

improve diagnostics for tests with custom return values

This is an attempt at getting the ball rolling to improve the diagnostics for test functions that return custom `impl Termination` values (see #52436).

An alternative could be to use `eprintln!`, but including this in the panic message felt nicely consistent with how failing test assertions would be reported.

Let me know what you think!

Auto merge of #53164 - davidtwco:issue-52663-span-decl-captured-variables, r=nikomatsakis

Provide span for declaration of captured variables

Part of #52663.

r? @nikomatsakis

Tidy: ignore non-Markdown files when linting for the Unstable Book

Remove outdated Unstable Book sections

Visit all attributes for feature collection

Fix since of Iterator::flatten to be a proper semver

Make cloudapi enums #[non_exhaustive]

Make std::io::Error #[non_exhaustive]

Make proc_macro Level #[non_exhaustive]

Make core::sync::atomic::Ordering #[non_exhaustive]

Clarify unused_as_mut_slices

Auto merge of #53237 - overdrivenpotato:wasm-export-table, r=alexcrichton

Export WASM table by default

This allows loading a rust-generated `.wasm` binary in a host and using the exported table much like the `memory` export.

Fix links' color

Use CGU name as LLVM module name and add some caching to CGU name generation.

Auto merge of #53133 - Zoxc:gen-int, r=eddyb

Record adjustments and original type for expressions in the generator interior

Fixes https://github.com/rust-lang/rust/issues/50878 and https://github.com/rust-lang/rust/issues/52398.

r? @eddyb

Two small improvements

In `librustc_apfloat/ieee.rs`, use the iterator.[r]find methods to
simplify the code. In `libserialize/json.rs`, make use of the fact
that `Vec.last` on an empty `Vec` returns `None` to simplify the
code to a single match.

Fix some run-make tests after object file naming has changed.

Adapt codegen-unit tests to new CGU naming scheme.

Clean up CodegenUnit name generation.

Auto merge of #53367 - ehuss:update-cargo, r=alexcrichton

Update cargo

- Update transitioning url (rust-lang/cargo#5889)
- Resolve some clippy lint warnings (rust-lang/cargo#5884)
- Don't kill child processes on normal exit on Windows (rust-lang/cargo#5887)
- fix a bunch of clippy warnings (rust-lang/cargo#5876)
- Add support for rustc's --error-format short (rust-lang/cargo#5879)
- Support JSON with rustdoc. (rust-lang/cargo#5878)
- Fix rustfmt instructions in CONTRIBUTING.md (rust-lang/cargo#5880)
- Allow `cargo run` in workspaces. (rust-lang/cargo#5877)
- Change target filters in workspaces. (rust-lang/cargo#5873)
- Improve verbose console and log for finding git repo in package check (rust-lang/cargo#5858)
- Meta rename (rust-lang/cargo#5871)
- fetch: skip target tests when cross_compile is disabled (rust-lang/cargo#5870)
- Fully capture rustc and rustdoc output when -Zcompile-progress is passed (rust-lang/cargo#5862)
- Fix test --example docs. (rust-lang/cargo#5867)
- Add a feature to build a vendored OpenSSL (rust-lang/cargo#5865)

fix error for unsized packed struct field

Auto merge of #53212 - sunjay:nll-raw-cast, r=nikomatsakis

NLL - Prevent where clauses from extending the lifetime of bindings

Fixes https://github.com/rust-lang/rust/issues/53123

r? @nikomatsakis

Update cargo

pretty printing for btreemap

std: Use target_pointer_width for BACKTRACE_ELF_SIZE

The former code used `target.contains("64")` to detect Elf64 targets,
but this is inaccurate in a few cases:

- `s390x-unknown-linux-gnu` is 64-bit
- `sparcv9-sun-solaris` is 64-bit
- `x86_64-unknown-linux-gnux32` is 32-bit

Instead the `std` build script can use `CARGO_CFG_TARGET_POINTER_WIDTH`
to reliably detect 64-bit targets for libbacktrace.

Also update to backtrace-sys 0.1.24 for alexcrichton/backtrace-rs#122.

Auto merge of #52716 - tromey:rustup-lldb, r=alexcrichton

Add lldb to the build

This optionally adds lldb (and clang, which it needs) to the build.

Because rust uses LLVM 7, and because clang 7 is not yet released, a
recent git master version of clang is used.

The lldb that is used includes the Rust plugin.

lldb is only built when asked for, or when doing a nightly build on
macOS.  Only macOS is done for now due to difficulties with the Python
dependency.

Add doc examples for std::alloc::{alloc,alloc_zeroed}.

Account for warning in existing GAT tests

Warn when `generic_associated_types` feature gate is enabled

Add lldb to the build

This optionally adds lldb (and clang, which it needs) to the build.

Because rust uses LLVM 7, and because clang 7 is not yet released, a
recent git master version of clang is used.

The lldb that is used includes the Rust plugin.

lldb is only built when asked for, or when doing a nightly build on
macOS.  Only macOS is done for now due to difficulties with the Python
dependency.

Corrected typing description in doc

Auto merge of #52928 - Mark-Simulacrum:borrowck-cleanup, r=cramertj

(old) borrowck cleanup

Primarily moves dataflow code based on CFG to borrowck; this is mostly so we don't forget to delete it once it becomes unused after we fully move to MIR borrowck.

Updated the most glaring instances of weak tests w.r.t. NLL that came from #53196.

See also the bulletpoint list on #53351.

Auto merge of #52936 - felixrabe:patch-1, r=alexcrichton

Document #39364 – Panic in mpsc::Receiver::recv_timeout

I can still reproduce #39364 with the example code at https://github.com/rust-lang/rust/issues/39364#issuecomment-320637702.

I'm opening this PR in an attempt to document this bug as a known issue in [libstd/sync/mpsc/mod.rs](https://github.com/rust-lang/rust/blob/master/src/libstd/sync/mpsc/mod.rs).

Inputs very much welcome. ([Nightly docs for `recv_timeout`.](https://doc.rust-lang.org/nightly/std/sync/mpsc/struct.Receiver.html?search=#method.recv_timeout))

Ignore test that fails on stage1

syntax: Enforce attribute grammar in the parser

Update former compile-fail tests

Auto merge of #53361 - Manishearth:clippyup, r=nrc

Update clippy

r? @kennytm @nrc

Name the right expected and actual return lifetimes

Update tests

Tweek best_blame_constraint to slightly prefer specific explanations

Change "return" to "returning this value" when explaining free regions

Treat assigning the destination of a call as an assignment

Make assigning to the return type interesting

Adddressed #51602

Clarify dst condition

Don't drop values in other, just move the tail

Update clippy

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

Rollup of 11 pull requests

Successful merges:

 - #53112 (pretty print BTreeSet)
 - #53208 (Don't panic on std::env::vars() when env is null.)
 - #53226 (driver: set the syntax edition in phase 1)
 - #53229 (Make sure rlimit is only ever increased)
 - #53233 (targets: aarch64: Add bare-metal aarch64 target)
 - #53239 (rustc_codegen_llvm: Restore the closure env alloca hack for LLVM 5.)
 - #53246 (A few cleanups)
 - #53257 (Idiomatic improvements to IP method)
 - #53274 (Remove statics field from CodegenCx)
 - #53290 (Make LLVM emit assembly comments with -Z asm-comments)
 - #53317 (Mark prior failure to avoid ICE)

`{to,from}_{ne,le,be}_bytes` for unsigned integer types

Same as https://github.com/rust-lang/rust/pull/51919 did for signed integers.

Tracking issue: https://github.com/rust-lang/rust/issues/52963

Improved how upvars are detected when presenting errors using prefixes.

Rollup merge of #53317 - estebank:abolish-ice, r=oli-obk

Mark prior failure to avoid ICE

Fix #53251

Rollup merge of #53290 - whitequark:fix-35741, r=nagisa

Make LLVM emit assembly comments with -Z asm-comments

Fixes #35741, and makes `-Z asm-comments` actually do something useful.

Before:
```
.section .text.main,"ax",@progbits
.globl main
.p2align 4, 0x90
.type main,@function
main:
.cfi_startproc
pushq %rax
.cfi_def_cfa_offset 16
movslq %edi, %rax
leaq _ZN1t4main17he95a7d4f1843730eE(%rip), %rdi
movq %rsi, (%rsp)
movq %rax, %rsi
movq (%rsp), %rdx
callq _ZN3std2rt10lang_start17h3121da83b2bc3697E
movl %eax, %ecx
movl %ecx, %eax
popq %rcx
.cfi_def_cfa_offset 8
retq
.Lfunc_end8:
.size main, .Lfunc_end8-main
.cfi_endproc
```

After:
```
.section .text.main,"ax",@progbits
.globl main                    # -- Begin function main
.p2align 4, 0x90
.type main,@function
main:                                   # @main
.cfi_startproc
# %bb.0:
pushq %rax
.cfi_def_cfa_offset 16
movslq %edi, %rax
leaq _ZN1t4main17he95a7d4f1843730eE(%rip), %rdi
movq %rsi, (%rsp)            # 8-byte Spill
movq %rax, %rsi
movq (%rsp), %rdx            # 8-byte Reload
callq _ZN3std2rt10lang_start17h3121da83b2bc3697E
movl %eax, %ecx
movl %ecx, %eax
popq %rcx
.cfi_def_cfa_offset 8
retq
.Lfunc_end8:
.size main, .Lfunc_end8-main
.cfi_endproc
                                        # -- End function
```

Rollup merge of #53274 - bjorn3:remove_statics_field, r=nagisa

Remove statics field from CodegenCx

It doesnt seem to be used anywhere.

Rollup merge of #53257 - faern:ip-method-idiomatic-improvement, r=TimNN

Idiomatic improvements to IP method

Since match ergonomics and slice patterns are stable this might be more idiomatic modern Rust implementations of these methods? Or well, slice patterns with `..` are not stabilized yet, so maybe we want to specify all fields but with `_`?

Rollup merge of #53246 - ljedrz:cleanup_various, r=kennytm

A few cleanups

- change `skip(1).next()` to `nth(1)`
- collapse some `if-else` expressions
- remove a few explicit `return`s
- remove an unnecessary field name
- dereference once instead of matching on multiple references
- prefer `iter().enumerate()` to indexing with `for`
- remove some unnecessary lifetime annotations
- use `writeln!()` instead of `write!()`+`\n`
- remove redundant parentheses
- shorten some enum variant names
- a few other cleanups suggested by `clippy`

Rollup merge of #53239 - cuviper:llvm5-closure-alloca, r=eddyb

rustc_codegen_llvm: Restore the closure env alloca hack for LLVM 5.

This hack was removed in #50949, but without it I found that building
`std` with full debuginfo would print many LLVM `DW_OP_LLVM_fragment`
errors, then die `LLVM ERROR: Failed to strip malformed debug info`.

It doesn't seem to be a problem for LLVM 6, so we can re-enable the hack
just for older LLVM.

This reverts commit da579ef75e4a8ca11fb98b24a0a3ea0c7ccffeeb.
Fixes #53204.
r? @eddyb

Rollup merge of #53233 - andre-richter:master, r=alexcrichton

targets: aarch64: Add bare-metal aarch64 target

A generic AArch64 target that can be used for writing bare-metal code
for 64-bit ARM architectures.

Rollup merge of #53229 - varkor:rlimits_min, r=nikomatsakis

Make sure rlimit is only ever increased

`libc::setrlimit` will fail if we try to set the rlimit to a value lower than it is currently, so make sure we're never trying to do this. Fixes #52801.

Rollup merge of #53226 - QuietMisdreavus:editions-for-all, r=estebank

driver: set the syntax edition in phase 1

Fixes https://github.com/rust-lang/rust/issues/53203

It seems the way libsyntax handles the desired edition is to use a global, set via `syntax_pos::hygiene::set_default_edition`. Right now, this is set in the driver in `run_compiler`, which is the entry point for running the compiler all the way through to emitting files. Since rustdoc doesn't use this function, it wasn't properly setting this global. (When initially setting up editions in rustdoc, i'd assumed that setting `sessopts.edition` would have done this... `>_>`) This was "fixed" for doctests in https://github.com/rust-lang/rust/pull/52385, but rather than patching in a call to `set_default_edition` in all the places rustdoc sets up the compiler, i've instead moved the call in the driver to be farther in the process. This means that any use of `phase_1_parse_input` with the right session options will have the edition properly set without having to also remember to set libsyntax up separately.

r? @rust-lang/compiler

Rollup merge of #53208 - BurntPizza:protect-the-environment, r=alexcrichton

Don't panic on std::env::vars() when env is null.

Fixes #53200.

Reviewer(s):
* Do I need to do any `#[cfg()]` here?
* Is this use of libc ok for a dev-dependency?

Rollup merge of #53112 - fukatani:pretty-print-btreeset, r=michaelwoerister

pretty print BTreeSet

I want pretty printing for BTreeSet.
```rust
use std::collections::*;

fn main() {
  let mut s = BTreeSet::new();
  s.insert(5);
  s.insert(3);
  s.insert(7);
  s.remove(&3);
  println!("{:?}", s);
}
```

```
(gdb) b 9
(gdb) p s
$1 = BTreeSet<i32> with 2 elements = {[0] = 5, [1] = 7}
```
This is analogy of pretty printing for C++ std::set.

Label definition of captured variables in errors.

Added some debug logging.

Auto merge of #53085 - ljedrz:cleanup_syntax_structures, r=ljedrz

Move SmallVector and ThinVec out of libsyntax

- move `libsyntax::util::SmallVector` tests to `librustc_data_structures::small_vec`
- remove `libsyntax::util::SmallVector`
- move `libsyntax::util::thin_vec` to `librustc_data_structures::thin_vec`

Other than moving these data structures where they belong it allows modules using `SmallVector<T>` (`SmallVec<[T; 1]>`) to specify their own length (e.g. 8 or 32) independently from `libsyntax`.

Add regression test for issue #30104

#30104

Add regression test for issue #48697

#48697

add copyright from template