Skip to content

Rollup of 9 pull requests - #96108

Merged
bors merged 28 commits into
rust-lang:masterfrom
Dylan-DPC:rollup-t5f2fc9
Apr 16, 2022
Merged

Rollup of 9 pull requests#96108
bors merged 28 commits into
rust-lang:masterfrom
Dylan-DPC:rollup-t5f2fc9

Conversation

@Dylan-DPC

Copy link
Copy Markdown
Member

Successful merges:

Failed merges:

r? @ghost
@rustbot modify labels: rollup

Create a similar rollup

Michcioperzand others added 28 commits March 4, 2022 13:39
Also sort sidebar alphabetically by document filename
The code was broken anyway, if the deaggregator is disabled, it would have ICE on any non-enum Adt
* Links MaybeUninit::uninit_array to meta-tracking issue
* Links MaybeUninit::array_assume_init to meta-tracking issue
* Unstably constifies MaybeUninit::array_assume_init
…pnkfelix
Only add codegen backend to dep info if -Zbinary-dep-depinfo is used
I am currently migrating the cg_clif build system from using a binary linked to the codegen backend as rustc replacement to passing `-Zcodegen-backend` instead. Without this PR this would force cargo to rebuild the sysroot on any change to the codegen backend even if I explicitly specify that I want it to be preserved, which would make development of cg_clif a lot slower. If you still want to have changes to the codegen backend invalidate the cargo build cache you can explicitly specify `-Zbinary-dep-depinfo`.
cc ``@eddyb`` as the codegen backend was initially added to the depinfo for rust-gpu.
Add missing links in platform support docs
I was looking at m68k support and saw that https://doc.rust-lang.org/rustc/platform-support.html and the sidebar there were missing some links to target documentation
…i-obk
make unaligned_references lint deny-by-default
This lint has been warn-by-default for a year now (since rust-lang#82525), so I think it is time to crank it up a bit. Code that triggers the lint causes UB (without `unsafe`) when executed, so we really don't want people to write code like this.
…comment, r=petrochenkov
Improve diagnostics for unterminated nested block comment
closerust-lang#95283
(This is my first time try to messing around with rust compiler and might get a lot of things wrong... 🙇 )
…ubilee
implement SIMD gather/scatter via vector getelementptr
Fixesrust-lang/portable-simd#271
However, I don't *really* know what I am doing here... Cc ``@workingjubilee`` ``@calebzulawski``
I didn't do anything for cranelift -- ``@bjorn3`` not sure if it's okay for that backend to temporarily break. I'm happy to cherry-pick a patch that adds cranelift support. :)
…rochenkov
Consider lifetimes when comparing types for equality in MIR validator
Closesrust-lang#95978 .
…fJung
Remove some now-dead code that was only relevant before deaggregation.
The code was broken anyway, if the deaggregator is disabled, it would have ICEd on any non-enum Adt
r? ```@RalfJung```
…thomcc
[test] Add test cases for untested functions for BTreeMap
- add `pop_first()`, `pop_last()`, `get_key_value()` and `try_insert()` test cases
…up, r=dtolnay
MaybeUninit array cleanup
* Links `MaybeUninit::uninit_array` to meta-tracking issue
* Links `MaybeUninit::array_assume_init` to meta-tracking issue
* Unstably constifies `MaybeUninit::array_assume_init`
Another thing worth mentioning: this splits the const feature flag for `maybe_uninit_uninit_array` into `const_maybe_uninit_uninit_array` to avoid weird cases where only one gets stabilised.
Note that it may be desired to keep the `array_assume_init` method linked to its dedicated issue, but at least for now, I decided to link to the meta-tracking issue so that all of the methods lead users to the same place. But I can revert that bit if desired.
The meta-tracking issue that I filed is rust-lang#96097.
@rustbotrustbot added T-compiler Relevant to the compiler team, which will review and decide on the PR/issue. T-libs Relevant to the library team, which will review and decide on the PR/issue. labels Apr 16, 2022
@rustbotrustbot added the rollup A PR which is a rollup label Apr 16, 2022
@Dylan-DPC

Copy link
Copy Markdown
MemberAuthor

@bors r+ rollup=never p=5

@bors

bors commented Apr 16, 2022

Copy link
Copy Markdown
Collaborator

📌 Commit f559cf9 has been approved by Dylan-DPC

@borsbors added the S-waiting-on-bors Status: Waiting on bors to run and complete tests. Bors will change the label on completion. label Apr 16, 2022
@bors

bors commented Apr 16, 2022

Copy link
Copy Markdown
Collaborator

⌛ Testing commit f559cf9 with merge c842240...

@bors

bors commented Apr 16, 2022

Copy link
Copy Markdown
Collaborator

☀️ Test successful - checks-actions
Approved by: Dylan-DPC
Pushing c842240 to master...

@borsbors added the merged-by-bors This PR was explicitly merged by bors. label Apr 16, 2022
@bors
bors merged commit c842240 into rust-lang:masterApr 16, 2022
@rustbotrustbot added this to the 1.62.0 milestone Apr 16, 2022
@rust-highfive

Copy link
Copy Markdown
Contributor

📣 Toolstate changed by #96108!

Tested on commit c842240.
Direct link to PR: #96108

💔 miri on windows: test-pass → test-fail (cc @RalfJung@oli-obk@eddyb).
💔 miri on linux: test-pass → test-fail (cc @RalfJung@oli-obk@eddyb).
💔 embedded-book on windows: test-pass → test-fail (cc @andre-richter@therealprof@jamesmunns@adamgreig).
💔 embedded-book on linux: test-pass → test-fail (cc @andre-richter@therealprof@jamesmunns@adamgreig).

rust-highfive added a commit to rust-lang-nursery/rust-toolstate that referenced this pull request Apr 16, 2022
Tested on commit rust-lang/rust@c842240.
Direct link to PR: <rust-lang/rust#96108>
💔 miri on windows: test-pass → test-fail (cc @RalfJung@oli-obk@eddyb).
💔 miri on linux: test-pass → test-fail (cc @RalfJung@oli-obk@eddyb).
💔 embedded-book on windows: test-pass → test-fail (cc @andre-richter@therealprof@jamesmunns@adamgreig).
💔 embedded-book on linux: test-pass → test-fail (cc @andre-richter@therealprof@jamesmunns@adamgreig).
@rust-timer

Copy link
Copy Markdown
Collaborator

Finished benchmarking commit (c842240): comparison url.

Summary:

  • Primary benchmarks: no relevant changes found
  • Secondary benchmarks: mixed results
Regressions 😿
(primary)
Regressions 😿
(secondary)
Improvements 🎉
(primary)
Improvements 🎉
(secondary)
All 😿 🎉
(primary)
count103020
mean2N/A1.1%N/A-0.5%N/A
maxN/A1.1%N/A-0.7%N/A

If you disagree with this performance assessment, please file an issue in rust-lang/rustc-perf.

Next Steps: If you can justify the regressions found in this perf run, please indicate this with @rustbot label: +perf-regression-triaged along with sufficient written justification. If you cannot justify the regressions please open an issue or create a new PR that fixes the regressions, add a comment linking to the newly created issue or PR, and then add the perf-regression-triaged label to this PR.

@rustbot label: +perf-regression

Footnotes

  1. number of relevant changes

  2. the arithmetic mean of the percent change

@rustbotrustbot added the perf-regression Performance regression. label Apr 16, 2022
@ehussehuss mentioned this pull request Apr 16, 2022
@Dylan-DPC
Dylan-DPC deleted the rollup-t5f2fc9 branch April 16, 2022 21:41
borsBot added a commit to rust-embedded/book that referenced this pull request Apr 20, 2022
317: Fix use of unaligned references in example r=eldruin a=adamgreig
A recent change in Rust to deny unaligned reference by default (rust-lang/rust#95372) caused the upstream rust build of this book to break: rust-lang/rust#96108 (comment)
This PR removes the unaligned references, instead using `addr_of!()` to get a raw pointer to print.
Co-authored-by: Adam Greig <adam@adamgreig.com>
@pnkfelix

Copy link
Copy Markdown
Contributor
  • This observed 1% regression on ctfe-stress-5.
  • Much like PR Rollup of 6 pull requests #96015, I do not think I would put individual blame on this rollup PR for the problems we may be seeing over time in ctfe-stress-5.

@rustbot label: +perf-regression-triaged

@rustbotrustbot added the perf-regression-triaged The performance regression has been triaged. label Apr 21, 2022
@pnkfelixpnkfelix mentioned this pull request Apr 21, 2022
Sign up for freeto join this conversation on GitHub. Already have an account? Sign in to comment

Labels

merged-by-borsThis PR was explicitly merged by bors.perf-regressionPerformance regression.perf-regression-triagedThe performance regression has been triaged.rollupA PR which is a rollupS-waiting-on-borsStatus: Waiting on bors to run and complete tests. Bors will change the label on completion.T-compilerRelevant to the compiler team, which will review and decide on the PR/issue.T-libsRelevant to the library team, which will review and decide on the PR/issue.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

14 participants

@Dylan-DPC@bors@rust-highfive@rust-timer@pnkfelix@rustbot@Michcioperz@bjorn3@RalfJung@JakobDegen@yue4u@oli-obk@Gumichocopengin8@clarfonthey