[PATCH] D158857: [clang][aarch64] Add support for the MS qualifiers __ptr32, __ptr64, __sptr, __uptr for aarch64

Eli Friedman via Phabricator via cfe-commits cfe-commits at lists.llvm.org
Mon Aug 28 15:19:33 PDT 2023


efriedma added a comment.

If you don't have autoupgrade, that means you can't link IR built with older versions of LLVM to IR built with newer versions.  llvm::UpgradeDataLayoutString is designed to fix this incompatibility.  It's probably worth doing here given that it's relatively straightforward.

The issue with the i128 datalayout ugrade isn't the datalayout string itself, it's that the datalayout change affects existing code in a non-trivial way.  That isn't really an issue here: there shouldn't be any existing AArch64 IR using address-space 270, so changing its meaning won't have any effect.


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

https://reviews.llvm.org/D158857



More information about the cfe-commits mailing list