rustc_codegen_llvm: privatize as much of attributes::* as possible.

rustc_codegen_llvm: take an Instance in attributes::from_fn_attrs.

rustc_codegen_llvm: rewrite debuginfo::get_function_signature to use FnAbi.

rustc: take a PolyFnSig instead of an FnSig in FnAbi::of_fn_ptr.

rustc: rename FnAbi::new to FnAbi::of_fn_ptr.

rustc: compute FnAbi's for virtual calls through FnAbi::of_instance.

rustc: use ReifyShim for reifying Virtual call instances.

rustc_codegen_ssa: use FnAbi::of_instance wherever possible.

rustc_codegen_llvm: move NoReturn attribute to apply_attrs_llfn.

rustc_codegen_ssa: take a FnAbi instead of a FnSig in declare_fn.

rustc_codegen_ssa: remove define_fn and define_internal_fn.

rustc_codegen_ssa: clean up lifetimes on TerminatorCodegenHelper's methods.

rustc_codegen_ssa: make codegen_instance a simple wrapper for codegen_mir.

rustc_codegen_ssa: use &'tcx mir::Body<'tcx> instead of &'a ... for the MIR body.

rustc: expose the mir::Body reference lifetime from mir::ReadOnlyBodyCache (#64736 fallout).

rustc_mir: don't hardcode InstanceDef::VtableShim behavior to Adjustment::DerefMove.

rustc: combine Instance::fn_sig_noadjust and Instance::fn_sig.

Auto merge of #66256 - CAD97:patch-2, r=RalfJung

Layout::pad_to_align is infallible

As per [this comment](https://github.com/rust-lang/rust/issues/55724#issuecomment-441421651) (cc @glandium).

> Per https://github.com/rust-lang/rust/blob/eb981a1/src/libcore/alloc.rs#L63-L65, `layout.size()` is always <= `usize::MAX - (layout.align() - 1)`.
>
> Which means:
>
> * The maximum value `layout.size()` can have is already aligned for `layout.align()` (`layout.align()` being a power of two, `usize::MAX - (layout.align() - 1)` is a multiple of `layout.align()`)
> * Incidentally, any value smaller than that maximum value will align at most to that maximum value.
>
> IOW, `pad_to_align` can not return `Err(LayoutErr)`, except for the layout not respecting its invariants, but we shouldn't care about that.

This PR makes `pad_to_align` return `Layout` directly, representing the fact that it cannot fail.

Auto merge of #64736 - Nashenas88:mir_predecessors_cache_cleanup, r=oli-obk

Remove interior mutability in mir predecessors cache

Fix issues caused during rebasing

Fix issues caused during rebasing

Use new HashStable proc macro

Fix tidy issues

Fix rebasing errors, convert some BodyCache::body() calls to reborrows

Fix type errors created during rebasing

Remove unchecked inline attribute, remove unused functions, make chache mod private again

Remove inline attributes that hadn't been profiled, unexport Cache since it no longer needs to be public

Fix type errors cause during rebasing

Compute predecessors in mir_build query and use existing cache for generating ReadOnlyBodyCache, remove unneeded fns

Remove files created during conflict resolution

Fix typos caused during rebase

Remove HasLocalDecls impl from BodyCache's, properly reborrow to Body, rename all body_cache back to body

Remove BodyCache.body and rely on Deref as much as possible for ReadOnlyBodyCache

Undo minor changes that weren't needed, fix one lifetime typo

Fix compilation errors created during rebase

Fix typo

Fix tidy errors

Fix typos caused during rebase

Fix remaining compilation issues

Simplify BodyCache impl and fix all remaining type errors in librustc_mir (lifetime errors still exist)

Fix remaining Body -> (ReadOnly)BodyCache type errors in librustc_mir outside of librustc_mir/transform

Fix typo caused by rebasing

Add predecessors fn to ReadOnlyBodyCache, fix more Body -> (ReadOnly)BodyCache type errors

Add read_only fn to BodyCache<&mut...> impl, fix more Body -> (ReadOnly)BodyCache type errors

Fix a large number of Body -> (ReadOnly)BodyCache type errors, add predecessor_locations fn to ReadOnlyBodyCache

Account for new maybe_sideeffect helper that requires predecessors

Convert &mut to & since the reference didn't need to be mutable

Revert back to using FunctionCx's Body

Add Body back as field of FunctionCx, but under a different lifetime

Improved BodyCache body impl so it only returns a sharable ref, add new body_mut method, fix visit macros, simplify usage in codegen_ssa analyzer

Remove Body from FunctionCx, pass it along during librustc_codegen_ssa

Simplify Cache wrapper to single type, impl Deref on it, fix all compilation errors in librustc_codegen_ssa

Move predecessor cache outside of Body, use wrapper types to manage Cache and Body (WIP, amend this commit)

Move predecessors cache back to its own type

This ensures that the cache can be properly ignored during encoding and decoding.
Fix panics that arose due to lack of encoding

Add back cache invalidation to basic_blocks_and_local_decls_mut

Address nits and remove unneeded pass

Address excessive line length that was triggering warning during linting

Move predecessors cache invalidation back to basic_blocks_mut, add a couple more ensure_predecessors to prevent panics

Ensure predecessors are recomputed at critical points, fixes panics

Address linting errors caught by CI

Fix Mir visitor macro to ensure it calls the proper method to invalidate the predecessors cache

Add pass to ensure predecessors cache is generated after optimization

Stop invalidating predecessors cache when accessing unique basic block, invalidate cache when accessing unique terminator

Get rid of old comment

Inline cache impl into Body, rename predecessor fns, change output of predecessors_for to slice

Remove interior mutability in mir predecessors cache

Auto merge of #66874 - RalfJung:miri-assert-panic, r=oli-obk

Miri engine: proper support for `Assert` MIR terminators

This puts down the basis for https://github.com/rust-lang/miri/issues/1070, and I also did some clean-up. The Miri side of this is at https://github.com/rust-lang/miri/pull/1084.

r? @oli-obk

Auto merge of #66950 - RalfJung:rollup-12d0zx8, r=RalfJung

Rollup of 5 pull requests

Successful merges:

 - #66245 (Conditional compilation for sanitizers)
 - #66654 (Handle const-checks for `&mut` outside of `HasMutInterior`)
 - #66822 (libunwind_panic: adjust miri panic hack)
 - #66827 (handle diverging functions forwarding their return place)
 - #66834 (rustbuild fixes)

Failed merges:

r? @ghost

Rollup merge of #66834 - infinity0:master, r=Mark-Simulacrum

rustbuild fixes

When upgrading Debian's rustc to 1.38 I needed these patches:

(1) In order to cross-compile rustc 1.38 and take it through the full rustbuild process including install, I needed the first patch.

(2) In order to build rustc 1.38 using rustc 1.38 itself I need to set --cap-lints warn, otherwise I get this error:

~~~~
error: unnecessary `unsafe` block
   --> src/bootstrap/builder.rs:148:19
    |
148 |             name: unsafe { ::std::intrinsics::type_name::<S>() },
    |                   ^^^^^^ unnecessary `unsafe` block
    |
note: lint level defined here
   --> src/bootstrap/lib.rs:107:9
    |
107 | #![deny(warnings, rust_2018_idioms, unused_lifetimes)]
    |         ^^^^^^^^
    = note: `#[deny(unused_unsafe)]` implied by `#[deny(warnings)]`

error: aborting due to previous error

error: could not compile `bootstrap`.

~~~~

In order to set --cap-lints warn however, I need bootstrap.py not to clobber RUSTFLAGS. (This worked previously, not sure if it was broken intentionally but we would like support for it.)

Rollup merge of #66827 - RalfJung:miri-missing-ret-place, r=oli-obk

handle diverging functions forwarding their return place

Fixes https://github.com/rust-lang/miri/issues/1075: the shim around diverging closures turned into function pointers actually "obtains" a return place inside a diverging function, but just uses it as the return place for a diverging callee. Handle this by using NULL places.

This is kind of a hack as it breaks our invariant that all places are dereferencable, but we'd eventually let raw pointers break that anyway I assume so that seems fine.

r? @oli-obk

Rollup merge of #66822 - RalfJung:miri-panic, r=oli-obk

libunwind_panic: adjust miri panic hack

We adjust the Miri hack in libpanic_unwind such that even with `cfg(miri)`, we build a version of libpanic_unwind that actually works.

This is needed to resolve https://github.com/integer32llc/rust-playground/issues/548.

r? @oli-obk @alexcrichton

Rollup merge of #66654 - ecstatic-morse:check-consts-ref, r=eddyb,matthewjasper

Handle const-checks for `&mut` outside of `HasMutInterior`

Addresses [this comment](https://github.com/rust-lang/rust/pull/64470#discussion_r328200508).

Const-checking relied on `HasMutInterior` to forbid `&mut` in a const context. This was strange because all we needed to do was look for an `Rvalue::Ref` with a certain `BorrowKind`, whereas the `Qualif` traits are specifically meant to get the qualifs for a *value*. This PR removes that logic from `HasMutInterior` and moves it into `check_consts::Validator`.

As a result, we can now properly handle qualifications for `static`s, which had to be ignored previously since you can e.g. borrow a static `Cell` from another `static`. We also remove the `derived_from_illegal_borrow` logic, since it is no longer necessary; we give good errors for subsequent reborrows/borrows of illegal borrows.

Rollup merge of #66245 - tmiasko:cfg-sanitize, r=oli-obk

Conditional compilation for sanitizers

Configure sanitize option when compiling with a sanitizer to make
it possible to execute different code depending on whether given
sanitizer is enabled or not.

fix init_allocation_extra

Assert terminator is never executed in ConstProp

fat -> wide

make alloc_extra machine hook a bit nicer

add and use helper method to get SourceInfo of current instruction in frame

Miri: add helper fn to allocate string; simplify alloc_caller_location

Add From instances for Pointer -> ScalarMaybeUndef and Pointer -> Immediate

expose span to M::assert_panic, and provide helper to turn that into CallerLocation

Miri: add machine hook for MIR-level assertion panics

Auto merge of #66944 - Centril:rollup-ojsszx6, r=Centril

Rollup of 7 pull requests

Successful merges:

 - #66346 (Replace .unwrap() with ? in std::os::unix::net)
 - #66789 (rustc: move mir::SourceScopeLocalData to a field of SourceScopeData.)
 - #66850 (rustc: hide HirId's fmt::Debug output from -Z span_free_formats.)
 - #66905 (rustc_plugin: Remove some remaining plugin features)
 - #66907 (rustc: don't just show raw DefIndex's in BrNamed's fmt::Debug impl.)
 - #66918 (Add crc and crypto to target feature whitelist on arm)
 - #66926 (add reusable MachineStop variant to Miri engine error enum)

Failed merges:

r? @ghost

Rollup merge of #66926 - RalfJung:miri-stop, r=oli-obk

add reusable MachineStop variant to Miri engine error enum

Replace the Miri-tool-specific `Exit` error variant with something dynamically typed that all clients of the Miri engine can use.

r? @oli-obk
Cc https://github.com/rust-lang/rust/issues/66902

Rollup merge of #66918 - makotokato:aarch32-crc-crypto, r=petrochenkov

Add crc and crypto to target feature whitelist on arm

aarch32 (ARMv8 32-bit) supports crc and crypto.

Rollup merge of #66907 - eddyb:br-nicer-named, r=oli-obk

rustc: don't just show raw DefIndex's in BrNamed's fmt::Debug impl.

Context: these `fmt::Debug` impls only get used with `-Z verbose` (which some tests use).

I was going to print the path like in #66850 (or rather, use `DefId`'s `fmt::Debug`, which is close but not as nice), but then I realized that most of the `DefId`s were `crate0:DefIndex(0)`, i.e. the crate root.
As the crate root is not a lifetime, they're clearly dummies of some sort, and we don't have to print anything other than the name for them.

This means that out of all the tests, there's only 5 instances of `BrNamed` that now print the full path to the lifetime parameter, and everything else is shorter instead, which doesn't feel too bad.

cc @nikomatsakis

Rollup merge of #66905 - petrochenkov:rmplugin2, r=Centril

rustc_plugin: Remove some remaining plugin features

- Plugin arguments (`#![plugin(my_plugin(args))]`) are no longer supported.
- Registering additional plugins from command line (`-Z extra-plugins=my_plugin`) is no longer supported, `-Z crate-attr=plugin(my_plugin)` can be used instead.
- Lint `plugin_as_library` is removed as mostly useless now, when plugins exist as a compatibility feature with greatly reduced functionality.
- Plugins registering additional LLVM passes (`Registry::register_llvm_pass`) are no longer supported, `-C                   passes=my_passes` can be used instead.

r? @Centril

Rollup merge of #66850 - eddyb:span-free-formats, r=oli-obk

rustc: hide HirId's fmt::Debug output from -Z span_free_formats.

This replaces the only occurrences of `HirId {...}` from tests with paths, i.e.:
```rust
[closure@HirId { owner: DefIndex(4), local_id: 15 } q:&i32, t:&T]
```
becomes, after this PR:
```rust
[closure@foo<T>::{{closure}}#0 q:&i32, t:&T]
```

r? @oli-obk cc @michaelwoerister

Rollup merge of #66789 - eddyb:mir-source-scope-local-data, r=oli-obk

rustc: move mir::SourceScopeLocalData to a field of SourceScopeData.

By having one `ClearCrossCrate<SourceScopeLocalData>` for each scope, as opposed to a single `ClearCrossCrate` for all the `SourceScopeLocalData`s, we can represent the fact that some scopes have `SourceScopeLocalData` associated with them, and some don't.

This is useful when doing MIR inlining across crates, because the `ClearCrossCrate` will be `Clear` for the cross-crate MIR scopes and `Set` for the local ones.

Also see https://github.com/rust-lang/rust/pull/66203#issuecomment-555589574 for some context around this approach.

Fixes #51314.

Rollup merge of #66346 - linkmauve:try-in-docstring, r=Dylan-DPC

Replace .unwrap() with ? in std::os::unix::net

As people like to copy examples, this gives them good habits.

Incorporate fixes from review

rustc_plugin: Some further cleanup

Remove a useless test

rustc_plugin: Remove support for plugins adding LLVM passes

rustc_lint: Remove lint `plugin_as_library`

rustc_plugin: Remove support for adding plugins from command line

rustc_plugin: Remove support for plugin arguments

add reusable MachineStop variant to Miri engine error enum

Conditional compilation for sanitizers

Configure sanitize option when compiling with a sanitizer to make
it possible to execute different code depending on whether given
sanitizer is enabled or not.

Auto merge of #66917 - Centril:rollup-xj2enik, r=Centril

Rollup of 9 pull requests

Successful merges:

 - #66503 (More useful test error messages on should_panic(expected=...) mismatch)
 - #66662 (Miri: run panic-catching tests in liballoc)
 - #66679 (Improve lifetime errors with implicit trait object lifetimes)
 - #66726 (Use recursion_limit for const eval stack limit)
 - #66790 (Do `min_const_fn` checks for `SetDiscriminant`s target)
 - #66832 (const_prop: detect and avoid catching Miri errors that require allocation)
 - #66880 (Add long error code explanation message for E0203)
 - #66890 (Format liballoc with rustfmt)
 - #66896 (pass Queries to compiler callbacks)

Failed merges:

r? @ghost

Rollup merge of #66896 - RalfJung:queries, r=Zoxc

pass Queries to compiler callbacks

https://github.com/rust-lang/rust/pull/66791 made it impossible to access the tcx in the callbacks; this should fix that.

r? @Zoxc