lkail added inline comments.
================
Comment at: clang/lib/Basic/Targets/PPC.cpp:336
.Default(false);
+ Features["quadword-atomics"] = llvm::StringSwitch<bool>(CPU)
+ .Case("pwr10", true)
----------------
qiucf wrote:
> What about `ppc64`?
>
> Also, seems there's no need to add `pwr10` here.
> What about ppc64?
Instructions needed for inline quadword atomics like `lqarx` and `stqcx` are
only user-space viable in pwr8 and above. However, pwr6 and pwr7 also feature
ppc64.
Repository:
rG LLVM Github Monorepo
CHANGES SINCE LAST ACTION
https://reviews.llvm.org/D103501/new/
https://reviews.llvm.org/D103501
_______________________________________________
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
