craig.topper added a comment.

Is this ok with the backend fixed? Or do you want me factor this into HasCX16 
which I think is only used by the defineMacro and the return for 
hasFeature("cx16")? And I think hasFeature("cx16") is only used by that 
getMaxAtomicWidth() code which is only called on 64 bit.

Or we could maybe ignore "cx16" in setFeatureEnabled on 32 bit targets? But I 
think that would break always_inline on a target attribute with cx16 in 32 bit 
mode which gcc does allow. https://godbolt.org/z/TW985s


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