-
Notifications
You must be signed in to change notification settings - Fork 13.3k
Thin AssocItem
#95884
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
Thin AssocItem
#95884
Conversation
This comment has been minimized.
This comment has been minimized.
☔ The latest upstream changes (presumably #95931) made this pull request unmergeable. Please resolve the merge conflicts. |
@bors try @rust-timer queue |
Awaiting bors try build completion. @rustbot label: +S-waiting-on-perf |
⌛ Trying commit b17e94966ca80bf15c89506ec454045e02ea2eba with merge 900520465146285405f968f619001b5f7eb21c7a... |
This comment has been minimized.
This comment has been minimized.
💔 Test failed - checks-actions |
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 09a25c3c8553868c930bb312c2193d88f5601309 with merge 09bd25116aaa08279ee6842808402afe18c82967... |
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 0b36938cbf416c9701250d31ae6e44e00379700d with merge a5ebbfe4684f0bac93cb0b0b3c5a00e3940d3901... |
☀️ Try build successful - checks-actions |
Queued a5ebbfe4684f0bac93cb0b0b3c5a00e3940d3901 with parent c907b6f, future comparison URL. |
Finished benchmarking commit (a5ebbfe4684f0bac93cb0b0b3c5a00e3940d3901): comparison url. Instruction count
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. Next Steps: If you can justify the regressions found in this try perf run, please indicate this with @bors rollup=never Footnotes |
☔ The latest upstream changes (presumably #100024) made this pull request unmergeable. Please resolve the merge conflicts. |
🔒 Merge conflict This pull request and the master branch diverged in a way that cannot be automatically merged. Please rebase on top of the latest master branch, and let the reviewer approve again. How do I rebase?Assuming
You may also read Git Rebasing to Resolve Conflicts by Drew Blessing for a short tutorial. Please avoid the "Resolve conflicts" button on GitHub. It uses Sometimes step 4 will complete without asking for resolution. This is usually due to difference between how Error message
|
@bors r=lcnr |
☀️ Test successful - checks-actions |
Finished benchmarking commit (21de280): comparison url. Instruction count
Max RSS (memory usage)Results
CyclesResults
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 Footnotes |
This looks perf-neutral, with the improvements roughly the same size as the regressions. @rustbot label: +perf-regression-triaged |
get panic in any cargo projects building after this PR, delete building cache of projects and panic is resolved.
|
Signed-off-by: Yuki Okushi <jtitor@2k36.org>
Rustup to rust-lang/rust#95884 Fixes #337 Signed-off-by: Yuki Okushi <jtitor@2k36.org>
This PR removes a few fields from
AssocItem
that should be easily computed using other queries.This simplifies some of the metadata decoding.