-
Notifications
You must be signed in to change notification settings - Fork 12.9k
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
Rollup of 4 pull requests #113022
Rollup of 4 pull requests #113022
Conversation
Signed-off-by: Yuki Okushi <jtitor@2k36.org>
`/*can_access_statics:*/ false` is one of the ways to do this, but not the one I like.
…ark-Simulacrum display PID of process holding lock Displays PID of process holding lock when trying to run multiple instances of x.py
Add a regression test for rust-lang#96699 Closes rust-lang#96699 r? `@BoxyUwU`
…i-obk Add enum for `can_access_statics` boolean `/*can_access_statics:*/ false` is one of the ways to do this, but not the one I like. r? oli-obk
Fix test for rust-lang#96258 rust-lang#98644 did not properly test enabling the problematic lint as a warning due to improper use of `compile-flags:` (missing `:`). This makes it use `#![warn]` instead, like in the reproducer. cc rust-lang#96258
@bors r+ rollup=never p=4 |
☀️ Test successful - checks-actions |
📌 Perf builds for each rolled up PR: previous master: 3c5d71a99d In the case of a perf regression, run the following command for each PR you suspect might be the cause: |
Finished benchmarking commit (0d03812): comparison URL. Overall result: ❌ regressions - ACTION NEEDEDNext Steps: If you can justify the regressions found in this perf run, please indicate this with @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)ResultsThis 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.
CyclesResultsThis 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: 663.05s -> 660.819s (-0.34%) |
@rust-timer build 051db0382d4a2e9a87975627659679c5055fd9eb |
This comment has been minimized.
This comment has been minimized.
Finished benchmarking commit (051db0382d4a2e9a87975627659679c5055fd9eb): comparison URL. Overall result: ❌ regressions - no action neededInstruction countThis is a highly reliable metric that was used to determine the overall result at the top of this comment.
Max RSS (memory usage)ResultsThis 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.
CyclesResultsThis 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: 663.05s -> 662.622s (-0.06%) |
Hmm, that was part of the cycle wins, but not all. In any case, it was the only PR that changed the compiler, and cycles had nice improvements. It all smells like noise. |
Successful merges:
can_access_statics
boolean #113011 (Add enum forcan_access_statics
boolean)r? @ghost
@rustbot modify labels: rollup
Create a similar rollup