-
Notifications
You must be signed in to change notification settings - Fork 13.2k
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
Handle rustc_middle
cases of rustc::potential_query_instability
lint
#131160
base: master
Are you sure you want to change the base?
Handle rustc_middle
cases of rustc::potential_query_instability
lint
#131160
Conversation
Thanks for the pull request, and welcome! The Rust team is excited to review your changes, and you should hear from @compiler-errors (or someone else) some time within the next two weeks. Please see the contribution instructions for more information. Namely, in order to ensure the minimum review times lag, PR authors and assigned reviewers should ensure that the review label (
|
These commits modify the If this was unintentional then you should revert the changes before this PR is merged. Some changes occurred in src/tools/clippy cc @rust-lang/clippy |
This comment has been minimized.
This comment has been minimized.
da9a8fd
to
4c46978
Compare
This comment has been minimized.
This comment has been minimized.
4c46978
to
afeabb2
Compare
e8cc67a
to
0000613
Compare
☔ The latest upstream changes (presumably #131263) made this pull request unmergeable. Please resolve the merge conflicts. |
0000613
to
01e27a3
Compare
☔ The latest upstream changes (presumably #132282) made this pull request unmergeable. Please resolve the merge conflicts. |
I'm gonna re-roll this so this PR gets more attention, sorry for my inactivity. r? compiler But also reassigning to author sicne I think this PR still needs updating. @rustbot author |
@ismailarilik Can you rebase the PR today? I'm in holiday tomorrow so I"d like to have a review before that :) |
r? compiler cause I'm going to the holidays in the next few days. And I suggest to run a rustc perf when it's ready since this PR change the cache struct |
@ismailarilik any updates on this? thanks |
There are merge commits (commits with multiple parents) in your changes. We have a no merge policy so these commits will need to be removed for this pull request to be merged. You can start a rebase with the following commands:
The following commits are merge commits: |
Sorry, I missed the previous comment. I've just fixed conflicts. But since I have no access to the git tool in a machine nowadays, I had to do it using the web UI so there is a merge commit now. I can fix it later or feel free to fix. |
This PR removes
#![allow(rustc::potential_query_instability)]
line fromcompiler/rustc_middle/src/lib.rs
and convertsFxHash{Map,Set}
types intoFxIndex{Map,Set}
to suppress lint errors.A somewhat tracking issue: #84447
r? @compiler-errors