-
-
Notifications
You must be signed in to change notification settings - Fork 14.3k
Merge associated_const_equality feature gate into MGCA
#150640
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
Merge associated_const_equality feature gate into MGCA
#150640
Conversation
|
HIR ty lowering was modified cc @fmease Some changes occurred in src/tools/clippy cc @rust-lang/clippy rust-analyzer is developed in its own repository. If possible, consider making this change to rust-lang/rust-analyzer instead. cc @rust-lang/rust-analyzer This PR changes a file inside |
|
|
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Thanks for working on this!
Could you additionally replace all occurrences of FIXME(associated_const_equality) with FIXME(mgca)? Thanks in advance!
This comment has been minimized.
This comment has been minimized.
Sure. |
This comment has been minimized.
This comment has been minimized.
tests/ui/traits/next-solver/dont-ice-on-assoc-projection.stderr
Outdated
Show resolved
Hide resolved
|
@rustbot author |
|
Reminder, once the PR becomes ready for a review, use |
This comment has been minimized.
This comment has been minimized.
a7752be to
9885ffb
Compare
|
@rustbot ready |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Thanks this all looks good to me. I think this PR will merge conflict with #150650 so we'll wait for that to land before merging this. Once it lands you'll need to rebase this PR
It'd also be good if you could squash all of your commits into one
4db26df to
62b56b7
Compare
|
☔ The latest upstream changes made this pull request unmergeable. Please resolve the merge conflicts. |
|
☔ The latest upstream changes (presumably #150669) made this pull request unmergeable. Please resolve the merge conflicts. |
|
This PR was rebased onto a different main commit. Here's a range-diff highlighting what actually changed. Rebasing is a normal part of keeping PRs up to date, so no action is needed—this note is just to help reviewers. |
This comment has been minimized.
This comment has been minimized.
This removes `associated_const_equality` as a separate feature gate and makes it part of `min_generic_const_args` (mgca). Key changes: - Remove `associated_const_equality` from unstable features, add to removed - Update all test files to use `min_generic_const_args` instead - Preserve the original "associated const equality is incomplete" error message by specially handling `sym::associated_const_equality` spans in `feature_gate.rs` - Rename FIXME(associated_const_equality) to FIXME(mgca)
88b620e to
4421270
Compare
|
@rustbot ready |
|
@bors r+ rollup=never |
|
☀️ Test successful - checks-actions |
What is this?This is an experimental post-merge analysis report that shows differences in test outcomes between the merged PR and its parent PR.Comparing 7c04f5d (parent) -> da476f1 (this PR) Test differencesShow 2 test diffs2 doctest diffs were found. These are ignored, as they are noisy. Test dashboardRun cargo run --manifest-path src/ci/citool/Cargo.toml -- \
test-dashboard da476f1942868cdf94ed88b01ea31170cfe95047 --output-dir test-dashboardAnd then open Job duration changes
How to interpret the job duration changes?Job durations can vary a lot, based on the actual runner instance |
|
Finished benchmarking commit (da476f1): comparison URL. Overall result: no relevant changes - no action needed@rustbot label: -perf-regression Instruction countThis benchmark run did not return any relevant results for this metric. Max RSS (memory usage)Results (secondary 2.9%)A less reliable metric. May be of interest, but not used to determine the overall result above.
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: 475.09s -> 473.133s (-0.41%) |
…-equality, r=BoxyUwU Merge `associated_const_equality` feature gate into MGCA Tracking Issues: rust-lang#132980 rust-lang#92827 Merge `associated_const_equality`(ACE) feature gate into `min_generic_const_args`(MGCA). - Replaces `features().associated_const_equality()` checks with `features().min_generic_const_args()` - Updates the parser to gate associated const equality under `min_generic_const_args` - Moves `associated_const_equality` to the removed features list - Removes the `associated_const_equality` method from the `Features` trait - Updates all affected tests and tools (rust-analyzer, clippy) Closes rust-lang#150617 r? `@BoxyUwU`
Tracking Issues: #132980 #92827
Merge
associated_const_equality(ACE) feature gate intomin_generic_const_args(MGCA).features().associated_const_equality()checks withfeatures().min_generic_const_args()min_generic_const_argsassociated_const_equalityto the removed features listassociated_const_equalitymethod from theFeaturestraitCloses #150617
r? @BoxyUwU