diff options
| author | Eduard-Mihai Burtescu <edy.burt@gmail.com> | 2017-12-18 15:06:34 +0200 |
|---|---|---|
| committer | Eduard-Mihai Burtescu <edy.burt@gmail.com> | 2017-12-18 21:31:52 +0200 |
| commit | 087f1c23a70f889ea157c68b9db36c524e95ba8f (patch) | |
| tree | 85b0abaf45fa2914f386f8c6c3a678ea3f9d4d15 /src/test | |
| parent | a3a7203e2c9ed30a501da86f3fa1f9efe707ac94 (diff) | |
rustc: ensure optimized enums have a properly aligned size.
Diffstat (limited to 'src/test')
| -rw-r--r-- | src/test/run-pass/packed-struct-optimized-enum.rs | 13 |
1 files changed, 10 insertions, 3 deletions
diff --git a/src/test/run-pass/packed-struct-optimized-enum.rs b/src/test/run-pass/packed-struct-optimized-enum.rs index 876b74a042f..b8a1e6f2f54 100644 --- a/src/test/run-pass/packed-struct-optimized-enum.rs +++ b/src/test/run-pass/packed-struct-optimized-enum.rs @@ -16,14 +16,21 @@ impl<T: Copy> Clone for Packed<T> { fn clone(&self) -> Self { *self } } -fn main() { - let one = (Some(Packed((&(), 0))), true); +fn sanity_check_size<T: Copy>(one: T) { let two = [one, one]; let stride = (&two[1] as *const _ as usize) - (&two[0] as *const _ as usize); + assert_eq!(stride, std::mem::size_of_val(&one)); +} +fn main() { // This can fail if rustc and LLVM disagree on the size of a type. // In this case, `Option<Packed<(&(), u32)>>` was erronously not // marked as packed despite needing alignment `1` and containing // its `&()` discriminant, which has alignment larger than `1`. - assert_eq!(stride, std::mem::size_of_val(&one)); + sanity_check_size((Some(Packed((&(), 0))), true)); + + // In #46769, `Option<(Packed<&()>, bool)>` was found to have + // pointer alignment, without actually being aligned in size. + // E.g. on 64-bit platforms, it had alignment `8` but size `9`. + sanity_check_size(Some((Packed(&()), true))); } |
