]> git.lizzy.rs Git - rust.git/commit - src/tools/rust-analyzer
Auto merge of #92932 - ouz-a:master, r=oli-obk
authorbors <bors@rust-lang.org>
Thu, 3 Feb 2022 12:46:02 +0000 (12:46 +0000)
committerbors <bors@rust-lang.org>
Thu, 3 Feb 2022 12:46:02 +0000 (12:46 +0000)
commit8b7853fe1f87a40ceaddf63aa404817bbfa69676
tree50f563e57cb1672245225b9113a139e8a8ea3ab3
parent796bf14f2e129283d9daee7f05d14c2dfa76d643
parentfd5be23a96274cb1d1f83acdd8f7370ee15e00fa
Auto merge of #92932 - ouz-a:master, r=oli-obk

Temporary fix for the layout of aligned enums

Fix for the issue #92464

~~I was after this issue for quite some time now, I have a temporary fix for it.
I think the current problem is [here](https://github.com/ouz-a/rust/blob/e75f96763f99d56d03ada939fe05cbeb2254888d/compiler/rustc_middle/src/ty/layout.rs#L1305-L1310) created `tag` value might be wrong, because when I checked `min` and `max` values it's always between 0..1, which results in wrong size comparison in a few lines down below.
I think `min` and `max` values don't take `#[repr(aligned(8))]` into consideration and just act from base values assigned inside the enum. If what I am saying is true, aligned enums were created with the wrong layout for some time.~~

~~As stated in the title this is only a temporary fix and I think this needs further investigation, if someone wants to mentor it I would like to work on that too.~~ ðŸ˜¸

**Edit: Weird some tests fail now going to close this for now...**

**Edit2: I made it work again.**

I think I figured out the main problem of the issue, layout types of aligned enums with custom discriminant types were not handled, which resulted in confusing(such as this issue) behavior down the line, this is a kinda hacky fix for the issue.
compiler/rustc_middle/src/ty/layout.rs