Rollup merge of #31055 - steveklabnik:alt-tags, r=alexcrichton

Rollup merge of #31054 - steveklabnik:a11y, r=alexcrichton

I've been interested in the accessibility of Rustdoc's output, but never did anything about it. Today, I ran a tool, and it suggested adjusting some colors.

Here's some screen shots. Before:

After:


As you can see, the link text is just a _shade_ darker.

Browsable: http://www.steveklabnik.com/rust-a11y-doc-testing/doc/std/

Rollup merge of #31050 - apasel422:issue-31048, r=Manishearth

Closes #31048

r? @Manishearth

Rollup merge of #31045 - Manishearth:diag-prim-shadow, r=steveklabnik

I tried to add an inline `span_suggestion()` to the error as well, but since generics don't have their own span it becomes too fragile/complicated to work.

r? @steveklabnik

fixes #19477

Rollup merge of #31035 - nathankleyn:improve-visibility-of-entry-api, r=steveklabnik

Responding to [a thread of discussion on the Rust subreddit](https://www.reddit.com/r/rust/comments/3racik/mutable_lifetimes_are_too_long_when_matching_an/),
it was identified that the presence of the Entry API is not duly
publicised. This commit aims to add some reasonable examples of
common usages of this API to the main example secion of the `HashMap`
documentation.

This is part of issue #29348.

Rollup merge of #31031 - brson:issue-30123, r=nikomatsakis

This was fixed in passing. Adding a regression test.

Rollup merge of #31019 - andreabedini:patch-1, r=alexcrichton

Documentation of `CStr::from_ptr` suggests using `str::from_utf8(slice.to_bytes()).unwrap()`
to obtain a `&str` but `CStr` has `CStr::to_str` that does exactly that.

(First PR, be nice :)

Rollup merge of #30997 - bluss:trait-default, r=nikomatsakis

Fix type parameter default error to mention type and trait definitions

Introduced in PR #30724, needs to mention that type parameter defaults
are legal in trait and type definitions too.

Auto merge of #31121 - Manishearth:prim-link, r=alexcrichton

We currently nest `<a>` tags which is doubleplusungood. So, for example, `(u8, u8)` will show up
with the left paren linked to the tuple primitive page, and the right paren unlinked, which looks ugly.

Handle generics in raw pointers

Auto merge of #30958 - dhodder:master, r=brson

The COPYRIGHT file should be updated to note that Rust no longer ships with AUTHORS.txt.

    $ git log -1 -- AUTHORS.txt
    commit 402749c539edcbc2d850ac3a782cace8661c68e6
    Author: Brian Anderson <banderson@mozilla.com>
    Date:   Wed Dec 2 22:16:08 2015 +0000

        Remove AUTHORS.txt and add-authors.sh

        Keeping this file up to date requires hours of work every release,
        even with the script. It is a fool's errand and we shall not do it
        any longer.

Auto merge of #30807 - kolloch:patch-1, r=steveklabnik

When I read the book, the following sentence of the reference documentation helped me greatly to understand slices:

"Slices are a view into a block of memory represented as a pointer and a length."

In this commit, I tried to integrate the gist of that into the slice section inside of "primitive-types.md". I am not a native speaker, so feel very free to improve the wording.

r? @steveklabnik

Auto merge of #31126 - arielb1:remote-errors, r=eddyb

Also removed an unused and similarly buggy error path.

Fixes #30535

r? @eddyb

Auto merge of #31124 - dirk:dirk/display-and-error-impls-for-string-parse-error, r=alexcrichton

Fixes #31106.

- [ ] I wasn't sure of the correct `#[stable(...)]` definition to use here. Happy to fix it if it's incorrect.
- [ ] `ParseError` is sort of an ephemeral non-error, but do let me know if the implementation of `error::Error` for it should return something more descriptive than "parse error".

Auto merge of #31116 - bluss:expect-out-cold, r=alexcrichton

Use cold functions for panic formatting Option::expect, Result::unwrap, expect

These methods are marked inline, but insert a big chunk of formatting
code, as well as other error path related code, such as
deallocating a std::io::Error if you have one.

We can explicitly separate out that code path into a function that is
never inline, since the panicking case should always be rare.

Auto merge of #31104 - nrc:save-fix-variant-data, r=alexcrichton

Auto merge of #31095 - alexcrichton:remove-old-files, r=brson

These are either not used at all today or they're super old and haven't been used in ages, so clean them out.

Simplify return for error::Error impl for string::ParseError

astconv: don't use as_local_node_id for ids in a Def

Fixes #30535

Implement error::Error and fmt::Display for string::ParseError

Fixes #31106.

Auto merge of #30917 - arthurprs:bs_bounds_check, r=alexcrichton

Avoid bounds checking for binary search. All calculated indexes are safe and the branch is useless.

Move cold panic functions in Option and Result

Move functions out of their impl blocks; avoids unnecessary
monomorphization by type parameters that are irrelevant to the message
output.

Use cold functions for panic formatting Option::expect, Result::unwrap etc

Option::expect, Result::unwrap, unwrap_err, expect

These methods are marked inline, but insert a big chunk of formatting
code, as well as other error path related code, such as deallocating
a std::io::Error if you have one.

We can explicitly separate out that code path into a function that is
never inline, since the panicking case should always be rare.

Correctly output links for primitive types which enclose their contents

Auto merge of #31087 - nikomatsakis:incr-comp-fulfillment-cache, r=arielb1

This is a fix for #30741. It simplifies dep-graph tracking for trait matching. I was experimenting with having a greater resolution here, but decided to pare back to just have one dep node for "trait resolutions on trait `Foo`", which means that adding an impl to the trait `Foo` will invalidate all fns that had to do any trait matching at all on `Foo`. This seems like a reasonable starting place.

Independently, I realized I had neglected to record a dependency from trans on typeck -- this is obviously needed, since trans consumes a bunch of data structures that typeck produces (but which are not currently individually tracked) -- and because trans assumes that typeck has been done. Eventually those are going to go away and be replaced with MIR, which will be tracked, so this edge would presumably be derived automatically then, but it's an obvious enough thing to want for now.

r? @arielb1

cc @michaelwoerister -- this might indirectly fix the problem you observed with the trans cache, though it'd be nice to try and craft an independent test case for that.

Avoid bounds check for slice binary search

Auto merge of #31084 - ranma42:cmd-no-unsafe, r=alexcrichton

Instead of transmuting, use a match; the compiler has learnt how to
optimize it.

Auto merge of #31080 - dhuseby:updating_libc_dep, r=alexcrichton

this pulls in a fix for the missing O_ACCMODE that was breaking the bsd builds.

Auto merge of #31077 - nagisa:mir-temp-promotion, r=dotdash

Fixes #31002

r? @nikomatsakis

Auto merge of #31074 - gmbonnet:stage0-libdir, r=alexcrichton

This fixes #27254.

On a 64-bit Linux machine, for example, `configure --libdir=/usr/local/lib64` was creating both `x86_64-unknown-linux-gnu/stage0/lib/rustlib` and `x86_64-unknown-linux-gnu/stage0/lib64/rustlib`. Crates from the stage0 snapshot, like `libcore`, are extracted to `x86_64-unknown-linux-gnu/stage0/lib/rustlib`, but the stage0 compiler was attempting to find them in `x86_64-unknown-linux-gnu/stage0/lib64/rustlib`, which has the highest priority on a 64-bit system.

The issue can be fixed by creating only `x86_64-unknown-linux-gnu/stage0/lib/rustlib`, since this is the only rustlib directory needed for stage0 anyways.

Auto merge of #31072 - arielb1:method-callee-cleanup, r=michaelwoerister

The old code was terribly ugly and was duplicated in several places.

r? @michaelwoerister

Auto merge of #31070 - sfackler:bufreader-box-slice, r=alexcrichton

Saves a word, and also prevents the impl from accidentally changing the
buffer length.

r? @alexcrichton

save-analysis: emit the right kind of variant

Auto merge of #31068 - sfackler:bufwriter-panic, r=alexcrichton

We don't want to write the same data twice.

Closes #30888

r? @alexcrichton

Auto merge of #31064 - retep998:msvc-host-dlls, r=alexcrichton

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

r? @alexcrichton

Auto merge of #30944 - fhahn:issue-29789-use-constant2, r=nagisa

This PR for  #29789 uses `rustc::repr::mir::Constant` in `ExprKind::Repeat`, which seems to fit quite nicely. Is there a reason for not re-using that type?

etc: Remove old num/libc generation code

etc: Remove the mingw-fix-include directory

This isn't used anywhere

etc: Remove old mklldef.py script

The compiler has since gained better support for this, so the script is no
longer necessary

mk: Remove all perf-related targets

I don't believe these have been used at all recently, and I doubt many of them
still work, so remove the stale support.

Auto merge of #30882 - petrochenkov:varnamesp, r=nrc

Tuple and unit variants from other crates weren't put into type namespace.
Now variant namespacing is aligned with struct namespacing and is not affected by the variant's crate of origin (struct -> type, tuple/unit -> type/value).
Additionally, struct variants from other crates are put into value namespace (struct variants from local crate were already in it). This is not a necessity, but a future proofing measure.

This fix can result in some new shadowing errors in cross-crate scenarios, crater reports [three regressions](https://github.com/rust-lang/rust/pull/30882#issuecomment-172369883).
[breaking-change]

Add Debug impl and erase region for TypedConstVal

Introduce and use TypedConstVal for Repeat

[MIR] use mir::repr::Constant in ExprKind::Repeat, close #29789

Auto merge of #31052 - bluss:split-at-mut-str, r=alexcrichton

Use raw pointers to avoid aliasing in str::split_at_mut

Introduce private function  from_raw_parts_mut for str to factor out the logic.

We want to use raw pointers here instead of duplicating a &mut str, to
be on safer ground w.r.t rust aliasing rules.

This has already been fixed for slices in PR #27358, issue #27357

resolve: Fix variant namespacing

add fixme

Remove `unsafe` code from `core::cmp`

Instead of transmuting, use a match; the compiler has learnt how to
optimize it.

Indicate that trans is always dependent on typeck

Revamp the fulfillment cache tracking to only cache trait-refs, which
was the major use-case, and to update the dep-graph. Other kinds of
predicates are now excluded from the cache because there is no easy way
to make a good dep-graph node for them, and because they are not
believed to be that useful. :)

Fixes #30741. (However, the test still gives wrong result for trans,
for an independent reason which is fixed in the next commit.)

