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

Eli Friedman via Phabricator via cfe-commits cfe-commits at lists.llvm.org
Thu Jul 13 10:43:39 PDT 2023


efriedma added a comment.

> @efriedma would you consider the changes suggested by @hvdijk sufficient under any circumstances or would you still insist on fully compatible AutoUpgrade, given the above discussion?

If the requirement is "we can mix old and new IR", we have to do it correctly, to the extent old versions of clang do it correctly.

If we're willing to refuse to compile old IR and/or refuse to LTO together old and new IR, there are other possible solutions.  I'm not sure what workflows depend on having working autoupgrade.


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

https://reviews.llvm.org/D86310



More information about the cfe-commits mailing list