make repr_transparent_non_zst_fields a hard error#155299
make repr_transparent_non_zst_fields a hard error#155299RalfJung wants to merge 1 commit intorust-lang:mainfrom
Conversation
|
rustbot has assigned @jdonszelmann. Use Why was this reviewer chosen?The reviewer was selected based on:
|
|
@bors try |
This comment has been minimized.
This comment has been minimized.
…try> make repr_transparent_non_zst_fields a hard error
This comment has been minimized.
This comment has been minimized.
5b074cf to
3bb6c15
Compare
|
@craterbot check |
|
👌 Experiment ℹ️ Crater is a tool to run experiments across parts of the Rust ecosystem. Learn more |
|
This looks nice Ralf, indeed let's nominate for lang after this completes |
|
☔ The latest upstream changes (presumably #155796) made this pull request unmergeable. Please resolve the merge conflicts. |
This was a deny-by-default FCW since #147185, which landed almost 6 months ago and shipped with Rust 1.93. (If this PR lands now it will ship with 1.96.) Already back then we found hardly any crater impact. The tracking issue has had no new relevant backrefs since that PR landed.
So, I think it is time to make this a hard error. Will nominate for t-lang once the crater run is complete.
Fixes #78586 (tracking issue)
Fixes rust-lang/unsafe-code-guidelines#552 because this means the
repr(transparent)ABI compatibility rule no longer ever "ignores"repr(C)fields.