-
Notifications
You must be signed in to change notification settings - Fork 13.2k
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
Update to LLVM 20 rc 3 #137661
Update to LLVM 20 rc 3 #137661
Conversation
|
This comment has been minimized.
This comment has been minimized.
https://github.com/rust-lang/rust/pull/137594/files#diff-e5255e85d66ab827c6eff26a377f41800be547791e268e7e2a9abf487943cf12 (removal of cc @RalfJung |
Oh that is unfortunate, the presence of |
@bors r+ rollup=never |
@bors r=cuviper |
🌲 The tree is currently closed for pull requests below priority 100. This pull request will be tested once the tree is reopened. |
@bors p=5 (encouraging more rollup-nevers to go through before more rollups) |
Due to the usage of smaller CI runners, jobs that have to rebuild LLVM (such as in this PR) take a very long time. If this times out (the timeout is 6h, so it's likely, given that it's 5h30m now and two jobs are still running), just retry it. It will be faster after retried due to Docker and sccache being primed. |
☀️ Test successful - checks-actions |
Finished benchmarking commit (f4a216d): 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 8.7%)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 (secondary -1.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.
Binary sizeThis benchmark run did not return any relevant results for this metric. Bootstrap: 774.434s -> 773.437s (-0.13%) |
The fix to this issue was synced in [1] so we should no longer need to keep aarch64 pinned. This reverts commit b2bcfc8. [1]: rust-lang/rust#137661
The fix to this issue was synced in [1] so we should no longer need to keep aarch64 pinned. This reverts commit b2bcfc8. [1]: rust-lang/rust#137661
Fixes a compiler-builtins miscompile (see rust-lang/compiler-builtins#760).