Inline move_path_for_projection inside move_path_for

Make eval_place iterate instead of recurse

Add comment to explain why we change the layout for Projection

Turn ICE on type arguments on variables into an error

review comments: move back some methods and clean up wording

Move some methods to `diagnostics.rs` away from `parser.rs`

Move a bunch of error recovery methods to `diagnostics.rs` away from
`parser.rs`.

Tweak `self` arg not as first argument of a method diagnostic

Mention that `self` is only valid on "associated functions"
```
error: unexpected `self` argument in function
  --> $DIR/self-in-function-arg.rs:1:15
   |
LL | fn foo(x:i32, self: i32) -> i32 { self }
   |               ^^^^ not valid as function argument
   |
   = note: `self` is only valid as the first argument of an associated function
```

When it is a method, mention it must be first
```
error: unexpected `self` argument in function
  --> $DIR/trait-fn.rs:4:20
   |
LL |     fn c(foo: u32, self) {}
   |                    ^^^^ must be the first associated function argument
```

Reword malformed attribute input diagnostics

- Handle empty `cfg_attr` attribute
- Reword empty `derive` attribute error
- Use consistend error message: "malformed `attrname` attribute input"
- Provide suggestions when possible
- Move note/help to label/suggestion
- Use consistent wording "ill-formed" -> "malformed"
- Move diagnostic logic out of parser

Add support for suggesting as_ref to Result accesses

Auto merge of #59276 - oli-obk:cleanups, r=eddyb

Cleanup (pretty) printing of `ty::Const`

r? @eddyb

addresses https://github.com/rust-lang/rust/pull/59178/files/5cd28066214a4a255bac6655c20bf05cb95730fd#r266243236 and https://github.com/rust-lang/rust/pull/59178/files/5cd28066214a4a255bac6655c20bf05cb95730fd#r266243336

Update nll ui tests

Add additional trace statements to the const propagator

This makes it easier to figure out when const propagation fails.

Add clippy and fix commands to x.py

Auto-derive Encode and Decode implementations

Auto merge of #61151 - Centril:rollup-5rpyhfo, r=Centril

Rollup of 6 pull requests

Successful merges:

 - #61092 (Make sanitize_place iterate instead of recurse)
 - #61093 (Make borrow_of_local_data iterate instead of recurse)
 - #61094 (Make find_local iterate instead of recurse)
 - #61099 (Make ignore_borrow iterate instead of recurse)
 - #61103 (Make find iterate instead of recurse)
 - #61104 (Make eval_place_to_op iterate instead of recurse)

Failed merges:

r? @ghost

shadow as_ptr as as_mut_ptr in Vec to avoid going through Deref

add test checking that Vec push/pop does not invalidate pointers

Fix missing tcx

Only print integers in symbol path's constants

Update ui tests

Print types for unevaluated constants

Fix rebase fallout

Break cycle during array length printing

rustc: integrate ty::Const into ty::print as print_const.

Update ui tests

Print unevaluted constants as `_` or as their source representation

Use `write_char` to skip the formatting infrastructure

Fix tidy

Print generic args in function calls in MIR

Group common printing code during constant pretty printing

Render unresolved anon consts like closures

`u8` is printed as a number, not a character

Render const byte slices in MIR

Make `ConstValue::Slice` solely take `[u8]` and `str`

Refactor string constant printing to prep for byte string printing

Add test showing how byte slices are printed in MIR

Don't use `ty::Const` without immediately interning

Print const chars escaped with surrounding quotes

Merge the string printing paths of ty::Const

There's a tcx in scope, don't use the tls one

Reuse the pretty printing architecture for printing of constants

Ignore .vscode even if it is a symlink

Remove unnecessary secondary recursion

MaybeUninit doctest: remove unnecessary type ascription

split core::ptr module into multiple files

Auto merge of #61150 - Centril:rollup-wmm7qga, r=Centril

Rollup of 13 pull requests

