-
Notifications
You must be signed in to change notification settings - Fork 13.2k
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 hashbrown to 0.12.0 #92998
Update hashbrown to 0.12.0 #92998
Conversation
(rust-highfive has picked a reviewer for you, use r? to override) |
@bors try @rust-timer queue |
Awaiting bors try build completion. @rustbot label: +S-waiting-on-perf |
⌛ Trying commit 2c3656b9d9d11327f285bec1531540d6d892435f with merge 0d9d73ee6d3cf25e92a024f7586015300a9ba2e4... |
☀️ Try build successful - checks-actions |
Queued 0d9d73ee6d3cf25e92a024f7586015300a9ba2e4 with parent fd20513, future comparison URL. |
Finished benchmarking commit (0d9d73ee6d3cf25e92a024f7586015300a9ba2e4): comparison url. Summary: This change led to very large relevant mixed results 🤷 in compiler performance.
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 led to changes in compiler perf. Next Steps: If you can justify the regressions found in this try perf run, please indicate this with @bors rollup=never |
This seems like an overall win but with a bit of noise since this code is extremely sensitive to inlining. |
Agreed. @bors r+ |
📌 Commit 2c3656b9d9d11327f285bec1531540d6d892435f has been approved by |
⌛ Testing commit 2c3656b9d9d11327f285bec1531540d6d892435f with merge ac5c9425520abf5bfa3abb8f47288fa86abafa50... |
💔 Test failed - checks-actions |
This comment has been minimized.
This comment has been minimized.
@bors r=Mark-Simulacrum |
📌 Commit 361a2f9 has been approved by |
⌛ Testing commit 361a2f9 with merge 90ad6b55f65fdfac80fda6c58f892afe3d8ea7a1... |
This comment has been minimized.
This comment has been minimized.
💔 Test failed - checks-actions |
@bors r=Mark-Simulacrum |
📌 Commit 537439c has been approved by |
⌛ Testing commit 537439c with merge 91321b82ef6f989725ac5e41f5159dfa3075ecc1... |
💥 Test timed out |
One of the apple builders seems very slow, it's 5 hours in and still building. I don't think it's related to this PR since perf showed no negative impact on bootstrap times. @bors retry |
☀️ Test successful - checks-actions |
Finished benchmarking commit (10c4c4a): comparison url. Summary: This change led to very large relevant mixed results 🤷 in compiler performance.
If you disagree with this performance assessment, please file an issue in rust-lang/rustc-perf. Next Steps: If you can justify the regressions found in this perf run, please indicate this with @rustbot label: +perf-regression |
Changelog