[PATCH] D86310: [X86] Align i128 to 16 bytes in x86-64 datalayout
Harald van Dijk via Phabricator via llvm-commits
llvm-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 llvm-commits
mailing list