-
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
rustdoc: Pretty-print assoc const defaults on-demand #91812
Conversation
Some changes occurred in cc @camelid |
(rust-highfive has picked a reviewer for you, use r? to override) |
This should improve performance, clean up the code, and help pave the way for rust-lang#83035.
@bors try @rust-timer queue |
Awaiting bors try build completion. @rustbot label: +S-waiting-on-perf |
⌛ Trying commit caec282fdf3aa37367933dcdffbb6f7bf785519f with merge 146ad07df7795b01adada901a9d9581d2bfbb6fd... |
@bors rollup=never |
caec282
to
3c79d50
Compare
This comment has been minimized.
This comment has been minimized.
This comment has been minimized.
This comment has been minimized.
This comment has been minimized.
This comment has been minimized.
This comment has been minimized.
This comment has been minimized.
This comment has been minimized.
This comment has been minimized.
This comment has been minimized.
This comment has been minimized.
It can always be re-added later if we decide to display associated const default values.
3c79d50
to
719d7a5
Compare
I ran tests locally, so should be good now. @bors try @rust-timer queue |
Awaiting bors try build completion. @rustbot label: +S-waiting-on-perf |
⌛ Trying commit 719d7a5 with merge 14d26c019a4f5cc06517d5296e7a8a0712175082... |
☀️ Try build successful - checks-actions |
Queued 14d26c019a4f5cc06517d5296e7a8a0712175082 with parent 928783d, future comparison URL. |
Finished benchmarking commit (14d26c019a4f5cc06517d5296e7a8a0712175082): 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. 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 |
Thanks! @bors: r+ |
📌 Commit 719d7a5 has been approved by |
⌛ Testing commit 719d7a5 with merge 4cc3158620cf9447a02ba755394cd7e79eedd2b6... |
💥 Test timed out |
Pretty sure that's spurious, so @bors retry |
☀️ Test successful - checks-actions |
Finished benchmarking commit (16d8a91): 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 |
@rylev rustc-perf is reporting extreme, spurious perf changes. |
Hopefully those shouldn't happen again now that rust-lang/rustc-perf#1123 is merged? |
This should improve performance, clean up the code, and help pave the
way for #83035.