rustdoc: Don't add extra newlines for fully opaque structs

Changes the definition for opaque structs to look like `pub struct Vec<T>
{ /* fields omitted */ }` to save space on the page.

Also only use one line for empty braced structs.

Auto merge of #35745 - jroesch:soundness-fix-29859, r=nikomatsakis

Fix soundness bug described in #29859

This is an attempt at fixing the problems described in #29859 based on an IRC conversation between @nikomatsakis and I today. I'm waiting on a full build to come back, otherwise both tests trigger the correct error.

Auto merge of #36048 - GuillaumeGomez:code_clean, r=brson

Clean code a bit

Auto merge of #36316 - jseyfried:custom_derive_internal_unstable, r=eddyb

Avoid instaiblity errors in code generated by `syntax_ext::deriving::call_intrinsic()`

r? @eddyb

Auto merge of #36310 - jstnlef:remove-extraneous-not-equal-impl, r=sfackler

Removing the extraneous not_equal implementation for slices

Happened to stumble upon this one awhile back. Seemed a bit silly to have both the equals and not equals implementation when they're so similar.

Auto merge of #36214 - jseyfried:stackless_expansion, r=nrc

macros: stackless expansion

After this PR, macro expansion cannot overflow the stack unless the expanded crate is too deep to fold.
Everything but the stackless placeholder expansion commit is also groundwork for macro modularization.

r? @nrc or @eddyb

Fix duplicate error code

Auto merge of #36292 - japaric:musl-root, r=alexcrichton

rustbuild: per target musl-root

config.toml now accepts a target.$TARGET.musl-root key that lets you
override the "build" musl-root value, which is set via the --musl-root
flag or via the build.musl-root key.

With this change, it's now possible to compile std for several musl
targets at once. Here's are the sample commands to do such thing:

```
$ configure \
    --enable-rustbuild \
    --target=x86_64-unknown-linux-musl,arm-unknown-linux-musleabi \
    --musl-root=/musl/x86_64-unknown-linux-musl/

$ edit config.toml && tail config.toml
[target.arm-unknown-linux-musleabi]
musl-root = "/x-tools/arm-unknown-linux-musleabi/arm-unknown-linux-musleabi/sysroot/usr"

$ make
```

r? @alexcrichton
With this we should be able to start producing releases of std for arm musl targets

Improve `directory` computation during invocation collection.

Implement stackless placeholder expansion.

Strip unconfigured nodes in the `InvocationCollector` fold.

Refactor code out of the folder implementation for `StripUnconfigured`.

Auto merge of #36266 - Sawyer47:issue-35169, r=alexcrichton

rustdoc: Filter more incorrect methods inherited through Deref

Old code filtered out only static methods. This code also excludes &mut self methods if there is no DerefMut implementation.

Fixes #35169

Auto merge of #36304 - TimNN:fix-36299, r=eddyb

re-add accidentally removed line in wfcheck

Fixes #36299, introduced in #36119.

r? @eddyb

Auto merge of #36289 - euclio:self-suggestion, r=jseyfried

resolve: Suggest `use self` when import resolves

Improves errors messages by replacing "Maybe a missing `extern crate`" messages
with "Did you mean `self::...`" when the `self` import would succeed.

Fixes #34191.

Thank you for the help @jseyfried!

Auto merge of #36296 - nagisa:pass-timing, r=eddyb

Count and report time taken by MIR passes

There’s some desire for deeper introspectability into what MIR passes cost us.

-Z time-passes after this PR:

