jfb added a comment.

In D59287#1427485 <https://reviews.llvm.org/D59287#1427485>, @craig.topper 
wrote:

> Isn’t that setMaxAtomicWidth in the x86-64 derived class?


Right you are!

> As far as preventing “cx16” from being set in 32-bit mode, we’ll need to 
> check the behavior of CPUID in 32-bit mode or -march=native might still end 
> up setting it.

What I want to make sure is that your fix to the macro (which seems correct!) 
doesn't diverge from what the rest LLVM ends up doing with atomics. i.e. the 
macro says "no cmpxchg16b" but somehow codegen does generate it.


Repository:
  rC Clang

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

https://reviews.llvm.org/D59287



_______________________________________________
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits

Reply via email to