[PATCH] D28990: Align i128 to 16 bytes
Simonas Kazlauskas via Phabricator via llvm-commits
llvm-commits at lists.llvm.org
Sun Jan 22 10:07:16 PST 2017
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.
More information about the llvm-commits