diff --git a/src/librustc/ty/layout.rs b/src/librustc/ty/layout.rs index 3212c6f07d1d6..4b8b39c1f590b 100644 --- a/src/librustc/ty/layout.rs +++ b/src/librustc/ty/layout.rs @@ -285,11 +285,13 @@ impl Size { } /// Alignment of a type in bytes, both ABI-mandated and preferred. -/// Since alignments are always powers of 2, we can pack both in one byte, -/// giving each a nibble (4 bits) for a maximum alignment of 215 = 32768. +/// Each field is a power of two, giving the alignment a maximum +/// value of 2^(2^8 - 1), which is limited by LLVM to a i32, with +/// a maximum capacity of 2^31 - 1 or 2147483647. #[derive(Copy, Clone, PartialEq, Eq, Hash, Debug)] pub struct Align { - raw: u8 + abi: u8, + pref: u8, } impl Align { @@ -298,7 +300,7 @@ impl Align { } pub fn from_bytes(abi: u64, pref: u64) -> Result { - let pack = |align: u64| { + let log2 = |align: u64| { // Treat an alignment of 0 bytes like 1-byte alignment. if align == 0 { return Ok(0); @@ -312,7 +314,7 @@ impl Align { } if bytes != 1 { Err(format!("`{}` is not a power of 2", align)) - } else if pow > 0x0f { + } else if pow > 30 { Err(format!("`{}` is too large", align)) } else { Ok(pow) @@ -320,31 +322,30 @@ impl Align { }; Ok(Align { - raw: pack(abi)? | (pack(pref)? << 4) + abi: log2(abi)?, + pref: log2(pref)?, }) } pub fn abi(self) -> u64 { - 1 << (self.raw & 0xf) + 1 << self.abi } pub fn pref(self) -> u64 { - 1 << (self.raw >> 4) + 1 << self.pref } pub fn min(self, other: Align) -> Align { - let abi = cmp::min(self.raw & 0x0f, other.raw & 0x0f); - let pref = cmp::min(self.raw & 0xf0, other.raw & 0xf0); Align { - raw: abi | pref + abi: cmp::min(self.abi, other.abi), + pref: cmp::min(self.pref, other.pref), } } pub fn max(self, other: Align) -> Align { - let abi = cmp::max(self.raw & 0x0f, other.raw & 0x0f); - let pref = cmp::max(self.raw & 0xf0, other.raw & 0xf0); Align { - raw: abi | pref + abi: cmp::max(self.abi, other.abi), + pref: cmp::max(self.pref, other.pref), } } } diff --git a/src/librustc/ty/mod.rs b/src/librustc/ty/mod.rs index 88aef53ec9de5..0ce91b33c510d 100644 --- a/src/librustc/ty/mod.rs +++ b/src/librustc/ty/mod.rs @@ -1398,7 +1398,7 @@ impl_stable_hash_for!(struct ReprFlags { #[derive(Copy, Clone, Eq, PartialEq, RustcEncodable, RustcDecodable, Default)] pub struct ReprOptions { pub int: Option, - pub align: u16, + pub align: u32, pub flags: ReprFlags, } diff --git a/src/libsyntax/attr.rs b/src/libsyntax/attr.rs index f0fc849c0c596..a247fe7f8a56a 100644 --- a/src/libsyntax/attr.rs +++ b/src/libsyntax/attr.rs @@ -974,11 +974,11 @@ pub fn find_repr_attrs(diagnostic: &Handler, attr: &Attribute) -> Vec let mut align_error = None; if let ast::LitKind::Int(align, ast::LitIntType::Unsuffixed) = value.node { if align.is_power_of_two() { - // rustc::ty::layout::Align restricts align to <= 32768 - if align <= 32768 { - acc.push(ReprAlign(align as u16)); + // rustc::ty::layout::Align restricts align to <= 2147483647 + if align <= 2147483647 { + acc.push(ReprAlign(align as u32)); } else { - align_error = Some("larger than 32768"); + align_error = Some("larger than 2147483647"); } } else { align_error = Some("not a power of two"); @@ -1027,7 +1027,7 @@ pub enum ReprAttr { ReprExtern, ReprPacked, ReprSimd, - ReprAlign(u16), + ReprAlign(u32), } #[derive(Eq, Hash, PartialEq, Debug, RustcEncodable, RustcDecodable, Copy, Clone)] diff --git a/src/test/compile-fail/repr-align.rs b/src/test/compile-fail/repr-align.rs index eb0b27fe9c07e..bc9cf065e5a0a 100644 --- a/src/test/compile-fail/repr-align.rs +++ b/src/test/compile-fail/repr-align.rs @@ -17,7 +17,7 @@ struct A(i32); #[repr(align(15))] //~ ERROR: invalid `repr(align)` attribute: not a power of two struct B(i32); -#[repr(align(65536))] //~ ERROR: invalid `repr(align)` attribute: larger than 32768 +#[repr(align(4294967296))] //~ ERROR: invalid `repr(align)` attribute: larger than 2147483647 struct C(i32); fn main() {} diff --git a/src/test/run-pass/align-struct.rs b/src/test/run-pass/align-struct.rs index 0b9a3594502b0..59d0539916730 100644 --- a/src/test/run-pass/align-struct.rs +++ b/src/test/run-pass/align-struct.rs @@ -9,6 +9,7 @@ // except according to those terms. #![feature(attr_literals)] #![feature(repr_align)] +#![feature(box_syntax)] use std::mem; @@ -60,6 +61,13 @@ struct AlignContainsPacked { b: Packed, } +// The align limit was originally smaller (2^15). +// Check that it works with big numbers. +#[repr(align(0x10000))] +struct AlignLarge { + stuff: [u8; 0x10000], +} + impl Align16 { // return aligned type pub fn new(i: i32) -> Align16 { @@ -193,4 +201,15 @@ pub fn main() { assert_eq!(mem::align_of_val(&a.b), 1); assert_eq!(mem::size_of_val(&a), 16); assert!(is_aligned_to(&a, 16)); + + let mut large = box AlignLarge { + stuff: [0; 0x10000], + }; + large.stuff[0] = 132; + *large.stuff.last_mut().unwrap() = 102; + assert_eq!(large.stuff[0], 132); + assert_eq!(large.stuff.last(), Some(&102)); + assert_eq!(mem::align_of::(), 0x10000); + assert_eq!(mem::align_of_val(&*large), 0x10000); + assert!(is_aligned_to(&*large, 0x10000)); }