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
_______________________________________________
cfe-commits mailing list
[email protected]
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits

Reply via email to