[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