Rollup merge of #58975 - jtdowney:iter_arith_traits_option, r=dtolnay

Implement `iter::Sum` and `iter::Product` for `Option`

This is similar to the existing implementation for `Result`. It will take each item into the accumulator unless a `None` is returned.

I based a lot of this on #38580. From that discussion it didn't seem like this addition would be too controversial or difficult. One thing I still don't understand is picking the values for the `stable` attribute. This is my first non-documentation PR for rust so I am open to any feedback on improvements.

Auto merge of #61296 - Centril:rollup-5ad68b0, r=Centril

Rollup of 9 pull requests

Successful merges:

 - #60742 (Allow const parameters in array sizes to be unified)
 - #60756 (Add better tests for hidden lifetimes in impl trait)
 - #60928 (Changes the type `mir::Mir` into `mir::Body`)
 - #61024 (tests: Centralize proc macros commonly used for testing)
 - #61157 (BufReader: In Seek impl, remove extra discard_buffer call)
 - #61195 (Special-case `.llvm` in mangler)
 - #61202 (Print PermissionExt::mode() in octal in Documentation Examples)
 - #61259 (Mailmap fixes)
 - #61273 (mention that MaybeUninit is a bit like Option)

Failed merges:

r? @ghost

Bump Sum and Product for Option<T> to 1.37

Rollup merge of #61273 - RalfJung:maybe-uninit, r=Centril

mention that MaybeUninit is a bit like Option

Rollup merge of #61259 - JosephTLyons:mailmap-fixes, r=nikomatsakis

Mailmap fixes

I've made a few fixes and additions to the .mailmap file.

- I've added my information to it, as I noticed it was popping up twice at the [Thanks](https://thanks.rust-lang.org/rust/all-time/) website (I've made contributions to the Rust Book).
- I noticed that @carols10cents‘ information was popping up in multiple places at the [Thanks](https://thanks.rust-lang.org/rust/all-time/) website and when running `git shortlog -s -e > out.txt`, and fixed her information
- I alphabetized the list using Atom's [Sort Lines package](https://github.com/atom/sort-lines)

The output from before and after running `git shortlog -s -e > out.txt` for Carol:
<img width="708" alt="Carol Before" src="https://user-images.githubusercontent.com/19867440/58472843-2d8ca880-8115-11e9-9624-d31ed23ae9b5.png">
<img width="523" alt="Carol After" src="https://user-images.githubusercontent.com/19867440/58472844-2d8ca880-8115-11e9-95c6-80b17cc04790.png">

The output from before and after running `git shortlog -s -e > out.txt` for myself:

<img width="489" alt="Joseph After" src="https://user-images.githubusercontent.com/19867440/58472941-6e84bd00-8115-11e9-8266-1d7fcd53d0bc.png">
<img width="421" alt="Joseph Before" src="https://user-images.githubusercontent.com/19867440/58472943-6e84bd00-8115-11e9-9fd2-d7ace7e3d636.png">

Rollup merge of #61202 - oberien:permissionext-print-octal, r=varkor

Print PermissionExt::mode() in octal in Documentation Examples

Printing the file permission mode on unix systems in decimal feels unintuitive. Printing it in octal gives the expected form of e.g. `664`.

Rollup merge of #61195 - davidtwco:seg-fault-mangler, r=eddyb

Special-case `.llvm` in mangler

Fixes #60925 and fixes #53912.

r? @michaelwoerister
cc @eddyb

Rollup merge of #61157 - czipperz:BufReader-Seek-remove-extra-discard_buffer, r=nikomatsakis

BufReader: In Seek impl, remove extra discard_buffer call

As far as I can tell, this code does nothing.  I'm not sure why it even is there.

Rollup merge of #61024 - petrochenkov:proctest, r=nikomatsakis

tests: Centralize proc macros commonly used for testing

Many proc macros in `ui\proc-macro\auxiliary` were doing same things.
(I added a fair share of those myself.)

Now commonly used macros (empty, identity, etc) are collected in one place - `ui\proc-macro\auxiliary\test-macros.rs`.

Rollup merge of #60928 - TheSirC:fix/60229, r=eddyb

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

Fixes part 1 of #60229 (previously attempted in #60242).

I stumbled upon the issue and it seems that the previous attempt at solving it was not merged. This is a second try more up-to-date.

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

Rollup merge of #60756 - matthewjasper:extra-impl-trait-tests, r=nikomatsakis

Add better tests for hidden lifetimes in impl trait

cc #60670

Rollup merge of #60742 - varkor:fn-const-array-parameter, r=eddyb

Allow const parameters in array sizes to be unified

Fixes https://github.com/rust-lang/rust/issues/60632.
Fixes https://github.com/rust-lang/rust/issues/60744.
Fixes https://github.com/rust-lang/rust/pull/60923.
(The last commit should probably be viewed in isolation, as it just renames things from `type` to `kind`.)

r? @eddyb

syntax: bail out of `find_width_of_character_at_span` if the span doesn't start and end in the same file.

rustc: rename Mir to mir::Body in comments and to MIR in error strings.

Use assert_eq! instead of println! in tests

Fix nits

Make sure array length diagnostic doesn't regress

Correct pluralisation of tuple/array/associated type binding mismatch errors

Reintroduce `TypeError::FixedArraySize`

Use Display rather than Debug printing for const mismatch

Update tests after pretty printing

Fix test after rebase

Add broken MIR regression tests

Remove FixedArraySize error

Eagerly evaluate in `super_relate_consts`

Update test output

Rename `OpportunisticTypeResolver` to `OpportunisticVarResolver`

Add tests for uninferred consts during codegen

Resolve consts in OpportunisticTypeResolver

Add a test for a function taking a const param array as an argument

Relate identical parameters in array lengths

Special-case `.llvm` in mangler to fix segfaults

This commit special cases `.llvm` in the mangler to print `.llvm$6d$`
instead. This will avoid segfaults when names in a user's Rust code are
`llvm`.

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

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`

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.

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.

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

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.

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

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

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.