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

[Qiu Chaofan via Phabricator via cfe-commits]

qiucf added inline comments.


================
Comment at: clang/lib/Basic/Targets/PPC.cpp:336
                         .Default(false);
+  Features["quadword-atomics"] = llvm::StringSwitch<bool>(CPU)
+                                     .Case("pwr10", true)
----------------
What about `ppc64`?

Also, seems there's no need to add `pwr10` here.


================
Comment at: clang/lib/Basic/Targets/PPC.h:442
+  void setMaxAtomicWidth() override {
+    if (getTriple().isOSAIX() && getTriple().isArch64Bit() &&
+        hasFeature("quadword-atomics"))
----------------
We don't need this at all for Linux?


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