[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