-
Notifications
You must be signed in to change notification settings - Fork 13.8k
Update LLVM to 21.1.3 #147518
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 LLVM to 21.1.3 #147518
Conversation
@bors try jobs=x86_64-gnu-distcheck |
This comment has been minimized.
This comment has been minimized.
Update LLVM to 21.1.3 try-job: x86_64-gnu-distcheck
This comment has been minimized.
This comment has been minimized.
💔 Test for d6659ce failed: CI. Failed jobs:
|
@bors try jobs=x86_64-gnu-distcheck |
Update LLVM to 21.1.3 try-job: x86_64-gnu-distcheck
This comment has been minimized.
This comment has been minimized.
rustbot has assigned @Mark-Simulacrum. Use |
|
|
cc @marcoieni (largedisk runner) |
On that note, would you be willing to approve that "Remove a temporary directory" commit for beta backport as well? I think I would just grab that on top of the already-approved #146953 (which failed distcheck in backport before), since we don't have enough motivation for backporting the additional LLVM changes here AFAIK. |
You can also r=me for the bootstrap change (or we can land it in a separate PR if that helps for backporting). |
.current_dir(plain_src_dir) | ||
.run(builder); | ||
// Mitigate pressure on small-capacity disks. | ||
fs::remove_dir_all(plain_src_dir).unwrap(); |
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.
Should these be builder.remove_dir(..)
?
But anyway, somehow this unwrap failed NotFound
when I tried it on beta:
#147571 (comment)
I'm not sure how that can be when it was the current_dir
of the commands just before!
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.
I found there is a dry run before.
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.
Oh, I was close to the answer with my suggested change, and didn't even realize it! 😅
@bors try jobs=x86_64-gnu-distcheck |
This comment has been minimized.
This comment has been minimized.
Update LLVM to 21.1.3 try-job: x86_64-gnu-distcheck
This comment has been minimized.
This comment has been minimized.
💔 Test for b118d5e failed: CI. Failed jobs:
|
@bors try jobs=x86_64-gnu-distcheck |
Update LLVM to 21.1.3 try-job: x86_64-gnu-distcheck
This comment has been minimized.
This comment has been minimized.
@bors r=cuviper,Kobzol rollup=never |
[beta] backports - Don't normalize higher-ranked assumptions if they're not used #147299 - Fix target list of `link_section` #147418 - bootstrap: add `Builder::rustc_cmd` that includes the lib path #147419 - Update LLVM to 21.1.2 #146953 - Remove the temporary directory when a check ends -- part of #147518 r? cuviper
☀️ 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 36e4f5d (parent) -> 3545698 (this PR) Test differencesShow 11 test diffsStage 2
Additionally, 4 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 35456985fab3419baf4bf111b8bbbd5e8565615d --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 (3545698): 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 4.8%)A less reliable metric. May be of interest, but not used to determine the overall result above.
CyclesResults (secondary -2.4%)A less reliable metric. May be of interest, but not used to determine the overall result above.
Binary sizeResults (secondary -0.0%)A less reliable metric. May be of interest, but not used to determine the overall result above.
Bootstrap: 473.739s -> 474.753s (0.21%) |
Fixes #146742.
After #146124, we need more space to run x86_64-gnu-distcheck if building LLVM from source. According to the building log, the space freed by
free-disk-space-linux.sh
is not entirely available.