Auto merge of #31028 - erickt:ast-json, r=michaelwoerister

The protocol for `serialize::{En,De}code` doesn't allow for two integers to be serialized next to each other.

Closes #31025.

cc @michaelwoerister

updating the libc dependency to pull in build breaker fixes

Auto merge of #31043 - shahn:arc_weak, r=alexcrichton

This was accidentally introduced in
7e2ffc7090a70fe8c77a0e03fcec3cb1387141f2,
b44ee371b8beea77aa1364460acbba14a8516559 and
36ba96ea3cfef575ddc5eea7754a1b70b50e2080.

[MIR] Promote temps to alloca on multi-assignment

Fixes #31002

Auto merge of #31024 - oli-obk:move_checks_out_of_librustc, r=arielb1

-    check_const
-    check_static_recursion
-    check_loop
-    check_rvalues

r? @arielb1

core: Use raw pointers to avoid aliasing in str::split_at_mut

Introduce private function from_raw_parts_mut for str to factor out the logic.

We want to use raw pointers here instead of duplicating a &mut str, to
be on safer ground w.r.t rust aliasing rules.

Auto merge of #31057 - bluss:memrchr-fallback, r=alexcrichton

Use the fallback impl for memrchr on non-linux

The memrchr code was never used(!). This brings the memrchr improvements to
non-linux platforms (LineWriter / buffered stdout benefits).