```
   Compiling test_shim v0.1.0 (file:///home/nagisa/Documents/rust/rust/src/rustc/test_shim)
time: 0.000; rss: 29MB parsing
time: 0.000; rss: 29MB configuration
time: 0.000; rss: 29MB recursion limit
time: 0.000; rss: 29MB crate injection
time: 0.000; rss: 29MB plugin loading
time: 0.000; rss: 29MB plugin registration
time: 0.032; rss: 54MB expansion
time: 0.000; rss: 54MB maybe building test harness
time: 0.000; rss: 54MB assigning node ids
time: 0.000; rss: 54MB checking for inline asm in case the target doesn't support it
time: 0.000; rss: 54MB complete gated feature checking
time: 0.000; rss: 54MB collecting defs
time: 0.004; rss: 54MB external crate/lib resolution
time: 0.000; rss: 54MB early lint checks
time: 0.000; rss: 54MB AST validation
time: 0.001; rss: 54MB name resolution
time: 0.000; rss: 54MB lowering ast -> hir
time: 0.000; rss: 56MB indexing hir
time: 0.000; rss: 56MB attribute checking
time: 0.000; rss: 56MB language item collection
time: 0.000; rss: 56MB lifetime resolution
time: 0.000; rss: 56MB looking for entry point
time: 0.000; rss: 56MB looking for plugin registrar
time: 0.000; rss: 56MB region resolution
time: 0.000; rss: 56MB loop checking
time: 0.000; rss: 56MB static item recursion checking
time: 0.000; rss: 56MB compute_incremental_hashes_map
time: 0.000; rss: 56MB load_dep_graph
time: 0.000; rss: 56MB type collecting
time: 0.000; rss: 56MB variance inference
time: 0.011; rss: 59MB coherence checking
time: 0.000; rss: 59MB wf checking
time: 0.000; rss: 59MB item-types checking
time: 0.000; rss: 59MB item-bodies checking
time: 0.000; rss: 59MB drop-impl checking
time: 0.000; rss: 59MB const checking
time: 0.000; rss: 59MB privacy checking
time: 0.000; rss: 59MB stability index
time: 0.000; rss: 59MB intrinsic checking
time: 0.000; rss: 59MB effect checking
time: 0.000; rss: 59MB match checking
time: 0.000; rss: 59MB liveness checking
time: 0.000; rss: 59MB rvalue checking
time: 0.000; rss: 59MB MIR dump
  time: 0.000; rss: 59MB SimplifyCfg
  time: 0.000; rss: 59MB QualifyAndPromoteConstants
  time: 0.000; rss: 59MB TypeckMir
  time: 0.000; rss: 59MB SimplifyBranches
  time: 0.000; rss: 59MB SimplifyCfg
time: 0.000; rss: 59MB MIR passes
time: 0.000; rss: 59MB borrow checking
time: 0.000; rss: 59MB reachability checking
time: 0.000; rss: 59MB death checking
time: 0.000; rss: 59MB stability checking
time: 0.000; rss: 59MB unused lib feature checking
time: 0.000; rss: 59MB lint checking
time: 0.000; rss: 59MB resolving dependency formats
  time: 0.000; rss: 59MB NoLandingPads
  time: 0.000; rss: 59MB SimplifyCfg
  time: 0.000; rss: 59MB EraseRegions
  time: 0.000; rss: 59MB AddCallGuards
  time: 0.000; rss: 59MB ElaborateDrops
  time: 0.000; rss: 59MB NoLandingPads
  time: 0.000; rss: 59MB SimplifyCfg
  time: 0.000; rss: 59MB Deaggregator
  time: 0.000; rss: 59MB AddCallGuards
  time: 0.000; rss: 59MB PreTrans
time: 0.000; rss: 59MB Prepare MIR codegen passes
  time: 0.000; rss: 59MB write metadata
  time: 0.000; rss: 61MB translation item collection
  time: 0.000; rss: 61MB codegen unit partitioning
  time: 0.000; rss: 61MB internalize symbols
time: 0.007; rss: 61MB translation
time: 0.000; rss: 61MB assert dep graph
time: 0.000; rss: 61MB serialize dep graph
time: 0.000; rss: 61MB llvm function passes [2]
time: 0.000; rss: 61MB llvm function passes [3]
time: 0.000; rss: 61MB llvm function passes [1]
time: 0.000; rss: 61MB llvm function passes [0]
time: 0.000; rss: 61MB llvm module passes [2]
time: 0.000; rss: 61MB llvm module passes [1]
time: 0.000; rss: 61MB llvm module passes [0]
time: 0.000; rss: 61MB llvm module passes [3]
time: 0.001; rss: 62MB codegen passes [1]
time: 0.001; rss: 62MB codegen passes [2]
time: 0.001; rss: 62MB codegen passes [0]
time: 0.001; rss: 62MB codegen passes [3]
time: 0.001; rss: 63MB codegen passes [1]
time: 0.005; rss: 63MB LLVM passes
time: 0.000; rss: 63MB serialize work products
time: 0.001; rss: 63MB linking
```

