-
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
rustc: Move features
from Session
to GlobalCtxt
#114723
Conversation
Removes two pieces of mutable state. Follow up to rust-lang#114622.
r? @davidtwco (rustbot has picked a reviewer for you, use r? to override) |
These commits modify the If this was unintentional then you should revert the changes before this PR is merged. |
@bors r+ |
⌛ Testing commit 7353c96 with merge d2c5db06389ecc1784dae0b622b0862b2f7a5e29... |
☀️ Test successful - checks-actions |
👀 Test was successful, but fast-forwarding failed: 422 Update is not a fast forward |
Finished benchmarking commit (d2c5db06389ecc1784dae0b622b0862b2f7a5e29): comparison URL. Overall result: ✅ improvements - no action needed@rustbot label: -perf-regression Instruction countThis is a highly reliable metric that was used to determine the overall result at the top of this comment.
Max RSS (memory usage)ResultsThis is a less reliable metric that may be of interest but was not used to determine the overall result at the top of this comment.
CyclesThis benchmark run did not return any relevant results for this metric. Binary sizeThis benchmark run did not return any relevant results for this metric. Bootstrap: 632.358s -> 633.497s (0.18%) |
@bors retry : Test was successful, but fast-forwarding failed: 422 Update is not a fast forward |
☀️ Test successful - checks-actions |
Finished benchmarking commit (5f3abbc): comparison URL. Overall result: ✅ improvements - no action needed@rustbot label: -perf-regression Instruction countThis is a highly reliable metric that was used to determine the overall result at the top of this comment.
Max RSS (memory usage)ResultsThis is a less reliable metric that may be of interest but was not used to determine the overall result at the top of this comment.
CyclesThis benchmark run did not return any relevant results for this metric. Binary sizeThis benchmark run did not return any relevant results for this metric. Bootstrap: 634.852s -> 634.77s (-0.01%) |
This was removed in rust-lang#114723
…er-errors docs(dev): Update the feature-gate instructions `features_untracked` was removed in rust-lang#114723 features are now functions as of rust-lang#132027
…er-errors docs(dev): Update the feature-gate instructions `features_untracked` was removed in rust-lang#114723 features are now functions as of rust-lang#132027
…er-errors docs(dev): Update the feature-gate instructions `features_untracked` was removed in rust-lang#114723 features are now functions as of rust-lang#132027
Rollup merge of rust-lang#137227 - epage:features_untracked, r=compiler-errors docs(dev): Update the feature-gate instructions `features_untracked` was removed in rust-lang#114723 features are now functions as of rust-lang#132027
Removes one more piece of mutable state.
Follow up to #114622.
The rule I used for passing feature in function signatures:
rustc_middle
, thenSession
is replaced withTyCtxt
sess: &Session, features: &Features
The code in
rustc_lint
is ultimately used for implementing a trait fromrustc_expand
, so it also doesn't use tcx despite the dependency onrustc_middle
.