[PATCH] D86310: [X86] Align i128 to 16 bytes in x86 datalayouts

[Harald van Dijk via Phabricator via cfe-commits]

hvdijk added a comment.

In D86310#4596712 <https://reviews.llvm.org/D86310#4596712>, @tmgross wrote:

> Is clang still doing something wrong? From my testing, it seems like clang and GCC now agree with each other, I am not sure what would still be incorrect

My understanding is that the code clang generates for `__int128` will still allow it to be passed half-in-register, half-in-memory, exactly what D158169 <https://reviews.llvm.org/D158169> sets out to fix, because D158169 <https://reviews.llvm.org/D158169> only fixes it for LLVM's `i128` which clang bypasses.


Repository:
  rG LLVM Github Monorepo

CHANGES SINCE LAST ACTION
  https://reviews.llvm.org/D86310/new/

https://reviews.llvm.org/D86310