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

Harald van Dijk via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Thu Aug 17 12:30:43 PDT 2023


hvdijk added a comment.

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

> @nikic posted a patch that fixes the register passing at https://reviews.llvm.org/D158169. I think that patch plus this one should resolve all the problems we have

Thanks for the link, that will save a lot of time. I don't think it will resolve all the problems, but that it's a significant additional step in the right direction. We also need to make clang use `i128` for `__int128` in order to actually use this fixed calling convention.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D86310



More information about the llvm-commits mailing list