[clang] [llvm] [AArch64] Implement the atomic store with hint intrinsic (PR #198316)
Kerry McLaughlin via llvm-commits
llvm-commits at lists.llvm.org
Fri Sep 25 03:54:33 PDT 2026
kmclaughlin-arm wrote:
> Our internal builds are seeing failures due to this change when trying to build Clang with Clang
>
> Which makes sense, given that `HINT_STSHH_KEEP` is a macro here:
>
> https://github.com/llvm/llvm-project/blob/cb4c63ef8f72e93b58277bb5443251e70dadb401/clang/lib/Headers/arm_acle.h#L747
>
> And then you also use it as an enum field:
>
> https://github.com/llvm/llvm-project/blob/cb4c63ef8f72e93b58277bb5443251e70dadb401/llvm/include/llvm/Support/AArch64MemoryHints.h#L15
>
> So, if you use Clang to build Clang then the macro and enum fields collide.
Thank you @dpaoliello - I have opened https://github.com/llvm/llvm-project/pull/226432 to address this, which renames the enum values in AArch64MemoryHints.h to avoid the naming collision.
https://github.com/llvm/llvm-project/pull/198316
More information about the llvm-commits
mailing list