Update test/ui/parser for bare_trait_object warnings

Set bare_trait_objects lint to warn

rustbuild: Tweak how stage1 compilers are selected

This commit furthers the previous one to ensure that we don't build an
extra stage of the compiler in CI. A test has been added to rustbuild to
ensure that this doesn't regress, and then in debugging this test it was
hunted down that the `dist::Std` target was the one erroneously pulling
in the wrong compiler.

The `dist::Std` step was updated to instead account for the "full
bootstrap" or not flag, ensuring that the correct compiler for compiling
the final standard library was used. This was another use of the
`force_use_stage1` function which was in theory supposed to be pretty
central, so existing users were all evaluated and a new function,
`Builder::compiler_for`, was introduced. All existing users of
`force_use_stage1` have been updated to use `compiler_for`, where the
semantics of `compiler_for` are similar to that of `compiler` except
that it doesn't guarantee the presence of a sysroot for the arguments
passed (as they may be modified).

Perhaps one day we can unify `compiler` and `compiler_for`, but the
usage of `Builder::compiler` is so ubiquitous it would take quite some
time to evaluate whether each one needs the sysroot or not, so it's
hoped that can be done in parallel.

Auto merge of #61274 - Centril:rollup-23dekk4, r=Centril

Rollup of 4 pull requests

Successful merges:

 - #61123 (Allow to specify profiling data output directory as -Zself-profile argument.)
 - #61159 (split core::ptr module into multiple files)
 - #61164 (rename Scalar::Bits to Scalar::Raw and bits field to data)
 - #61250 (Remove special case for *ios* builds in run-make-fulldeps/print-target-list Makefile)

Failed merges:

r? @ghost

Revert "Disable solaris target since toolchain no longer builds"

This reverts commit e764f475ca7fffd6167ea991afc7d1b2b3f642dc.
Fixes #61174.

Changes the type `mir::Mir` into `mir::Body`

The commit should have changed comments as well.
At the time of writting, it passes the tidy and check tool.

Revisions asked by eddyb :
- Renamed of all the occurences of {visit/super}_mir
- Renamed test structures `CachedMir` to `Cached`

Fixing the missing import on `AggregateKind`

implicit `Option`-returning doctests

This distinguishes `Option` and `Result`-returning doctests with
implicit `main` method, where the former tests must end with
`Some(())`.

Rename Place::local to Place::local_or_deref_local

Use closure to avoid self.describe_place(...).unwrap_or_else(...) repetition

is_union returns ty to avoid computing it twice

Update src/libcore/mem.rs

Co-Authored-By: Mazdak Farrokhzad <twingoow@gmail.com>

Rollup merge of #61250 - rye:print-target-list--rm-ios-case, r=alexcrichton

Remove special case for *ios* builds in run-make-fulldeps/print-target-list Makefile

