-
Notifications
You must be signed in to change notification settings - Fork 13.1k
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 1deeee3 #124044
Update LLVM to 1deeee3 #124044
Conversation
|
@bors r+ rollup=never |
bors rollup=neeeev3r |
Oh no, version 18.1.4 has already been released just now. The new changes don't seem important: llvm/llvm-project@1deeee3...e6c3289. Hmm, it includes another MIPS commit. I think it's easier to consider a backport while keeping the current state. |
☀️ Test successful - checks-actions |
Finished benchmarking commit (9f432d7): comparison URL. Overall result: ✅ improvements - no action needed@rustbot label: -perf-regression Instruction countThis is a highly reliable metric that was used to determine the overall result at the top of this comment.
Max RSS (memory usage)This benchmark run did not return any relevant results for this metric. CyclesThis benchmark run did not return any relevant results for this metric. Binary sizeThis benchmark run did not return any relevant results for this metric. Bootstrap: 677.135s -> 676.169s (-0.14%) |
Beta backport declined as per compiler team on Zulip. We're not sure this backport aligns with our backport policies (which btw are a bit unclear) and because maybe it doesn't feel that urgent (besides fixing a regression we introduced in the latest LLVM upgrade on a tier 3 platform). The overlap with LLVM upgrade to 1.8.1.4. add also a bit of uncertainity. @rustbot label -beta-nominated |
Fixes #123772.
Diff: rust-lang/llvm-project@af8f9eb...5399a24
Diff (From 18.1.2): rust-lang/llvm-project@0af6c73...5399a24
r? @cuviper