Previous PR #30381

clean up trans_static_method_callee and friends

configure: Do not generate two stage0 rustlib dirs when --libdir is set

move more checks out of librustc

Auto merge of #29520 - retep998:staticlib-naming-fiasco, r=alexcrichton

I'm not sure if this was the best way to go about it, but it seems to work.

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

r? @alexcrichton

Auto merge of #31042 - shahn:weak_annotations, r=alexcrichton

This was accidentally introduced in
7e2ffc7090a70fe8c77a0e03fcec3cb1387141f2.

Use a Box<[u8]> in BufReader

Saves a word, and also prevents the impl from accidentally changing the
buffer length.

Don't flush in BufWriter destructor after a panic in write

We don't want to write the same data twice.

Closes #30888

Auto merge of #31041 - brson:version, r=alexcrichton

Auto merge of #31037 - nrc:cached-ids, r=nikomatsakis

Blocks #30884

r? @nikomatsakis

cc @durka

Add host toolchain DLLs to PATH when executing link.exe
Fixes https://github.com/rust-lang/rust/issues/31063

Signed-off-by: Peter Atashian <retep998@gmail.com>

Auto merge of #31010 - petrochenkov:def, r=arielb1

All structs and their constructors are defined as `DefStruct`.
`DefTy` is splitted into `DefEnum` and `DefTyAlias`.
Ad hoc flag `bool is_structure` is removed from `DefVariant`, it was required in one place in resolve and could be obtained by other means.
Flag `bool is_ctor` is removed from `DefFn`, it wasn't really used for constructors outside of metadata decoding.

