[Openmp-commits] [llvm] [openmp] [OpenMP] Publish target-init team state with acq_rel barrier (PR #208701)

[Joseph Huber via Openmp-commits]

================
@@ -112,7 +112,7 @@ int32_t __kmpc_target_init(KernelEnvironmentTy &KernelEnvironment,
   if (IsSPMD) {
     initializeRuntime(/*IsSPMD=*/true, KernelEnvironment,
                       KernelLaunchEnvironment);
-    synchronize::threadsAligned(atomic::relaxed);
+    synchronize::threadsAligned(atomic::acq_rel);
----------------
jhuber6 wrote:

Seems weird that a runtime initialization, which has memory effects, wouldn't be flushed. But the usage of relaxed (i.e. only the thread sync with no memory ordering effects) makes me think it was intentional. Can you confirm that there's not a prior fence that handles the case where the runtime initialization is not manifested by the time a thread tries to use it after the barrier?

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