Skip to content
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

Closed
wants to merge 5 commits into from

Conversation

cjgillot
Copy link
Contributor

Reduces the boilerplace of implementing the DepContext trait.
The OnDiskCache is accessed directly through a dedicated trait.

@rust-highfive
Copy link
Collaborator

r? @ecstatic-morse

(rust_highfive has picked a reviewer for you, use r? to override)

@rust-highfive rust-highfive added the S-waiting-on-review Status: Awaiting review from the assignee but also interested parties. label Nov 13, 2020
@jyn514 jyn514 added T-compiler Relevant to the compiler team, which will review and decide on the PR/issue. A-query-system Area: The rustc query system (https://rustc-dev-guide.rust-lang.org/query.html) C-cleanup Category: PRs that clean code up or issues documenting cleanup. labels Nov 16, 2020
@jyn514
Copy link
Member

jyn514 commented Nov 16, 2020

Just because queries are so hot:

@bors try @rust-timer queue

@rust-timer
Copy link
Collaborator

Awaiting bors try build completion

@bors
Copy link
Contributor

bors commented Nov 16, 2020

⌛ Trying commit 10eecf16565f2335d7e8ac6c284cc9e6b4cc1a6a with merge 87252d22753cd3d9ae6d687cf8b3764dc5c2778b...

@bors
Copy link
Contributor

bors commented Nov 16, 2020

☀️ Try build successful - checks-actions
Build commit: 87252d22753cd3d9ae6d687cf8b3764dc5c2778b (87252d22753cd3d9ae6d687cf8b3764dc5c2778b)

@rust-timer
Copy link
Collaborator

Queued 87252d22753cd3d9ae6d687cf8b3764dc5c2778b with parent 603ab5b, future comparison URL.

@rust-timer
Copy link
Collaborator

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 rollup- to bors.

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
@rustbot modify labels: +S-waiting-on-review -S-waiting-on-perf

@jyn514
Copy link
Member

jyn514 commented Nov 16, 2020

Perf looks like mostly noise I think - improvements of up to 1.5% on instructions counts, but also losses up to .7%.

@crlf0710
Copy link
Member

@cjgillot Triage: What's the current status of this?

@jyn514
Copy link
Member

jyn514 commented Dec 18, 2020

@crlf0710 this is waiting on review (although it needs a rebase at some point).

r? @wesleywiser maybe?

@bors
Copy link
Contributor

bors commented Dec 22, 2020

☔ 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:

@rustbot modify labels: +S-waiting-on-review -S-waiting-on-author

@cjgillot cjgillot closed this Jan 7, 2021
@cjgillot cjgillot deleted the simtrait branch September 21, 2021 16:10
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
A-query-system Area: The rustc query system (https://rustc-dev-guide.rust-lang.org/query.html) C-cleanup Category: PRs that clean code up or issues documenting cleanup. S-waiting-on-review Status: Awaiting review from the assignee but also interested parties. T-compiler Relevant to the compiler team, which will review and decide on the PR/issue.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

8 participants