[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
Mon Jul 10 10:21:50 PDT 2023
hvdijk added a comment.
In D86310#4485837 <https://reviews.llvm.org/D86310#4485837>, @tmgross wrote:
> What is the current status of this patch? Are the reviewers here OK with this fix in general but just need to see changes to autoupgrade?
> @craig.topper or @hvdijk since you worked on it, are you interested in doing these changes, or is this patch in need of new authors?
The `TT.isArch64Bit()` thing I commented on is something I could change, if desired, but from my perspective, the suggested changes to the upgrade mechanism are an unreasonable amount of work considering the benefit is that it keeps already broken code equally broken, so I am not planning on working on that, sorry.
CHANGES SINCE LAST ACTION
https://reviews.llvm.org/D86310/new/
https://reviews.llvm.org/D86310
More information about the llvm-commits
mailing list