[PATCH] D28990: Align i128 to 16 bytes

[David Majnemer via Phabricator via llvm-commits]

majnemer added a comment.

In https://reviews.llvm.org/D28990#652740, @joerg wrote:

> What is the motivation behind requiring 128bit alignment? E.g. normal lowering of __uint128_t arithmetic on x86_64 will use uint64_t arithmetic and naturally require the same alignment.
>
> I'm concerned about 32bit platforms because I really would like to see the artifical restriction of __uint128_t to 64bit targets be removed at some point, so that code can naturally depend on them everywhere.


The x86_64 psABI specifies an alignment of 16 bytes for __int128_t and __uint128_t.


https://reviews.llvm.org/D28990