[Openmp-commits] [PATCH] D62397: [OPENMP][NVPTX]Relax flush directive.

[Johannes Doerfert via Phabricator via Openmp-commits]

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

Potential improvement inlined but this LGTM. Thanks for fixing the race problem!



================
Comment at: libomptarget/deviceRTLs/nvptx/test/parallel/flush.c:34
+  printf("out=%d.\n", out);
+  return 0;
+}
----------------
Should we `return !(out == 42)`?


Repository:
  rOMP OpenMP

CHANGES SINCE LAST ACTION
  https://reviews.llvm.org/D62397/new/

https://reviews.llvm.org/D62397