Observable effects:
More specific error messages are selected in some cases.
Two name resolution bugs fixed (https://github.com/rust-lang/rust/issues/30992 and FIXME in compile-fail/empty-struct-braces-expr.rs).

Fixes https://github.com/rust-lang/rust/issues/30992
Closes https://github.com/rust-lang/rust/issues/30361

Auto merge of #30982 - KiChjang:zst-collections-tests, r=bluss

Fixes #28518.

Auto merge of #30945 - nagisa:mir-optional-block-dest, r=nikomatsakis

As an attempt to make loop body destination be optional, author implemented a pretty self contained
change and deemed it to be (much) uglier than the alternative of just keeping the unit temporary.
Having the temporary created lazily also has a nice property of not figuring in the MIR of
functions which do not use loops of any sort.

r? @nikomatsakis

Auto merge of #30985 - alexcrichton:downgrade-jemaloc, r=brson

We've been seeing a lot of timeouts in tests on the bots and investigation ended
pointing to jemalloc/jemalloc#315 as the culprit. Unfortunately it looks like
that doesn't seem to have a fix on the way soon, so let's temporarily downgrade
back to the previous version of jemalloc we were using (where #30434 was the
most recent upgrade)

Rename Def's variants and don't reexport them

Fix associated const resolution on structs

Refactor definitions of ADTs in rustc::middle::def

syntax: Fix encoding and decoding spans

The protocol for `serialize::{En,De}code` doesn't allow for two
integers to be serialized next to each other. This switches the
protocol to serializing `Span`s as a struct. rbml structs don't
have any overhead, so the metadata shouldn't increase in size,
but it allows the json format to be properly generated, albeit
slightly more heavy than when it was just serializing a span as
a u64.

Closes #31025.

s

Downgrade bundled jemalloc version

We've been seeing a lot of timeouts in tests on the bots and investigation ended
pointing to jemalloc/jemalloc#315 as the culprit. Unfortunately it looks like
that doesn't seem to have a fix on the way soon, so let's temporarily downgrade
back to the previous version of jemalloc we were using (where #30434 was the
most recent upgrade)

tweak trait css

tweak struct colors

Auto merge of #30872 - pitdicker:expand_open_options, r=alexcrichton

Tracking issue: #30014

This implements the RFC and makes a few other changes.
I have added a few extra tests, and made the Windows and
Unix code as similar as possible.

Part of the RFC mentions the unstable OpenOptionsExt trait
on Windows (see #27720). I have added a few extra methods
to future-proof it for CreateFile2.

std: Use the fallback impl for memrchr on non-linux

This code was never used(!). This brings the memrchr improvements to
non-linux platforms (LineWriter / buffered stdout benefits).

Add alt tags for logos

tweak colors for a11y

Add tests for collections to work with ZSTs

Correct code in E0382 explanation

Closes #31048

Add tests for various diverging & converging loops

Mostly testing borrowck and typeck behaviour in presence of `loop` expression

Improve E0317 long diagnostics

Auto merge of #31038 - gereeter:btree-panic, r=Gankro

…ving elements into it.

Fixes #31029.

Remove raw pointer from OpenOptions struct
Otherwise it is not Send and Sync anymore

Fix incorrect stability annotations for arc::Weak

This was accidentally introduced in
7e2ffc7090a70fe8c77a0e03fcec3cb1387141f2,
b44ee371b8beea77aa1364460acbba14a8516559 and
36ba96ea3cfef575ddc5eea7754a1b70b50e2080.

Fix incorrect stability annotations for Weak

This was accidentally introduced in
7e2ffc7090a70fe8c77a0e03fcec3cb1387141f2.

Bump version to 1.8

Auto merge of #30894 - antrik:debug-mpsc, r=brson

Minimal fix for https://github.com/rust-lang/rust/issues/30563

This covers all the public structs I think; except for Iter and
IntoIter, which I don't know if or how they should be handled.

In BTreeMap's merge function, adjust the length of the node before moving elements into it.

Fix a bug with caching ids in the HIR lowering with nested lowered nodes

Blocks #30884

Remove leftover import of `std::str` in doc test

Auto merge of #30820 - oli-obk:docs/wrapping_ops, r=alexcrichton

r? @steveklabnik

Add examples of the Entry API to the HashMap documentation.

Responding to [a thread of discussion on the Rust
subreddit](https://www.reddit.com/r/rust/comments/3racik/mutable_lifetimes_are_too_long_when_matching_an/),
it was identified that the presence of the Entry API is not duly
publicised. This commit aims to add some reasonable examples of
common usages of this API to the main example secion of the `HashMap`
documentation.

This is part of issue #29348.

remove the `SimplifiedType` from the `DepNode` for now, just to keep
size of `DepNode` smaller and because we are not that fine-grained yet
anyhow

[MIR] Reintroduce the unit temporary

An attempt to make loop body destination be optional, author implemented a pretty self contained
change and deemed it to be (much) uglier than the alternative of just keeping the unit temporary.
Having the temporary created lazily also has a nice property of not figuring in the MIR of
functions which do not use loops of any sort.

Add test for #30123