[patch] Correctly align arrays of int64_t on 32 bit systems

Rafael EspĂ­ndola rafael.espindola at gmail.com
Wed Aug 7 20:19:41 PDT 2013


>> To quote Clang, 'alignof' applied to an expression is a GNU extension.
>> Thus we should presumably do what GCC does -- C++ doesn't have a notion of
>> the alignment of a declaration as distinct from the alignment of its type.
>
>
> OK, so that's not strictly-speaking true, but it doesn't have a notion of
> the alignment being increased because all globals happen to be 16-byte
> aligned on the target system.

The attached patch should fix it. Is it OK?

Cheers,
Rafael
-------------- next part --------------
A non-text attachment was scrubbed...
Name: t.patch
Type: application/octet-stream
Size: 3638 bytes
Desc: not available
URL: <http://lists.llvm.org/pipermail/cfe-commits/attachments/20130807/44e27a2c/attachment.obj>


More information about the cfe-commits mailing list