Deny multiple EII impls on a single item#159326
Conversation
|
Some changes occurred in compiler/rustc_hir/src/attrs cc @jdonszelmann, @JonathanBrouwer Some changes occurred in compiler/rustc_passes/src/check_attr.rs cc @jdonszelmann, @JonathanBrouwer Some changes occurred in compiler/rustc_builtin_macros/src/autodiff.rs cc @ZuseZ4 The parser was modified, potentially altering the grammar of (stable) Rust cc @fmease |
|
|
This comment has been minimized.
This comment has been minimized.
1b8c264 to
c5b8085
Compare
|
cc @rust-lang/clippy |
This comment has been minimized.
This comment has been minimized.
This allows implementing EIIs that don't have a default impl without the involvement of symbol aliases by simply changing the mangled symbol name of the implementation, which would make them trivially compatible with all backends and targets. This change will be left to a followup PR.
1779491 to
57fb26e
Compare
|
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 allows implementing EIIs that don't have a default impl without the involvement of symbol aliases by simply changing the mangled symbol name of the implementation, which would make them trivially compatible with all backends and targets. This change will be left to a followup PR.
r? jdonszelmann