Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Rollup of 16 pull requests #77366

Closed
wants to merge 33 commits into from
Closed

Rollup of 16 pull requests #77366

wants to merge 33 commits into from

Conversation

jonas-schievink
Copy link
Contributor

Successful merges:

Failed merges:

r? @ghost

matthiaskrgr and others added 30 commits September 24, 2020 13:38
This fixes rust-lang/wg-cargo-std-aware#53

Now users will be able to do:
```
cargo build -Zbuild-std=core -Zbuild-std-features=compiler-builtins-mem
```
and correctly get the Rust implemenations for `memcpy` and friends.

Signed-off-by: Joe Richey <joerichey@google.com>
…k() in std::sys::unix::process::Command::spawn()

`assert!` panics on failure, which is not signal-safe.
mem::ManuallyDrop::new -> ManuallyDrop::new
Add regression test

This adds a regression test for #75777, effectively closing it since it is solved on nightly and beta.
more tiny clippy cleanups

commits stand alone and can be reviewed one by one
BTreeMap: keep an eye out on the size of the main components

r? @Mark-Simulacrum
Ensure that all LLVM components requested by tests are available on CI

Addresses #75064 (comment)

I used an environment variable because passing a command line option all the way from CI to compiletest would be just too much hassle for this task.
I added a new variable, but any of the already existing ones defined by CI could be used instead.
r? @Mark-Simulacrum
library: Forward compiler-builtins "mem" feature

This fixes rust-lang/wg-cargo-std-aware#53

Now users will be able to do:
```
cargo build -Zbuild-std=core -Zbuild-std-features=compiler-builtins-mem
```
and correctly get the Rust implemenations for `memcpy` and friends.

Signed-off-by: Joe Richey <joerichey@google.com>
liveness: Use Option::None to represent absent live nodes

No functional changes intended.
move candidate_from_obligation_no_cache

It's only called from `candidate_from_obligation` which is already in this file.
Add unstable book docs for `-Zunsound-mir-opts`

The `-Zunsound-mir-opts` flag was added in #76899.
…rochenkov

Don't fire `const_item_mutation` lint on writes through a pointer

Fixes #77321
Use `rtassert!` instead of `assert!` from the child process after fork() in std::sys::unix::process::Command::spawn()

As discussed in #73894, `assert!` panics on failure, which is not signal-safe, and `rtassert!` is a suitable replacement.

Fixes #73894.

r? @Amanieu @cuviper @joshtriplett
Add test for async/await combined with const-generics.

Fixes #74906.
Alloc vec use imported path

mem::ManuallyDrop::new -> ManuallyDrop::new

cc @the8472
… r=lcnr

Validate `rustc_args_required_const`

Fixes #74608.
Add test for issue #74761

Adds test for and closes #74761 which previously crashed compiler.
Update books

## rust-by-example

5 commits in 19f0a0372af497b34369cf182d9d16156cab2969..7d3ff1c12db08a847a57a054be4a7951ce532d2d
2020-08-26 09:38:48 -0300 to 2020-09-28 15:54:25 -0300
- Prevent duplicated error information in "wrapping errors" example (rust-lang/rust-by-example#1375)
- Avoid lifetime annotation in struct example (rust-lang/rust-by-example#1378)
- Clone.md comment and variable name change (rust-lang/rust-by-example#1379)
- Replace panic example with a simpler version (rust-lang/rust-by-example#1380)
- Prefer `const` over `static` (rust-lang/rust-by-example#1381)

## embedded-book

1 commits in 0cd2ca116274b915924c3a7e07c1e046b6f19b77..dd310616308e01f6cf227f46347b744aa56b77d9
2020-08-19 10:33:15 +0000 to 2020-09-26 08:54:08 +0000
- Add instructions for ARM v8 instruction sets  (rust-embedded/book#265)
@jonas-schievink
Copy link
Contributor Author

@bors r+ rollup=never p=16

@rustbot modify labels: rollup

@bors
Copy link
Contributor

bors commented Sep 30, 2020

📌 Commit 71e4b21 has been approved by jonas-schievink

@rustbot rustbot added the rollup A PR which is a rollup label Sep 30, 2020
@bors bors added the S-waiting-on-bors Status: Waiting on bors to run and complete tests. Bors will change the label on completion. label Sep 30, 2020
@Dylan-DPC-zz
Copy link

ah the rollup race.. will close the other one :D

@jonas-schievink
Copy link
Contributor Author

Sorry, should have approved this one immediately but I got sidetracked

@bors
Copy link
Contributor

bors commented Sep 30, 2020

⌛ Testing commit 71e4b21 with merge 614d8eb7a357215e07685f5d0118d08ceb77ed24...

@bors
Copy link
Contributor

bors commented Sep 30, 2020

💔 Test failed - checks-actions

@bors bors added S-waiting-on-review Status: Awaiting review from the assignee but also interested parties. and removed S-waiting-on-bors Status: Waiting on bors to run and complete tests. Bors will change the label on completion. labels Sep 30, 2020
@jonas-schievink jonas-schievink deleted the rollup-830m5on branch September 30, 2020 18:53
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
rollup A PR which is a rollup S-waiting-on-review Status: Awaiting review from the assignee but also interested parties.
Projects
None yet
Development

Successfully merging this pull request may close these issues.