Successful merges:

 - #61026 (Tweak macro parse errors when reaching EOF during macro call parse)
 - #61095 (Update cargo)
 - #61096 (tidy: don't short-circuit on license error)
 - #61107 (Fix a couple docs typos)
 - #61110 (Revert edition-guide toolstate override)
 - #61111 (Fixed type-alias-bounds lint doc)
 - #61113 (Deprecate `FnBox`. `Box<dyn FnOnce()>` can be called directly, since 1.35)
 - #61116 (Remove the incorrect warning from README.md)
 - #61118 (Dont ICE on an attempt to use GAT without feature gate)
 - #61121 (improve debug-printing of scalars)
 - #61125 (Updated my mailmap entry)
 - #61134 (Annotate each `reverse_bits` with `#[must_use]`)
 - #61138 (Move async/await tests to their own folder)

Failed merges:

r? @ghost

Rollup merge of #61104 - spastorino:eval-place-to-op-iterate, r=oli-obk

Make eval_place_to_op iterate instead of recurse

r? @oli-obk

Rollup merge of #61103 - spastorino:find-iterate, r=oli-obk

Make find iterate instead of recurse

r? @oli-obk

Rollup merge of #61099 - spastorino:ignore-borrow-iterate, r=oli-obk

Make ignore_borrow iterate instead of recurse

r? @oli-obk

Rollup merge of #61094 - spastorino:find-local-iterate, r=oli-obk

Make find_local iterate instead of recurse

r? @oli-obk

Rollup merge of #61093 - spastorino:borrow-of-local-data-iterate, r=oli-obk

Make borrow_of_local_data iterate instead of recurse

r? @oli-obk

Rollup merge of #61092 - spastorino:sanitize-place-iterative, r=oli-obk

Make sanitize_place iterate instead of recurse

r? @oli-obk

Rollup merge of #61138 - varkor:async-await-tests, r=cramertj

Move async/await tests to their own folder

This moves run-pass and ui async/await tests to their own folder `src/test/ui/async-await` and organises some into subfolders. (It does not move rustdoc tests for async/await.)

I also did some drive-by cleaning up of issues/error code tests into their own folders (which already existed). These are in separate commits, so easy to separate out if that's more desirable.

r? @cramertj

Rollup merge of #61134 - nvzqz:reverse_bits-must_use, r=varkor

Annotate each `reverse_bits` with `#[must_use]`

Because the name sounds like an in-place mutation like `[T]::reverse(&mut self)`, it may be confused for one.

This change was requested at https://github.com/rust-lang/rust/issues/48763#issuecomment-493743741.

Rollup merge of #61125 - XAMPPRocky:master, r=jonas-schievink

Updated my mailmap entry

Rollup merge of #61121 - RalfJung:miri-value-printing, r=oli-obk

improve debug-printing of scalars

* Prettier printing for `Pointer` and `Bits`.
* Don't print the `ScalarMaybeUndef(...)` around `Scalar`.

Before: `Immediate(ScalarMaybeUndef(Scalar(Ptr(Pointer { alloc_id: AllocId(3401), offset: Size { raw: 4 }, tag: Tagged(7723) }))))`
After: `Immediate(Scalar(AllocId(3401).0x4[<7723>]))`

Before: `Immediate(ScalarMaybeUndef(Scalar(Bits { size: 8, bits: 10 })))`
After: `Immediate(Scalar(0x000000000000000A))`

Before: `Immediate(ScalarMaybeUndef(Scalar(Bits { size: 1, bits: 1 })))`
After: `Immediate(Scalar(0x01))`

r? @oli-obk

Rollup merge of #61118 - pnkfelix:issue-60654-dont-ice-on-gat, r=varkor

Dont ICE on an attempt to use GAT without feature gate

Fix #60654

Rollup merge of #61116 - scottmcm:vcpp-download-link, r=alexcrichton

Remove the incorrect warning from README.md

My problem was user error; thanks to @Zoxc for fixing me!

