-
Notifications
You must be signed in to change notification settings - Fork 13k
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 compiler-builtins to 0.1.141 #135180
Conversation
Some changes occurred in compiler/rustc_codegen_cranelift cc @bjorn3 These commits modify the If this was unintentional then you should revert the changes before this PR is merged. |
r? ghost |
Failed to set assignee to
|
r? @ghost 🙃 |
Failed to set assignee to
|
Sorry for the noise Boxy |
lmao |
bors is such a fussy bot |
@bors r+ rollup=never (libm config is arch-dependent) |
Update compiler-builtins to 0.1.141 0.1.141 syncs changes from `libm`. Most of the `libm` changes are testing- or configuration-related.
This comment has been minimized.
This comment has been minimized.
💔 Test failed - checks-actions |
I don't understand why diagnostics in a UI test would be providing suggestions from compiler_builtins, asking about this on Zulip https://rust-lang.zulipchat.com/#narrow/channel/326414-t-infra.2Fbootstrap/topic/Compiletest.3A.20why.20is.20.60compiler_builtins.60.20in.20scope.3F. |
6ac74e1
to
4ebd153
Compare
HIR ty lowering was modified cc @fmease |
💔 Test failed - checks-actions |
4ebd153
to
23d1ef1
Compare
re-rebased to include a fix @bors try |
Update compiler-builtins to 0.1.141 0.1.141 syncs changes from `libm`. Most of the `libm` changes are testing- or configuration-related. try-job: x86_64-msvc
This comment has been minimized.
This comment has been minimized.
💔 Test failed - checks-actions |
23d1ef1
to
f662149
Compare
@bors try |
Update compiler-builtins to 0.1.141 INCLUDES EXPERIMENT, DO NOT MERGE 0.1.141 syncs changes from `libm`. Most of the `libm` changes are testing- or configuration-related. try-job: x86_64-msvc
☀️ Try build successful - checks-actions |
0.1.141 syncs changes from `libm`. Most of the `libm` changes are testing- or configuration-related.
f662149
to
fcc34b2
Compare
The fix merged, so this should be unblocked. @bors r+ |
☀️ Test successful - checks-actions |
Finished benchmarking commit (dd333ca): 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.3%, secondary 1.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.
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: 762.183s -> 763.632s (0.19%) |
0.1.141 syncs changes from
libm
. Most of thelibm
changes are testing- or configuration-related.