-
Notifications
You must be signed in to change notification settings - Fork 13.1k
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
Access Session and OnDiskCache in rustc_query_system #79035
Conversation
(rust_highfive has picked a reviewer for you, use r? to override) |
Just because queries are so hot: @bors try @rust-timer queue |
Awaiting bors try build completion |
⌛ Trying commit 10eecf16565f2335d7e8ac6c284cc9e6b4cc1a6a with merge 87252d22753cd3d9ae6d687cf8b3764dc5c2778b... |
☀️ Try build successful - checks-actions |
Queued 87252d22753cd3d9ae6d687cf8b3764dc5c2778b with parent 603ab5b, future comparison URL. |
Finished benchmarking try commit (87252d22753cd3d9ae6d687cf8b3764dc5c2778b): comparison url. Benchmarking this pull request likely means that it is perf-sensitive, so we're automatically marking it as not fit for rolling up. Please note that if the perf results are neutral, you should likely undo the rollup=never given below by specifying Importantly, though, if the results of this run are non-neutral do not roll this PR up -- it will mask other regressions or improvements in the roll up. @bors rollup=never |
Perf looks like mostly noise I think - improvements of up to 1.5% on instructions counts, but also losses up to .7%. |
@cjgillot Triage: What's the current status of this? |
@crlf0710 this is waiting on review (although it needs a rebase at some point). r? @wesleywiser maybe? |
☔ The latest upstream changes (presumably #80177) made this pull request unmergeable. Please resolve the merge conflicts. Note that reviewers usually do not review pull requests until merge conflicts are resolved! Once you resolve the conflicts, you should change the labels applied by bors to indicate that your PR is ready for review. Post this as a comment to change the labels:
|
Reduces the boilerplace of implementing the
DepContext
trait.The OnDiskCache is accessed directly through a dedicated trait.