r? @alexcrichton

Rollup merge of #61113 - SimonSapin:fnbox, r=alexcrichton

Deprecate `FnBox`. `Box<dyn FnOnce()>` can be called directly, since 1.35

FCP completion: https://github.com/rust-lang/rust/issues/28796#issuecomment-439731515

Rollup merge of #61111 - Cerberuser:patch-1, r=steveklabnik

Fixed type-alias-bounds lint doc

The example code under type-alias-bounds lint produced two warnings - one from the lint itself and another from the dead_code lint, and only the second one was in the doc. This looked like an error, so I've added `#[allow(dead_code)]` and replaced the example output with the expected one.

[Playground](https://play.rust-lang.org/?version=stable&mode=debug&edition=2018&code=%23%5Ballow(dead_code)%5D%0Atype%20SendVec%3CT%3A%20Send%3E%20%3D%20Vec%3CT%3E%3B)

According to guidelines,
r? @steveklabnik

Rollup merge of #61110 - ehuss:revert-edition-override, r=Mark-Simulacrum

Revert edition-guide toolstate override

Closes #60929

Rollup merge of #61107 - blkerby:docs_typos, r=Centril

Fix a couple docs typos

Also add a link to env::split_paths.

Rollup merge of #61096 - ehuss:tidy-license-short-circuit, r=Centril

tidy: don't short-circuit on license error

If there is more than one license error, tidy would only print the first
error. This changes it so that all license errors are printed.

Rollup merge of #61095 - ehuss:update-cargo, r=alexcrichton

Update cargo

Update cargo

14 commits in c4fcfb725b4be00c72eb9cf30c7d8b095577c280..545f354259be4e9745ea00a524c0e4c51df01aa6
2019-05-15 19:48:47 +0000 to 2019-05-23 17:45:30 +0000
- Bump to 0.38.0 (rust-lang/cargo#6979)
- cargo package: detect new empty directories (rust-lang/cargo#6973)
- Add message caching. (rust-lang/cargo#6933)
- Fix typo (rust-lang/cargo#6974)
- Set `Finished` line correctly for debug=0. (rust-lang/cargo#6971)
- Clippy fixes (rust-lang/cargo#6970)
- Remove rustdoc `can_add_color_process`. (rust-lang/cargo#6968)
- Document new `doctest` field. (rust-lang/cargo#6965)
- Update some man pages that missed --offline. (rust-lang/cargo#6964)
- add public & private prop tests. (rust-lang/cargo#6962)
- zsh completion: Pull list of commands from cargo --list (rust-lang/cargo#6956)
- Change docs "inequality" for semver requirement. (rust-lang/cargo#6963)
- Update im-rc requirement from 12.1.0 to 13.0.0 (rust-lang/cargo#6959)
- Add `doctest` field into metadata (rust-lang/cargo#6953)

Rollup merge of #61026 - estebank:macro-eof-spans, r=petrochenkov

Tweak macro parse errors when reaching EOF during macro call parse

Add detail on origin of current parser when reaching EOF, stop saying "found `<eof>`" and point at the end of macro calls.

Fix #27569.

Fix spelling in release notes

When encountering move error on an `Option`, suggest using `as_ref`

Suggest borrowing for loop head on move error

Auto merge of #60441 - vext01:try-to-kill-projection-params, r=oli-obk

Make place projections concrete.

**I'm not sure if we want this. I'm raising the PR  for discussion**

Whilst doing some work on our Rust fork, I noticed the following:

Once upon a time (commit 9bd35c07c26) there were two kinds of
projection: one for places, and one for constants. It therefore made
sense to share the `Projection` struct for both. Although the different
use-cases used different concrete types, sharing was made possible by
type-parameterisation of `Projection`.

Since then, however, the usage of projections in constants has
disappeared, meaning that (forgetting lifetimes for a moment) the
parameterised type is only every instantiated under one guise. So it may
as well be a concrete type. Right?

What do people think? This is entirely untested, although it does check.

If we *don't* want this, then we should at least update the incorrect comment against `Projection`.

Thanks

Add extra arc_wake

Revert changes that belong to separate PR

Make eval_place_to_op iterate instead of recurse

Make eval_place_to_mplace take PlaceBase::Static

Delete stray .stderr

Move error code tests to error code folder

Add issues folder in async-await

Add drop-order folder in test/ui/async-await

Add auxiliary issue file

Move some issues into the issues folder

Move async/await tests to test/ui/async-await

Annotate each `reverse_bits` with `#[must_use]`

Because the name sounds like an in-place mutation like
`[T]::reverse(&mut self)`, it may be confused for one.

Auto merge of #60777 - pietroalbini:azure-pipelines, r=alexcrichton

Add Azure Pipelines configuration

Huge thanks to ＠johnterickson and ＠willsmythe for writing the initial config! :heart:
I applied some changes to the initial config and disabled most of the builders since we're not going to run all of them during the initial step for the evaluation.

[More details about our plans for the Azure Pipelines evaluation.](https://internals.rust-lang.org/t/update-on-the-ci-investigation/10056)

r? ＠alexcrichton ＠kennytm
cc ＠rust-lang/infra ＠ethomson ＠rylev

Avoid catch-all arm, use Base(PlaceBase::Static(_))

Move diagnostic logic out of parser

Fix rebase

review comments

Tweak macro parse errors when reaching EOF during macro call parse

- Add detail on origin of current parser when reaching EOF and stop
  saying "found <eof>" and point at the end of macro calls
- Handle empty `cfg_attr` attribute
- Reword empty `derive` attribute error

Make ignore_borrow iterate instead of recurse

Make borrow_of_local_data iterate instead of recurse

Updated my mailmap entry

Make sanitize_place iterate instead of recurse

Make find iterate instead of recurse

this is for tidy

improve debug-printing of scalars

Before: Immediate(ScalarMaybeUndef(Scalar(Ptr(Pointer { alloc_id: AllocId(3401), offset: Size { raw: 4 }, tag: Tagged(7723) }))))
After: Immediate(Scalar(AllocId(3401).0x4[<7723>]))

Before: Immediate(ScalarMaybeUndef(Scalar(Bits { size: 8, bits: 10 })))
After: Immediate(Scalar(0x000000000000000A))

Before: Immediate(ScalarMaybeUndef(Scalar(Bits { size: 1, bits: 1 })))
After: Immediate(Scalar(0x01))

Regression test for issue #60654.

Make place projections concrete.

Once upon a time (commit 9bd35c07c26) there were two kinds of
projection: one for places, and one for constants. It therefore made
sense to share the `Projection` struct for both. Although the different
use-cases used different concrete types, sharing was made possible by
type-parameterisation of `Projection`.

Since then, however, the usage of projections in constants has
disappeared, meaning that (forgetting lifetimes for a moment) the
parameterised type is only every instantiated under one guise. So it may
as well be a concrete type.

Auto merge of #60568 - petrochenkov:debi, r=Mark-Simulacrum

rustbuild: Simplify debuginfo configuration

This is supposed to fix https://github.com/rust-lang/rust/issues/52179

This PR introduces one option `debuginfo-level` replacing `debuginfo` and `debuginfo-lines` and corresponding to the `rustc` flag `-C debuginfo=N`.

`debuginfo-level` serves as a default for all Rust code built during bootstrap, but it can be overridden for specific subsets of code using finer-grained options `debuginfo-level-{rustc,std,tools,tests}` replacing `debuginfo-only-std`, `debuginfo-tools` and `debuginfo-tests`.

Delay ICE in early_free_scope so feature gate has chance to stop compilation cleanly.

Delay ICE in fold_region so feature gate has chance to stop compilation cleanly.

Don't arena-allocate static symbols.

It's just a waste of memory. This also gets rid of the special case for
"".