[llvm] [OpenMP] Make each atomic helper take an atomic scope argument (PR #122786)

Joseph Huber via llvm-commits llvm-commits at lists.llvm.org
Tue Jan 14 07:27:04 PST 2025


================
@@ -64,12 +64,16 @@ uint32_t atomicInc(uint32_t *A, uint32_t V, atomic::OrderingTy Ordering,
 
 #define ScopeSwitch(ORDER)                                                     \
   switch (MemScope) {                                                          \
-  case atomic::MemScopeTy::all:                                                \
+  case atomic::MemScopeTy::system:                                             \
----------------
jhuber6 wrote:

`all` was a weird name, it was hacking around it before it even existed. `system` scope should not be the default since 99% of the time we only care about being coherent on the device.

https://github.com/llvm/llvm-project/pull/122786


More information about the llvm-commits mailing list