Rollup merge of #69200 - jonas-schievink:yield-print, r=eddyb,Zoxc

Fix printing of `Yield` terminator

Addresses the bug found in https://github.com/rust-lang/rust/issues/69039#issuecomment-586633495

Rollup merge of #69192 - JohnTitor:add-tests, r=Centril

Add more regression tests

Closes #39618
Closes #51798
Closes #62894
Closes #63952
Closes #68653

r? @Centril

Rollup merge of #69181 - skinny121:const-eval-return, r=oli-obk

Change const eval to just return the value

As discussed in https://github.com/rust-lang/rust/pull/68505#discussion_r370956535, the type of consts shouldn't be returned from const eval queries.

r? @eddyb
cc @nikomatsakis

Auto merge of #69234 - JohnTitor:clippy-up, r=JohnTitor

Update Clippy

Fixes #69221

r? @ghost

Clean up E0310 explanation

Auto merge of #69258 - JohnTitor:rollup-n2hljai, r=JohnTitor

Rollup of 8 pull requests

Successful merges:

 - #67272 (recursion_limit parsing handles overflows)
 - #68597 (Simplify `Skip::nth` and `Skip::last` implementations)
 - #68767 (macOS: avoid calling pthread_self() twice)
 - #69175 (Do not ICE when encountering `yield` inside `async` block)
 - #69223 (Ignore GDB versions with broken str printing.)
 - #69244 (configure: set LLVM flags with a value)
 - #69249 (Stabilize {f32, f64}::{LOG2_10, LOG10_2})
 - #69252 (Clean out unused directories for extra disk space)

Failed merges:

r? @ghost

Rollup merge of #69252 - Mark-Simulacrum:disk-try, r=pietroalbini

Clean out unused directories for extra disk space

This cleans out some of the unused (but large) directories on our linux builders to hopefully allow them to complete without running out of disk space.

Rollup merge of #69249 - LeSeulArtichaut:stabilize-logs-consts, r=sfackler

Stabilize {f32, f64}::{LOG2_10, LOG10_2}

Following the decision to stabilize `LOG2_10` and `LOG10_2` in https://github.com/rust-lang/rust/issues/50540#issuecomment-536627588.

Closes #50540.
r? @sfackler

Rollup merge of #69244 - cuviper:llvm-flags, r=Mark-Simulacrum

configure: set LLVM flags with a value

Rather than a boolean `--enable-cflags` etc., these options should
reflect that they are for LLVM, and that they need a value. You would
now use `./configure --llvm-cflags="..."`.

Rollup merge of #69223 - spunit262:empty-string-gdb-version, r=Dylan-DPC

Ignore GDB versions with broken str printing.

https://sourceware.org/bugzilla/show_bug.cgi?id=22236

Rollup merge of #69175 - estebank:shall-not-ice, r=petrochenkov

Do not ICE when encountering `yield` inside `async` block

Fix #67158.

Rollup merge of #68767 - kubo39:patch-macos, r=shepmaster

macOS: avoid calling pthread_self() twice

Rollup merge of #68597 - ollie27:skip_nth_last, r=Amanieu

Simplify `Skip::nth` and `Skip::last` implementations

The main improvement is to make `last` no longer recursive.

Rollup merge of #67272 - fisherdarling:master, r=varkor,hellow554

recursion_limit parsing handles overflows

This PR adds overflow handling to `#![recursion_limit]` attribute parsing. If parsing the given value results in an `IntErrorKind::Overflow`, then the recursion_limit is set to `usize::max_value()`.

closes #67265

Clean out some default-installed directories

This helps us have enough disk space for our builders to be able to complete
successfully. For now, the choices are ad-hoc and 'definitely not needed'. This
should never fail the build, as everything our build needs should be inside
Docker.

Inline various simple `emit_*` and `read_*` methods in `Decoder`.

Mostly, these are the ones whose body just contains `f(self)`.

Inline `Symbol::decode` and `Interner::intern`.

Always inline `run_utf8_validation`.

It only has two call sites, and the one within `from_utf8` is hot within
rustc itself.

Fix typo

Fix typo in comment

Use doc comment for explanation of `shared_borrow_allows_mutation`

