[PATCH] D28990: Align i128 to 16 bytes

[Simonas Kazlauskas via Phabricator via llvm-commits]

nagisa added a comment.

> At least for some x86 targets, this will unncessarily penalize the stack. Don't know all the other 32bit architectures, but many of those don't have a stack alignment larger than 32bit either.

I’m willing to adjust the PR to only touch 64-bit targets. For the 64-bit targets 128-bit alignment is necessary for correct FFI with Clang-generated code, which generates code with assumption that i128 is 128-aligned always.


https://reviews.llvm.org/D28990