bjoernager/rust
bjoernager/rust
1
Fork 0
Code Activity

Properly look for uninhabitedness of variants in niche-filling check

Browse source
This commit is contained in:
Anthony Ramine 2018-04-02 00:43:43 +02:00
parent 804d8c88d9
commit 5edfb53439
2 changed files with 10 additions and 3 deletions
Download patch file Download diff file Expand all files Collapse all files

6
src/librustc/ty/layout.rs
View file

@ -1471,10 +1471,10 @@ impl<'a, 'tcx> LayoutCx<'tcx, TyCtxt<'a, 'tcx, 'tcx>> {
// Find one non-ZST variant.
'variants: for (v, fields) in variants.iter().enumerate() {
if fields.iter().any(|f| f.abi == Abi::Uninhabited) {
continue 'variants;
}
for f in fields {
if f.abi == Abi::Uninhabited {
continue 'variants;
}
if !f.is_zst() {
if dataful_variant.is_none() {
dataful_variant = Some(v);

7
src/test/run-pass/type-sizes.rs
View file

@ -42,6 +42,12 @@ enum ReorderedEnum {
B(u8, u16, u8),
}
enum NicheFilledEnumWithInhabitedVariant {
A(&'static ()),
B(&'static (), !),
C,
}
pub fn main() {
assert_eq!(size_of::<u8>(), 1 as usize);
assert_eq!(size_of::<u32>(), 4 as usize);
@ -67,4 +73,5 @@ pub fn main() {
assert_eq!(size_of::<e3>(), 4 as usize);
assert_eq!(size_of::<ReorderedStruct>(), 4);
assert_eq!(size_of::<ReorderedEnum>(), 6);
assert_eq!(size_of::<NicheFilledEnumWithInhabitedVariant>(), size_of::<&'static ()>());
}