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

Jan Patrick Lehr via llvm-commits llvm-commits at lists.llvm.org
Tue Jan 14 06:40:56 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:                                             \
----------------
jplehr wrote:

I think this is alright, but I still gonna ask: Further up `all` was replaced with `device` what's different here?
Should the change further up be from `device` to `system` maybe?

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


More information about the llvm-commits mailing list