[PATCH] D28990: Align i128 to 16 bytes

[Joerg Sonnenberger via Phabricator via llvm-commits]

joerg added a comment.

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.


https://reviews.llvm.org/D28990