Auto merge of #29012 - tari:issue-28676, r=luqmana

Fixes #28676.

There doesn't seem to be a good way to add a test for this, but I tested the repro in #28676 and confirmed it now yields the correct result.

Add test case for #28676.

Auto merge of #29044 - Ms2ger:cleanup-log, r=alexcrichton

Auto merge of #29003 - tari:freebsd-current_exe-vec, r=alexcrichton

Fixes #28995.

Auto merge of #29001 - arielb1:normalized-foreign, r=eddyb

This is needed as item types are allowed to be unnormalized.

Fixes an ICE that occurs when foreign function signatures contained
an associated type.

Fixes #28983

Auto merge of #28909 - GuillaumeGomez:patch-3, r=pnkfelix

r? @Manishearth

Remove the unnecessary local variable in set_logger.

Use if-let rather than map() in parse_logging_spec.

Auto merge of #28827 - thepowersgang:unsafe-const-fn-2, r=Aatch

This is the original test implementation, which works according to the tests I wrote, but might need a review.

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

- Successful merges: #28991, #29004, #29006, #29013, #29016, #29024, #29027, #29028, #29029, #29032, #29035
- Failed merges:

fix tidy

Rollup merge of #29035 - rutsky:patch-6, r=steveklabnik

r? @steveklabnik

The link is broken here: <https://doc.rust-lang.org/std/io/#types>.

Looks like crate documentation generator uses only first paragraph of the module documentation and
so doesn't resolve the link defined below.

Probably this behaviour of the documentation generator should be considered as a bug and should be reported.

Rollup merge of #29032 - goyox86:goyox86/rusfmting-librustc_bitflags, r=alexcrichton

Hi Rustaceans!

This is the result of running latest rustfmt on librustc_bitflags!

//cc @nrc

Rollup merge of #29029 - steveklabnik:iter_function_docs, r=alexcrichton

Flesh the docs out with examples.

Rollup merge of #29028 - Seeker14491:patch-1, r=Manishearth

Having this code section hidden is misleading because it makes it look like implementing Circle for Foo automatically makes Foo implement Shape.

Rollup merge of #29024 - mseri:patch-1, r=nrc

r? @nrc

Rollup merge of #29016 - sanxiyn:data-layout, r=alexcrichton

Rollup merge of #29013 - chrisccerami:fix_broken_lifetime_elision_link, r=alexcrichton

This link was added in #28842 but doesn't work at https://doc.rust-lang.org/nightly/book/lifetimes.html. What works in my markdown preview doesn't work live, and vice versa.

Rollup merge of #29006 - arielb1:callee-outlives-call, r=pnkfelix

This rather crucial requirement was not checked. In most cases, that
didn't cause any trouble because the argument types are required to
outlive the call and are subtypes of a subformula of the callee type.

