-
Notifications
You must be signed in to change notification settings - Fork 13.3k
Use hash_stable for hashing str #92130
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
r? @cjgillot (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 c695b60 with merge f584adf005185386e6ac960083504fcadff3daea... |
☀️ Try build successful - checks-actions |
Queued f584adf005185386e6ac960083504fcadff3daea with parent 23f6923, future comparison URL. |
Finished benchmarking commit (f584adf005185386e6ac960083504fcadff3daea): comparison url. Summary: This change led to small relevant improvements 🎉 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. @bors rollup=never |
Well, this actually turned out better than I expected. |
@bors r+ |
📌 Commit c695b60 has been approved by |
⌛ Testing commit c695b60 with merge 90a6203bfbc3908c0acd5c02826af30c481d3145... |
The job Click to see the possible cause of the failure (guessed by this bot)
|
💔 Test failed - checks-actions |
@bors retry |
⌛ Testing commit c695b60 with merge 56b16ae710e80c2a85f70ce4b68d00c3c7ab2bae... |
The job Click to see the possible cause of the failure (guessed by this bot)
|
💔 Test failed - checks-actions |
@bors retry network error |
⌛ Testing commit c695b60 with merge 4482974705e42f5dd494fa7cae7cca455d8c8a49... |
💥 Test timed out |
@bors retry |
☀️ Test successful - checks-actions |
Finished benchmarking commit (41ce641): comparison url. Summary: This benchmark run did not return any relevant changes. If you disagree with this performance assessment, please file an issue in rust-lang/rustc-perf. @rustbot label: -perf-regression |
This seemed like an oversight. With this change the hash can go through the
HashStable
machinery directly.