Previous `TODO` comment in this file mentions [an issue that was closed](https://github.com/rust-lang/rust/issues/29812), and I was able to confirm locally that provided code in that issue no longer produces an ICE. Discussion on that issue seems to indicate this code was no longer needed as of 1.12.0.

I removed the `*ios*` branch from this `case` statement as it may cause confusion, then removed the case statement entirely as it only had a wildcard branch.

Rollup merge of #61164 - RalfJung:scalar, r=oli-obk

rename Scalar::Bits to Scalar::Raw and bits field to data

Also use this opportunity to seal some abstraction leaks (other modules constructing `Scalar::Bits` directly instead of using a constructor).

r? @oli-obk

Rollup merge of #61159 - RalfJung:ptr, r=alexcrichton

split core::ptr module into multiple files

Cc @Centril

Rollup merge of #61123 - michaelwoerister:self-profile-dir, r=wesleywiser

Allow to specify profiling data output directory as -Zself-profile argument.

The PR also makes `rustc` include the crate-name (if already available) in the output file name.

r? @wesleywiser

At some point we should add some basic tests for `-Zself-profile`.

mention that MaybeUninit is a bit like Option

Auto merge of #61246 - oli-obk:clippy, r=oli-obk

Update clippy submodule

fixes #61240

Rename PgoGenerate to something more general.

Allow to specify profiling data output directory as -Zself-profile argument.

Make test/codegen/pgo-instrumentation.rs work reliably on Windows.

Emit error when trying to use PGO in conjunction with unwinding on Windows.

Auto merge of #61258 - Centril:rollup-l2mof9t, r=Centril

Rollup of 9 pull requests

Successful merges:

 - #61084 (Clarify docs for unreachable! macro)
 - #61220 (Added error message for E0284)
 - #61227 (Use .await syntax instead of await!)
 - #61230 (avoid creating Boxes of uninitalized values in RawVec)
 - #61237 (Updated the Iterator docs with information about overriding methods.)
 - #61241 (Check place iterative)
 - #61242 (Make dest_needs_borrow iterate instead of recurse)
 - #61247 (Make eval_place iterate instead of recurse)
 - #61248 (Use Place::local)

Failed merges:

r? @ghost

Alphabetized lines with Atom's Sort Lines package

https://github.com/atom/sort-lines

Adding mailmap for myself

Fixing mailmap for Carol

Update clippy submodule

Rollup merge of #61248 - spastorino:use-place-local-fn, r=oli-obk

Use Place::local

r? @oli-obk

Rollup merge of #61247 - spastorino:eval-place-iterate, r=wesleywiser

Make eval_place iterate instead of recurse

r? @oli-obk

Rollup merge of #61242 - spastorino:dest-needs-borrow-iterate, r=oli-obk

Make dest_needs_borrow iterate instead of recurse

r? @oli-obk

Rollup merge of #61241 - spastorino:check-place-iterative, r=oli-obk

Check place iterative

r? @oli-obk

Rollup merge of #61237 - DevQps:expand-iterator-docs, r=Mark-Simulacrum

Updated the Iterator docs with information about overriding methods.

# Description

Updated the Iterator docs with information about overriding methods.

closes #60223

Rollup merge of #61230 - matklad:ub-comment, r=RalfJung

avoid creating Boxes of uninitalized values in RawVec

`RawVec<bool>::into_box` is definitely instant UB, if not all values are initialized.

See https://gankro.github.io/blah/initialize-me-maybe/

Rollup merge of #61227 - diwic:patch-2, r=Centril

Use .await syntax instead of await!

Tiny doc fix.

Rollup merge of #61220 - imbrem:error_explanations, r=estebank

Added error message for E0284

Work on #61137

Rollup merge of #61084 - blkerby:unreachable_doc, r=KodrAus

Clarify docs for unreachable! macro

Fixes #60754.

librustc_errors: Move annotation collection to own impl

Extracted from work on #59346. This moves the annotation collection to
the `FileWithAnnotatedLines` impl to allow re-use in a separate
EmitterWriter.

Auto merge of #60955 - agnxy:rename-assoc, r=oli-obk,Centril

Rename "Associated*" to "Assoc*"

This change is for #60163.

r? @oli-obk

Auto merge of #61036 - michaelwoerister:pgo-xlto-test, r=alexcrichton

PGO - Add a smoketest for combining PGO with cross-language LTO.

This PR

- Adds a test making sure that PGO can be combined with cross-language LTO.
- Does a little cleanup on how the `pgo-use` flag is handled internally.
- Makes the compiler error if the `pgo-use` file given to `rustc` doesn't actually exist. LLVM only gives a warning and then just doesn't do PGO. Clang, on the other hand, does give an error in this case.
- Makes the build system also build `compiler-rt` when building LLDB. This way the Clang compiler that we get from building LLDB can perform PGO, which is something that the new test case wants to do. CI compile times shouldn't be affected too much.

Fix an sh error

Didn't think it was this particular about things, but I also should
have tested locally.

It makes sense, though---`\` followed by LF would eat it, so we'd have
`sysroot done` instead of `sysroot; done` as it is parsed.  This should
pass now.

Signed-off-by: Kristofer Rye <kristofer.rye@gmail.com>
Tested-by: Kristofer Rye <kristofer.rye@gmail.com>

Incorporated suggested changes

Auto merge of #61078 - pietroalbini:nightly-next, r=Centril

Bump nightly to 1.37.0

r? @Mark-Simulacrum

Remove now-dead case statement in print-target-list Makefile

Since this case statement no longer has any branches, remove it.

Signed-off-by: Kristofer Rye <kristofer.rye@gmail.com>

Remove *ios* case from print-target-list Makefile

Based on the TODO, this case was added to short-circuit for ios builds,
which is no longer necessary.

The comment in this Makefile mentions rust-lang/rust#29812 as a
dependency, but that issue has been since closed, with a statement that
the ICE of concern was resolved circa 1.12.

Here we remove this case, and just run the same branch for all targets.

Signed-off-by: Kristofer Rye <kristofer.rye@gmail.com>

Use Place::local

Make eval_place iterate instead of recurse

Box::into_vec: use Box::into_raw instead of mem::forget

Make check_place iterate instead of recurse

Make dest_needs_borrow iterate instead of recurse

Auto merge of #61140 - estebank:attr-diagnostics, r=michaelwoerister

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

Split up from https://github.com/rust-lang/rust/pull/61026, where there's prior conversation.

Remove unused tcx and mir params

Update src/liballoc/boxed.rs

Co-Authored-By: Ralf Jung <post@ralfj.de>

make Box<str>::clone simpler & safer

Auto merge of #60827 - oli-obk:late_symbol, r=nnethercote

Use `Symbol` more in lint APIs

r? @nnethercote

This will cause clippy breakage, but super trivial to fix since we can then remove the hacky `match_def_path` function that I added and go back to calling rustc's `match_def_path` method.

Only build clang_rt when RUSTBUILD_FORCE_CLANG_BASED_TESTS is set.

Updated the Iterator docs with information about overriding methods.

Stabilize bufreader_buffer feature

Auto merge of #59627 - LooMaclin:issue_57128_improve_miri_error_reporting_in_check_in_alloc, r=RalfJung

Improve miri error reporting in check_in_alloc

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

r? @RalfJung @oli-obk

Fix unit test after pgo-use change.

Add a smoketest for combining PGO with xLTO.

rustbuild: Also build compiler-rt when building LLDB. This allows clang-based run-make tests to use PGO.

PGO: Check that pgo-use file actually exists. LLVM seems to only emit an easy-to-overlook warning otherwise.

Use a PathBuf instead of String for representing the pgo-use path internally.

factor out some common code and make the on-elimination truncation test debug-only

fix truncate and sign_extend for size == 0

Use `Symbol` more in lint APIs

Tests for external linkage symbol collision check.

Fix #61232

Auto merge of #60967 - Zoxc:fix-syntax-sync, r=michaelwoerister

Short circuit Send and Sync impls for TokenTree

Workaround to make the parallel compiler build after https://github.com/rust-lang/rust/pull/60444.

r? @nikomatsakis

Regression test for issue #59548.

Refine the message to at least *mention* the attribute itself.

Update pre-existing test's diagnostic output accordingly.

Always supply span to check_and_apply_linkage, sidestepping need to add `bug!`s to rustc.

Move existing `#[linkage]` tests to a subdirectory where I can add more tests.

avoid materializing unintialized Boxes in RawVec

Short circuit Send and Sync impls for TokenTree

Auto merge of #61035 - nnethercote:avoid-more-symbol-interning, r=petrochenkov

Avoid more symbol interning

r? @petrochenkov

Use .await syntax instead of await!

Use `Symbol` equality in `may_begin_with` and `parse_nt`.

Use `Symbol` equality in `check_ident_token`.

Use `Symbol` equality in `is_ident_named`.

Avoid some re-interning in `to_lit_token`.

Add `to_symbol` methods.

Auto merge of #61215 - paoloteti:armebv7r, r=alexcrichton

Docker: remove environment variable `CC_armebv7r_none_eabi`

`cc-rs` take care of selecting the tool-chain, so this `ENV` variable can be safely removed.

Pre-intern "0", "1", ..., "9", and use where appropriate.

Pass symbols to `ExtCtxt::std_path` instead of strings.

Because this function is hot.

Also remove the dead `ty_option` function.

Avoid unnecessary internings.

Most involving `Symbol::intern` on string literals.

Avoid interning in `resolve_place_op`.

This function is hot for `keccak`.

Auto merge of #61147 - estebank:suggest-as-ref, r=oli-obk

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

Fix #61109, cc #15457.

Auto merge of #61214 - oli-obk:clippy, r=Manishearth

Update clippy submodule

r? @Manishearth

Added error message for E0284

Auto merge of #61213 - pietroalbini:fix-azure-multiple-checkouts, r=alexcrichton

azure: fix multiple checkouts on azure

We were checking out the rustc repo multiple times on auto macOS, and
that was causing an error on the Azure side since multiple checkouts are
apparently not supported. This removes the extra checkout.

r? @alexcrichton
fixes #61170

Account for short-hand init structs when suggesting conversion

Docker: remove environment variable `CC_armebv7r_none_eabi`

`cc-rs` take care of selecting the tool-chain, so this ENV variable
can be safely removed.

Update cargo lockfile

Update clippy submodule

ci: increase timeout on the auto branch in azure

ci: fix multiple checkouts on azure

We were checking out the rustc repo multiple times on auto macOS, and
that was causing an error on the Azure side since multiple checkouts are
apparently not supported. This removes the extra checkout.

ci: Attempt to skip a full rustc compile on dist*

Currently when we're preparing cross-compiled compilers it can take
quite some time because we have to build the compiler itself three
different times. The first is the normal bootstrap, the second is a
second build for the build platform, and the third is the actual target
architecture compiler. The second compiler was historically built
exclusively for procedural macros, and long ago we didn't actually need
it.

This commit tries out avoiding that second compiled compiler, meaning we
only compile rustc for the build platform only once. Some local testing
shows that this is promising, but bors is of course the ultimate test!

Auto merge of #61210 - Centril:rollup-ofr6h5b, r=Centril

Rollup of 4 pull requests

Successful merges:

 - #61077 (Don't arena-allocate static symbols.)
 - #61102 (Move path for iterate)
 - #61120 (Make eval_place iterate instead of recurse)
 - #61205 (docs: fix typo #61197)

Failed merges:

r? @ghost

centralize Scalar size sanity checks and also do them in release builds

fix overflow error in signed wrapping offset

Improve miri's error reporting in check_in_alloc

Co-Authored-By: Ralf Jung <post@ralfj.de>