[PATCH] D103501: [clang][AIX] Enable inlined quadword atomic operations

[Kai Luo via Phabricator via cfe-commits]

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