Skip to content

Commit

Permalink
Fix and improve test for enum repr sizes
Browse files Browse the repository at this point in the history
  • Loading branch information
Gankra committed Nov 28, 2017
1 parent 7e79b9b commit 0e63d27
Showing 1 changed file with 49 additions and 3 deletions.
52 changes: 49 additions & 3 deletions src/test/run-pass/multiple-reprs.rs
Original file line number Diff line number Diff line change
Expand Up @@ -9,7 +9,8 @@
// except according to those terms.


use std::mem::size_of;
use std::mem::{size_of, align_of};
use std::os::raw::c_int;

// The two enums that follow are designed so that bugs trigger layout optimization.
// Specifically, if either of the following reprs used here is not detected by the compiler,
Expand All @@ -27,6 +28,38 @@ enum E2 {
B(u8, u16, u8)
}

// Check that repr(int) and repr(C) are in fact different from the above

#[repr(u8)]
enum E3 {
A(u8, u16, u8),
B(u8, u16, u8)
}

#[repr(u16)]
enum E4 {
A(u8, u16, u8),
B(u8, u16, u8)
}

#[repr(u32)]
enum E5 {
A(u8, u16, u8),
B(u8, u16, u8)
}

#[repr(u64)]
enum E6 {
A(u8, u16, u8),
B(u8, u16, u8)
}

#[repr(C)]
enum E7 {
A(u8, u16, u8),
B(u8, u16, u8)
}

// From pr 37429

#[repr(C,packed)]
Expand All @@ -37,7 +70,20 @@ pub struct p0f_api_query {
}

pub fn main() {
assert_eq!(size_of::<E1>(), 6);
assert_eq!(size_of::<E2>(), 6);
assert_eq!(size_of::<E1>(), 8);
assert_eq!(size_of::<E2>(), 8);
assert_eq!(size_of::<E3>(), 6);
assert_eq!(size_of::<E4>(), 8);
assert_eq!(size_of::<E5>(), align_size(10, align_of::<u32>()));
assert_eq!(size_of::<E6>(), align_size(14, align_of::<u64>()));
assert_eq!(size_of::<E7>(), align_size(6 + size_of::<c_int>(), align_of::<c_int>()));
assert_eq!(size_of::<p0f_api_query>(), 21);
}

fn align_size(size: usize, align: usize) -> usize {
if size % align != 0 {
size + (align - (size % align))
} else {
size
}
}

0 comments on commit 0e63d27

Please sign in to comment.