[Openmp-commits] [PATCH] D106310: [Libomptarget] Remove volatile from NVPTX work function

[Johannes Doerfert via Phabricator via Openmp-commits]

jdoerfert added a comment.

This is not needed. If this was needed none of the other variables here would work, I mean, take `execution_param` in line 58. Same thing happens, one thread writes all read it.
We use proper synchronization through barriers which ensures we see shared memory updates. If you think this does not work, please provide an example or at least some reference to
a documentation that would indicate so.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D106310