-
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
Remove box syntax from rustc_mir_dataflow and rustc_mir_transform #97281
Conversation
(rust-highfive has picked a reviewer for you, use r? to override) |
Also, maybe it would be a good idea to do a perf run, but I doubt there is going to be much of an impact. #87781 didn't have any outside of a very minor regression for rustdoc close to the noise limit, that was then reverted. |
This comment has been minimized.
This comment has been minimized.
@bors try @rust-timer queue |
Awaiting bors try build completion. @rustbot label: +S-waiting-on-perf |
⌛ Trying commit 99603ef with merge 454eb7ecb0429d36f9d0367d02449abb1f7b38f9... |
didn't realize this PR was so small, lol. Started that perf after just looking at the description. anyways r=me if that perf run comes back clean. |
☀️ Try build successful - checks-actions |
Queued 454eb7ecb0429d36f9d0367d02449abb1f7b38f9 with parent 4bb4dc4, future comparison URL. |
Finished benchmarking commit (454eb7ecb0429d36f9d0367d02449abb1f7b38f9): comparison url. Instruction countThis benchmark run did not return any relevant results for this metric. Max RSS (memory usage)Results
CyclesResults
If you disagree with this performance assessment, please file an issue in rust-lang/rustc-perf. Benchmarking this pull request likely means that it is perf-sensitive, so we're automatically marking it as not fit for rolling up. While you can manually mark this PR as fit for rollup, we strongly recommend not doing so since this PR may lead to changes in compiler perf. @bors rollup=never Footnotes |
@bors r+ |
📌 Commit 99603ef has been approved by |
☀️ Test successful - checks-actions |
Finished benchmarking commit (b2eed72): comparison url. Instruction countThis benchmark run did not return any relevant results for this metric. Max RSS (memory usage)Results
CyclesResults
If you disagree with this performance assessment, please file an issue in rust-lang/rustc-perf. @rustbot label: -perf-regression Footnotes |
Continuation of #87781, inspired by #97239. The usages that this PR removes have not appeared from nothing, instead the usage in
rustc_mir_dataflow
andrustc_mir_transform
was from #80522 which split uprustc_mir
, and which was filed before I filed #87781, so it was using the state from before my PR. But it was merged after my PR was merged, so thebox_syntax
uses were able to survive here. Outside of this introduction due to the code being outside of the master branch at the point of merging of my PR, there was only one other introduction of box syntax, in #95159. That box syntax was removed again though in #95555. Outside of that,box_syntax
has not made its reoccurrance in compiler crates.