[PATCH] D86310: [X86] Align i128 to 16 bytes in x86-64 datalayout

Harald van Dijk via Phabricator via cfe-commits cfe-commits at lists.llvm.org
Thu Jul 13 11:30:07 PDT 2023


hvdijk added a comment.

In D86310#4498551 <https://reviews.llvm.org/D86310#4498551>, @rnk wrote:

> Given that most non-clang frontends want the bug fix (ABI break), who exactly is asking for this level of IR ABI stability?

You were, I thought, or at least that's how I interpreted your earlier comment. :) If we're now all in agreement that that level of ABI stability is not needed, I can update this patch to address the comment that I had left (it should not be limited to 64-bit, it's needed for all X86). I'll probably be able to find time for this in the weekend.


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

https://reviews.llvm.org/D86310



More information about the cfe-commits mailing list