-
Notifications
You must be signed in to change notification settings - Fork 13.3k
Fix definition of Box in ssa-analysis-regression-50041.rs #94915
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
Conversation
(rust-highfive has picked a reviewer for you, use r? to override) |
This comment has been minimized.
This comment has been minimized.
The Box in liballoc always has a field for the allocator. It is quite hard to support both the old and new definition of Box in cg_clif so this change uses the new definition in this test too.
5891180
to
a303273
Compare
@bors r+ rollup |
📌 Commit a303273 has been approved by |
⌛ Testing commit a303273 with merge ab0615543b70d236e827d4e62f11221d1b82b86c... |
💔 Test failed - checks-actions |
The job Click to see the possible cause of the failure (guessed by this bot)
|
@bors retry spurious clang build error |
☀️ Test successful - checks-actions |
Finished benchmarking commit (0ac4658): comparison url. Summary: This benchmark run did not return any relevant results. 7 results were found to be statistically significant but too small to be relevant. If you disagree with this performance assessment, please file an issue in rust-lang/rustc-perf. @rustbot label: -perf-regression |
The Box in liballoc always has a field for the allocator. It is quite
hard to support both the old and new definition of Box in cg_clif so
this change uses the new definition in this test too.