[Openmp-commits] [PATCH] D21842: Fix omp_sections_nowait.c test to address Bugzilla Bug 28336

[Hal Finkel via Openmp-commits]

hfinkel accepted this revision.
hfinkel added a comment.
This revision is now accepted and ready to land.

Thanks! LGTM.


================
Comment at: test/worksharing/sections/omp_sections_nowait.c:23
@@ +22,3 @@
+    " and waits.\n", rank);
+  while (release == 0);
+  #pragma omp atomic
----------------
I don't recall how this part of OpenMP's memory model is specified. Do you technically need an omp flush(release) here too to force this thread to synchronize its local memory view with the global one?


Repository:
  rL LLVM

http://reviews.llvm.org/D21842