-
Notifications
You must be signed in to change notification settings - Fork 12.5k
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
linker: Remove the "--whole-archive
in test mode" backcompat hack
#128400
Conversation
r? @Nadrieril rustbot has assigned @Nadrieril. Use |
@bors try |
linker: Remove the "`--whole-archive` in test mode" backcompat hack Fixes rust-lang#116910.
r? compiler |
☀️ Try build successful - checks-actions |
@craterbot run mode=build-and-test +cargoflags=--no-run |
🚨 Error: failed to parse the command 🆘 If you have any trouble with Crater please ping |
@craterbot run mode=build-and-test |
👌 Experiment ℹ️ Crater is a tool to run experiments across parts of the Rust ecosystem. Learn more |
🚧 Experiment ℹ️ Crater is a tool to run experiments across parts of the Rust ecosystem. Learn more |
🎉 Experiment
|
Too many spurious regressions, I'll just do a second iteration. |
👌 Experiment ℹ️ Crater is a tool to run experiments across parts of the Rust ecosystem. Learn more |
🚧 Experiment ℹ️ Crater is a tool to run experiments across parts of the Rust ecosystem. Learn more |
🎉 Experiment
|
Much better. We need to land this in any case, because the change only affects test mode, doesn't affect dependent crates, and the removed hack could potentially hide issues in non-test mode. |
Keeping this out of a rollup may be beneficial for bisectability. @bors r+ rollup=iffy |
linker: Remove the "`--whole-archive` in test mode" backcompat hack Fixes rust-lang#116910.
@bors retry failed to remove executable with "Access is denied" |
☀️ Test successful - checks-actions |
Finished benchmarking commit (04dff01): 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 3.2%, secondary 2.5%)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 -4.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: 758.54s -> 756.781s (-0.23%) |
Fixes #116910.