-
Notifications
You must be signed in to change notification settings - Fork 13.3k
Stage0 bootstrap update #140887
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
Stage0 bootstrap update #140887
Conversation
Some changes occurred to MIR optimizations cc @rust-lang/wg-mir-opt Some changes occurred to the CTFE machinery Some changes occurred in compiler/rustc_attr_data_structures Some changes occurred in library/core/src/fmt/rt.rs cc @m-ou-se This PR changes how LLVM is built. Consider updating src/bootstrap/download-ci-llvm-stamp. Some changes occurred in compiler/rustc_sanitizers cc @rcvalle Some changes occurred in compiler/rustc_codegen_ssa Some changes occurred to the intrinsics. Make sure the CTFE / Miri interpreter cc @rust-lang/miri, @RalfJung, @oli-obk, @lcnr Some changes occurred in compiler/rustc_attr_parsing rust-analyzer is developed in its own repository. If possible, consider making this change to rust-lang/rust-analyzer instead. cc @rust-lang/rust-analyzer This PR modifies If appropriate, please update Some changes occurred to the platform-builtins intrinsics. Make sure the cc @antoyo, @GuillaumeGomez, @bjorn3, @calebzulawski, @programmerjake |
This comment has been minimized.
This comment has been minimized.
also @zetanumbers @petrochenkov since you were also involved with async drop. for some reason there is an ICE when compiling the standard library after the bootstrap bump, could you take a look at it? ideally there's some small bugfix that can be backported to beta |
Minimized repro, in case it helps: #140423 (comment) |
On mobile for the next 6h, so I cant provide PRs, but just early return from the async mir shim code to produce a dummy mir body. Not dropping anything isn't worse than before the PR and we'll fix it on nightly again. No stable code is affected by essentially disabling the feature on beta |
I'd say the code is sound, as a |
you can take the fix from #140902 it's hacky but it does the job with no risk and we'll fix it cleanly on master |
Prepared the backport PR. |
58651d1
to
779dd35
Compare
This comment has been minimized.
This comment has been minimized.
This comment has been minimized.
This comment has been minimized.
818b50e
to
6f6a9a5
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
r=me
@bors r+ rollup=never p=1 |
@bors r- squash pls :> |
6f6a9a5
to
8ce7783
Compare
@bors r=compiler-errors p=1 rollup=never |
☀️ Test successful - checks-actions |
What is this?This is an experimental post-merge analysis report that shows differences in test outcomes between the merged PR and its parent PR.Comparing f6b5da7 (parent) -> 4eca99a (this PR) Test differencesShow 33577 test diffsStage 0
Stage 1
Stage 2
(and 16586 additional test diffs) Additionally, 16891 doctest diffs were found. These are ignored, as they are noisy. Job group index
Test dashboardRun cargo run --manifest-path src/ci/citool/Cargo.toml -- \
test-dashboard 4eca99a18eab3d4e28ed1ce3ee620d442955a470 --output-dir test-dashboard And then open Job duration changes
How to interpret the job duration changes?Job durations can vary a lot, based on the actual runner instance |
Finished benchmarking commit (4eca99a): comparison URL. Overall result: no relevant changes - no action needed@rustbot label: -perf-regression Instruction countThis benchmark run did not return any relevant results for this metric. Max RSS (memory usage)Results (primary -0.7%, secondary -3.2%)This is a less reliable metric that may be of interest but was not used to determine the overall result at the top of this comment.
CyclesResults (primary -0.4%)This is a less reliable metric that may be of interest but was not used to determine the overall result at the top of this comment.
Binary sizeThis benchmark run did not return any relevant results for this metric. Bootstrap: 771.902s -> 771.795s (-0.01%) |
This PR follows the release process to update the stage0 compiler.
The only thing of note is 58651d1, which was flagged by clippy as a correctness fix. I think allowing that lint in our case makes sense, but it's worth to have a second pair of eyes on it.
r? @Mark-Simulacrum