Don't eliminate frame pointers on thumb targets

Add a test for proc macro generating `$ IDENT`

parser: Remove `Option`s from unnormalized tokens

They are always set synchronously with normalized tokens now

parser: Set previous and unnormalized tokens in couple more places

parser: Do not call `bump` recursively

Token normalization is merged directly into `bump`.
Special "unknown macro variable" diagnostic for unexpected `$`s is removed as preventing legal code from compiling.

configure: set LLVM flags with a value

Rather than a boolean `--enable-cflags` etc., these options should
reflect that they are for LLVM, and that they need a value. You would
now use `./configure --llvm-cflags="..."`.

Querify object_safety_violations.

parse: recover `mut (x @ y)` as `(mut x @ mut y)`.

Do not emit note suggesting to implement trait to foreign type

Update tests

Extend to other operations

Refractor check in a separate function

Fix more tests

Update Clippy

Auto merge of #69129 - Centril:macro-legacy-errors, r=petrochenkov

Transition macro_legacy_warnings into a hard error

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

r? @petrochenkov

ci: switch macOS builders to 10.15

ast: add a FIXME

Auto merge of #69226 - JohnTitor:rollup-syn03oj, r=JohnTitor

Rollup of 6 pull requests

Successful merges:

 - #68495 (Updating str.chars docs to mention crates.io.)
 - #68701 (Improve #Safety of various methods in core::ptr)
 - #69158 (Don't print block exit state in dataflow graphviz if unchanged)
 - #69179 (Rename `FunctionRetTy` to `FnRetTy`)
 - #69186 ([tiny] parser: `macro_rules` is a weak keyword)
 - #69188 (Clean up E0309 explanation)

Failed merges:

r? @ghost

Rollup merge of #69188 - GuillaumeGomez:clean-up-e0309, r=Dylan-DPC

Clean up E0309 explanation

r? @Dylan-DPC

Rollup merge of #69186 - petrochenkov:kwrules, r=Centril

[tiny] parser: `macro_rules` is a weak keyword

r? @Centril

Rollup merge of #69179 - JohnTitor:rename-to-fnretty, r=Centril

Rename `FunctionRetTy` to `FnRetTy`

As per FIXME comment

r? @Centril

Rollup merge of #69158 - ecstatic-morse:graphviz-diff, r=matthewjasper

Don't print block exit state in dataflow graphviz if unchanged

A small quality-of-life improvement I was using while working on #68528. It's pretty common to have a lot of zero-statement basic blocks, especially before a `SimplifyCfg` pass is run. When the dataflow state was dense, these blocks could take up a lot of vertical space since the full flow state was printed on both entry and exit. After this PR, we only print a block's exit state if it differs from that block's entry state. Take a look at the two basic blocks on the left.

Before:


After:

Rollup merge of #68701 - amosonn:patch-2, r=RalfJung

Improve #Safety of various methods in core::ptr

For `read`, `read_unaligned`,`read_volatile`, `replace`, and `drop_in_place`:

- The value they point to must be properly initialized

For `replace`, additionally:

- The pointer must be readable

Rollup merge of #68495 - sdegutis:patch-1, r=Mark-Simulacrum

Updating str.chars docs to mention crates.io.

This might spare someone else a little time searching the stdlib for unicode/grapheme support.

Auto merge of #68781 - ssomers:btree_miri_relief, r=RalfJung

BTree: lighten the load on Miri

Reduce the amount of work Miri ploughs through in btree code, in particular on `test_clone_from` (which takes up 5 minutes on my machine).

r? @crgl

Do not ICE when encountering `yield` inside `async` block

Select an appropriate unused lifetime name in suggestion

Rename `FunctionRetTy` to `FnRetTy`

Handle recursion_limit parsing errors

Ignore GDB versions with broken str printing.

https://sourceware.org/bugzilla/show_bug.cgi?id=22236

Auto merge of #67953 - cjgillot:split_infer, r=Zoxc

Split librustc::{traits,infer} to a separate crate rustc_infer

This is still very much work in progress.
Three functions are between dimensions (at the end of `rustc::traits`), waiting for some dependency breaking scheme.
Please tell me if the approach seems sound, and how you would like to split this PR up.