r? @eddyb or @nikomatsakis

cc @nrc, @Mark-Simulacrum

Add regression test.

Avoid instaiblity errors in code generated by `syntax_ext::deriving::call_intrinsic()`.

Auto merge of #36252 - joshtriplett:union-field-never-used, r=sanxiyn

Fix "field is never used" warning to take unions into account

When compiling code containing a union with an unused field, rustc says
"struct field is never used".

Rather than saying "struct or union", or adding logic to determine the
type of the item, just change the message to "field is never used",
dropping the "struct".

Update tests accordingly.

add utility musl_root method, update config.toml.example

remove the extraneous not_equal implementation for slices.

Auto merge of #36303 - jonathandturner:rollup, r=jonathandturner

Rollup of 8 pull requests

- Successful merges: #36121, #36128, #36241, #36243, #36263, #36267, #36273, #36298
- Failed merges:

Auto merge of #36025 - michaelwoerister:incr-comp-hash-spans, r=nikomatsakis

incr. comp.: Take spans into account for ICH

This PR makes the ICH (incr. comp. hash) take spans into account when debuginfo is enabled.

A side-effect of this is that the SVH (which is based on the ICHs of all items in the crate) becomes sensitive to the tiniest change in a code base if debuginfo is enabled. Since we are not trying to model ABI compatibility via the SVH anymore (this is done via the crate disambiguator now), this should be not be a problem.

Fixes #33888.
Fixes #32753.

re-add accidentally removed line

Rollup merge of #36298 - GuillaumeGomez:hashmap_doc, r=steveklabnik

Add missing urls

r? @steveklabnik

Rollup merge of #36273 - apasel422:unsafe_no_drop_flag, r=steveklabnik

Remove mention of `unsafe_no_drop_flag` from Reference and Nomicon

Rollup merge of #36267 - Cobrand:E0559, r=jonathandturner

Updated E0559 to new format

Refactored a method that printed one suggested field name,
into a method that returns an `Option` of a suggestion

(Updated test cases accordingly)

r? @jonathandturner

Closes #36197

Rollup merge of #36263 - apasel422:scoped, r=steveklabnik

Clean up thread-local storage docs

`std` no longer contains an implementation of scoped TLS.

r? @steveklabnik

Rollup merge of #36243 - GuillaumeGomez:hash_map_links, r=steveklabnik

Add missing urls

r? @steveklabnik

Rollup merge of #36241 - Cobrand:contributing-md, r=steveklabnik

doc: Contributing.md: Added mention of `make tidy`

r? @steveklabnik

Rollup merge of #36128 - gavinb:error_msgs_p2, r=jonathandturner

Update Error format for E0516, E0517, E0518

- E0518 Update error format #36111
- E0517 Update error format #36109
- E0516 Update error format #36108
- Part of #35233

r? @jonathandturner

Rollup merge of #36121 - Cobrand:master, r=jonathandturner

Updated E0527 to new error format

* Closes #36113

ICH: Make CachingCodemapView robustly handle invalid spans.

Auto merge of #36288 - nrc:save-var-value, r=eddyb

save-analysis: some refinement to the value string for variables

Add missing urls

Count and report time taken by MIR passes

Auto merge of #36276 - jseyfried:fix_unused, r=nrc

resolve: Fix unused import false positive with `item_like_imports`

Fixes #36249.
r? @nrc

rustbuild: per target musl-root

config.toml now accepts a target.$TARGET.musl-root key that lets you
override the "build" musl-root value, which is set via the --musl-root
flag or via the build.musl-root key.