However, binary ops are taken by ref only indirectly, without it being
marked in the argument types, which led to the argument types not being
constrained anywhere causing spurious errors (as these are basically
unconstrainable, I don't think this change can break code). Of course,
the old way was also incorrent with contravariance, but that is still
unsound for other reasons.

This also improves rustc::front to get RUST_LOG to *somewhat* work.

Fixes #28999. That issue is one of the several regression introduced by #28669.

r? @pnkfelix

Rollup merge of #29004 - frewsxcv:regression-test-22814, r=alexcrichton

Closes #22814

Rollup merge of #28991 - goyox86:goyox86/rustfmting-liblog-II, r=alexcrichton

Hi Rustaceans!

This is the second version of running rustfmt on liblog https://github.com/rust-lang/rust/pull/28898. There are only rustfmt suggested changes. Because I think that some patterns here (that I fixed up in the prev PR need to be discussed in detail.

//cc @nrc

Auto merge of #28816 - petrochenkov:unistruct, r=nrc

This patch uses the same data structures for structs and enum variants in AST and HIR. These changes in data structures lead to noticeable simplification in most of code dealing with them.
I didn't touch the top level, i.e. `ItemStruct` is still `ItemStruct` and not `ItemEnum` with one variant, like in the type checker.
As part of this patch, structures and variants get the `kind` field making distinction between "normal" structs, tuple structs and unit structs explicit instead of relying on the number of fields and presence of constructor `NodeId`. In particular, we can now distinguish empty tuple structs from unit structs, which was impossible before! Comprehensive tests for empty structs are added and some improvements to empty struct feature gates are made. Some tests don't pass due to issue https://github.com/rust-lang/rust/issues/28692 , they are still there for completeness, but are commented out.
This patch fixes issue mentioned in https://github.com/rust-lang/rust/issues/16819#issuecomment-139509861, now emptiness of tuple structs is checked after expansion.
It also touches https://github.com/rust-lang/rust/issues/28750 by providing span for visit_struct_def
cc https://github.com/rust-lang/rust/pull/28336

r? @nrc

Auto merge of #29026 - steveklabnik:rollup, r=steveklabnik

- Successful merges: #28988, #28989, #28990, #28997, #29007, #29015
- Failed merges: #28906

fix link on std::result::Result

The link is broken here: <https://doc.rust-lang.org/std/io/#types>.

Looks like crate documentation generator uses only first paragraph of the module documentation and
so doesn't resolve the link defined below.

Revert "fixups"

This reverts commit 5b8335ede4214f802eab765ca99b5afa9057973c.

Rustfmt-ing librustc_bitflags.

Document the free functions of std::iter

Flesh the docs out with examples.

Unhide some code from the Traits section

Having this code section hidden is misleading because it makes it look like implementing Circle for Foo automatically makes Foo implement Shape.

Rollup merge of #29015 - dripton:master, r=alexcrichton

Just a single-character typo fix.

Rollup merge of #29007 - dnwade:patch-2, r=alexcrichton

r? @steveklabnik

Rollup merge of #28997 - bezelga:master, r=steveklabnik

Fixing some words in the beginning of the Ownership system chapters.

Rollup merge of #28990 - ahmedcharles:libarena, r=nrc

Rollup merge of #28989 - ahmedcharles:liballoc_system, r=nrc

Rollup merge of #28988 - ahmedcharles:liballoc_jemalloc, r=nrc

Add E0511 error explanation

fixups

rustfmt libfmt_macros

Comment on the purpose(s) of NodeId in VariantData

Merge VariantData and VariantData_

Fix rebase 2

Merge struct fields and struct kind

Dict -> Struct, StructDef -> VariantData, def -> data

Fix rebase

Fix failing privacy test

Test and gate empty structures and variants better

Provide span for visit_struct_def + remove some dead code

Remove now redundant NodeId from Variant

Decouple structure kinds from NodeIds

Unify structures and enum variants in HIR

Unify structures and enum variants in AST

Manually alligned comments.

Re-running updated rustfmt on liblog.

rustfmt suggested changes.

Avoid using getDataLayout, deprecated in LLVM 3.7

Typo fix

Auto merge of #28993 - frewsxcv:regression-test-17336, r=alexcrichton

Closes #17336

Fix Lifetime Elision link in lifetimes.md

Auto merge of #28987 - ahmedcharles:liballoc, r=brson

typo

win64/msvc: large or oddly-sized types pass by-ref

require a method callee's type to outlive the call

This rather crucial requirement was not checked. In most cases, that
didn't cause any trouble because the argument types are required to
outlive the call and are subtypes of a subformula of the callee type.

However, binary ops are taken by ref only indirectly, without it being
marked in the argument types, which led to the argument types not being
constrained anywhere causing spurious errors (as these are basically
unconstrainable, I don't think this change can break code). Of course,
the old way was also incorrent with contravariance, but that is still
unsound for other reasons.

This also improves rustc::front to get RUST_LOG to *somewhat* work.

Fixes #28999

Auto merge of #28986 - ahmedcharles:error-index-generator, r=brson

Add regression test for #22814

Closes #22814

normalize the types of foreign functions

This is needed as item types are allowed to be unnormalized.

Fixes an ICE that occurs when foreign function signatures contained
an associated type.

Fixes #28983

Fix docs about borrowing and lifetimes

Add regression test for #17336

Closes #17336

Auto merge of #28914 - steveklabnik:doc_iterator, r=alexcrichton

This replaces what was there with a comprehensive overview.

Thanks to @hoverbear for suggesting that these docs needed improvement.

Write better module-level docs for std::iter

This replaces what was there with a comprehensive overview.

Don't use a Vec in os::current_exe on FreeBSD.

Auto merge of #28985 - ahmedcharles:driver, r=nrc

Run rustfmt on error-index-generator.

Run rustfmt on driver.

Run rustfmt on liballoc_system.

Run rustfmt on liballoc_jemalloc.

Run rustfmt on liballoc.

Run rustfmt on libarena.

Auto merge of #28969 - chrisccerami:link_to_ffi_in_concurrency_chapter, r=steveklabnik

Auto merge of #28973 - nrc:match-source, r=Manishearth

r? @Manishearth

Properly set the MatchSource for for loops

Auto merge of #28960 - barosl:home-temp-dir-doc, r=alexcrichton

This concern was raised by #28940.

Auto merge of #28948 - steveklabnik:gh28944, r=alexcrichton

"Crate features" isn't a thing in Rust, but you do need to add this line
to your crate root.

Fixes #28944

Link to FFI in Concurrency chapter

Auto merge of #28966 - ykomatsu:trpl, r=steveklabnik

Auto merge of #28964 - eminence:docfix, r=sfackler

r? @steveklabnik

Fix some typos

Auto merge of #28965 - bluss:ptr-write-doc, r=huonw

core: Update the safety docs for core::ptr::write

Updated to reflect that not dropping a value is safe (but discouraged).

core: Update the safety docs for core::ptr::write

Updated to reflect that not dropping a value is safe (but discouraged).

Trivial typo fix:  from_utrf8 should be from_utf8

Clarify the behavior of `std::env::home_dir` and `std::env::temp_dir`

This concern was raised by #28940.

Auto merge of #28941 - jcaudle:rustfmt/librand-distributions-normal, r=nrc

This is my first PR to rust and first collaboration on a rust project, but I think the formatting looked ok. If this works out, I hope to go through more of librand.

Auto merge of #28949 - iwillspeak:rustfmt-libflate, r=nrc

This commit updates the libflate crate with the formatting output of
`rustfmt`.

r? @nrc

Update Libflate Formatting

This commit updates the libflate crate with the formatting output of
`rustfmt`.

Diagnostic change: crate features -> crate root

"Crate features" isn't a thing in Rust, but you do need to add this line
to your crate root.

Fixes #28944

Auto merge of #28945 - shama:doc-typos, r=steveklabnik

Just a few typos found in the docs. Thanks!

doc: fixing typos

Auto merge of #28922 - panicbit:trpl-missing-docs, r=steveklabnik

Auto merge of #28938 - GlenDC:master, r=Manishearth

Here is my attempt to resolve issue #28822, @Manishearth.
Please let me know if it's fine. And if not, what should I do instead?

This issue felt like quite a good start for some rust contributions. It allows me to get used to the workflow  and codebase of rust in an easy-to-swallow manner. Are there any other issues you would recommend me to look at? :) Would love to do some more stuff!

Whitelisting PatWild for E0022. Fix #28822

Auto merge of #28930 - steveklabnik:update_pr, r=steveklabnik

https://github.com/rust-lang/rust/pull/27813#issuecomment-146842041

Auto merge of #28920 - dotdash:const_align, r=eddyb

For enum variants, the default alignment for a specific variant might be
lower than the alignment of the enum type itself. In such cases we, for
example, generate memcpy calls with an alignment that's higher than the
alignment of the constant we copy from.

To avoid that, we need to explicitly set the required alignment on
constants.

Fixes #28912.