The formatting is deliberately off, to ease rebasing.

cc #65031

Lighten tests, in particular for Miri, yet test and explain more

Fix comments outdated during #66648

Stabilize {f32, f64}::{LOG2_10, LOG10_2}

Auto merge of #67885 - tobithiel:fix_group_lint_allow_override, r=Mark-Simulacrum

rustc_session: allow overriding lint level of individual lints from a group

Fixes #58211 and fixes rust-lang/rust-clippy#4778 and fixes rust-lang/rust-clippy#4091

Instead of hard-coding the lint level preferences (from lowest to highest precedence: `lint::Allow -> lint::Warn -> lint::Deny -> lint::Forbid`), the position of the argument in the command line gets taken into account.

Examples:
1. Passing `-D unused -A unused-variables` denies everything in the lint group `unused` **except** `unused-variables` which is explicitly allowed.
1. Passing `-A unused-variables -D unused` denies everything in the lint group `unused` **including** `unused-variables` since the allow is specified before the deny (and therefore overridden by the deny).

This matches the behavior that is already being used when specifying `allow`/`deny` in the source code.

Allow whitespaces in revision flags

Auto merge of #68997 - Zoxc:pure-node-id, r=petrochenkov

Panic if NodeIds are used for incremental compilation

r? @michaelwoerister

Improve #Safety of core::ptr::drop_in_place

Added missing conditions:
- Valid for writes
- Valid for destructing

suspend -> yield

Prune rustc dependencies.

Gate macro use.

Prune features.

Remove librustc_infer crate re-exports.

Other crates.

Make librustc_traits compile.

Make librustc_mir compile.

macOS: avoid calling pthread_self() twice

Make librustc_typeck compile.

Make librustc_infer compile.

Make librustc compile.

Move librustc/{traits,infer} to librustc_infer.

Auto merge of #69156 - ecstatic-morse:unified-dataflow-impls2, r=eddyb

Use `ResultsCursor` for `elaborate_drops`

Some cleanup after #68241. The old code was using a custom cursor-like struct called `InitializationData`.

Auto merge of #68814 - Aaron1011:fix/proc-macro-order-two, r=petrochenkov

Record proc macro harness order for use during metadata deserialization

Fixes #68690

When we generate the proc macro harness, we now explicitly recorder the
order in which we generate entries. We then use this ordering data to
deserialize the correct proc-macro-data from the crate metadata.

Fix printing of `Yield` terminator

Fix running rustdoc-js test suite individually

Without `Compiletest.path` set running `x.py test src/test/rustdoc-js` would run the `rustdoc-js` test suite with everything filtered out.

As this was the only place setting `Compiletest.path` to `None` this removes the `Option` wrapper as well.

Determine type of intrinsics instead of assuming the destination type is the same.

Auto merge of #68668 - GuillaumeGomez:struct-variant-field-search, r=ollie27

Struct variant field search

Fixes #16017.

Reopening of #64724.

cc @tomjakubowski
cc @ollie27

r? @kinnison

fix incremental tests

ast: tweak AssocItemKind::Macro comment

print_item_const: remove extraneous space

reject assoc statics & extern consts during parsing

Code review changes.

Add additional comment

Record proc macro harness order for use during metadata deserialization

Fixes #68690

When we generate the proc macro harness, we now explicitly recorder the
order in which we generate entries. We then use this ordering data to
deserialize the correct proc-macro-data from the crate metadata.

fix codegen tests

visit: unify extern & assoc item visiting

pprust: unify extern & associated item printing

fuse extern & associated item parsing up to defaultness

ast: make ForeignItemKind an alias of AssocItemKind

parse extern consts

parse associated statics.

ast/parser: fuse `static` & `const` grammars in all contexts.

ast: make `= <expr>;` optional in free statics/consts.

Add FIXME note

remove no-longer-needed test

Add test for issue-68653

Add test for issue-63952

Add test for issue-62894

adjust run-fail tests

Add test for issue-51798

Add test for issue-39618

Panic when hashing node IDs

ast: normalize `ForeignItemKind::Ty` & `AssocItemKind::TyAlias`.