With this change, it's now possible to compile std for several musl
targets at once. Here's are the sample commands to do such thing:

```
$ configure \
    --enable-rustbuild \
    --target=x86_64-unknown-linux-musl,arm-unknown-linux-musleabi \
    --musl-root=/musl/x86_64-unknown-linux-musl/

$ edit config.toml && tail config.toml
[target.arm-unknown-linux-musleabi]
musl-root = "/x-tools/arm-unknown-linux-musleabi/arm-unknown-linux-musleabi/sysroot/usr"

$ make
```

Auto merge of #36078 - ollie27:rustdoc_search_assocconst, r=alexcrichton

rustdoc: Fix associated consts in search results

Associated consts can appear in none trait impls so need to be treated
like methods when generating the search index.

Fixes #36031

Add struct `AmbiguityError`.

Add field `used: Cell<bool>` to variant `NameBindingKind::Import`.

Add regression test.

Avoid false positive unused import warnings.

show `self` suggestion when items are in the block

resolve: Suggest `use self` when import resolves

Improves errors messages by replacing "Maybe a missing `extern crate`" messages
with "Did you mean `self::...`" when the `self` import would succeed.

save-analysis: some refinement to the value string for variables

Auto merge of #36274 - Manishearth:rollup, r=Manishearth

Rollup of 6 pull requests

- Successful merges: #35845, #35983, #36065, #36102, #36242, #36245
- Failed merges:

Auto merge of #35845 - frewsxcv:result-into-iter, r=GuillaumeGomez

Indicate where `core::result::IntoIter` is created.

None

Auto merge of #36281 - pnkfelix:fix-issue-36036, r=eddyb

Fix issue #36036.

Fix #36036.

We were treating an associated type as unsized even when the concrete instantiation was actually sized. Fix is to normalize before checking if it is sized.

rustdoc: Filter more incorrect methods inherited through Deref

Old code filtered out only static methods. This code also excludes
&mut self methods if there is no DerefMut implementation

Updated E0527 to new error format

* Closes #36113

Add test for unused field in union

Auto merge of #36200 - mattico:fix-llvm-linkage, r=arielb1

Fix incorrect LLVM Linkage enum

Followup of #33994 to actually work.

The `Linkage` enum in librustc_llvm got out of sync with the version in LLVM and it caused two variants of the `#[linkage=""]` attribute to break.

This adds the functions `LLVMRustGetLinkage` and `LLVMRustSetLinkage` which convert between the Rust Linkage enum and the LLVM one, which should stop this from breaking every time LLVM changes it.

Possible remaining concerns:

1. There could be a codegen test to make sure that the attributes are applied correctly (I don't know how to do this).
2. ~~The test does not exercise the `appending` linkage. I can't figure out how to make a global static raw pointer to an array. This might not even be possible? If not we should probably remove appending linkage as its unusable in rust.~~ Appending linkage is not 'emittable' anyway.
3. The test only runs on Linux.

Fixes #33992

r? @alexcrichton

E0518 Update error format #36111

- Fixes #36111
- Part of #35233

E0517 Update error format #36109

- Fixes #36109
- Part of #35233

E0516 Update error format #36108
- fixes #36108
- part of #35233

Auto merge of #35787 - canndrew:bang_type_refactor_check, r=eddyb

Typecheck refactor for `!`

Ping @nikomatsakis @eddyb. This is the PR for the typeck refactor for `!`. Is this what you guys had in mind? Is there anything else that needs doing on it?

Fix issue #36036.

We were treating an associated type as unsized even when the concrete
instantiation was actually sized. Fix is to normalize before checking
if it is sized.

Rollup merge of #36245 - alexcrichton:add-back-accident, r=arielb1

Add back feature accidentally removed

This feature was accidentally removed in
https://github.com/rust-lang/rust/pull/35957.

Rollup merge of #36242 - semarie:local-rebuild, r=alexcrichton

Use libraries from local-rust-root directory in configure when using …

…--enable-local-rebuild

When using --enable-local-rebuild configure options, the configure
script will test rustc version. But when running it, it will not use the
libraries in the local-rust-root directory.

So use `LD_LIBRARY_PATH` environment variable to correct it.

Under OpenBSD, we use `--enable-local-rebuild` for rebuilding rustc-1.11.0 using rustc-1.11.0. But as it is in use in -current (the developpment branch of OpenBSD), system libraries could easily have ABI/API changes. The (unofficial) bootstrap (rustc-1.11.0 used for building rustc-1.11.0) include several system libraries to permit it to run. The build system use libraries in stage0/lib directory. But the configure script doesn't.

Due to my special use of `--enable-local-rebuild`, I dunno if this PR is suitable for inclusion. But it corrects a difference between build and configure use of the binary.

r? @alexcrichton

Rollup merge of #36065 - acrrd:new_format_E0496, r=GuillaumeGomez

Update E0496 to new format

Fixes #36000.
Part of #35233.

r? @GuillaumeGomez

Rollup merge of #35983 - GuillaumeGomez:e0060_bonus, r=jonathandturner

E0060 and E0061 improvement

Fixes #35290.

r? @jonathandturner

Rollup merge of #35845 - frewsxcv:result-into-iter, r=GuillaumeGomez

Indicate where `core::result::IntoIter` is created.

None

Auto merge of #36242 - semarie:local-rebuild, r=alexcrichton

Use libraries from local-rust-root directory in configure when using …

…--enable-local-rebuild

When using --enable-local-rebuild configure options, the configure
script will test rustc version. But when running it, it will not use the
libraries in the local-rust-root directory.

So use `LD_LIBRARY_PATH` environment variable to correct it.

Under OpenBSD, we use `--enable-local-rebuild` for rebuilding rustc-1.11.0 using rustc-1.11.0. But as it is in use in -current (the developpment branch of OpenBSD), system libraries could easily have ABI/API changes. The (unofficial) bootstrap (rustc-1.11.0 used for building rustc-1.11.0) include several system libraries to permit it to run. The build system use libraries in stage0/lib directory. But the configure script doesn't.

Due to my special use of `--enable-local-rebuild`, I dunno if this PR is suitable for inclusion. But it corrects a difference between build and configure use of the binary.

r? @alexcrichton

Load macros from `extern crate`s in the `InvocationCollector` fold.

Implement stackless expansion.

Add module `ext::placeholders` with  `placeholder()` and `PlaceholderExpander`.

Refactor `expand_invoc(.., fld)` -> `self.expand_invoc(..)`.

Refactor `SyntaxEnv`.

Refactor `expand_*` into `expander.fold_*`.

Clean up module processing.

Refactor out `expand_item` (with better semantics than before).

Refactor away `expand_item`.

Generalize `Invocation` to include modifiers/decorators.

Refactor `with_exts_frame` from a macro to a function.

Add `Invocation` and `Expansion`, remove `MacroGenerable`.

Remove `syntax::config::strip_unconfigured`, add `syntax::config::features`.

Improve `expand_type`.

In `Parser` and `ExtCtxt`, replace fields `filename` and `mod_path_stack`
with a single field `directory: PathBuf`.

Remove expr_ty method completely

Undo unnecessary bookkeeping from last commit

Remove use of expr_ty from coercions code

Move write_ty to the bottom of check_pat

Factor write_ty out of more pattern-checking functions

Factor write_ty out of pattern-matching functions

check_block_with_expected returns the checked type

Move check_expr match block into its own method

Tidy. Rename variables.

Tidy

Add AdjustNeverToAny in check_expr

Factor write_ty out of function checking functions

Factor write_ty out of check_expr_closure

Factor write_ty out of check_struct_expr

Factor write_ty out of field/indexing methods

Factor write_ty out of more checking functions

Factor write_ty out of check_binop*

Remove redundant error checking around ExprMethodCall

Remove most uses of expr_ty

Make write_ty and friends return adjusted type

Remove mention of `unsafe_no_drop_flag` from Reference and Nomicon