[Openmp-commits] [PATCH] D62393: [OPENMP][NVPTX]Mark parallel level counter as volatile.

[Olivier Giroux via Phabricator via Openmp-commits]

__simt__ added a comment.

In D62393#1542042 <https://reviews.llvm.org/D62393#1542042>, @tra wrote:

> C++ volatile will give you that. You also rely on atomicity. C++ volatile does not guarantee that, even if NVPTX does happen to. It's a mere coincidence. What if next PTX revision provides a better way to implement C++ volatile without providing atomicity? Then your code will all of a sudden break in new and exciting ways.


I think it's completely healthy and fair to be skeptical that we'll do the right thing by you, but FWIW the changes I want to make to CUDA `volatile` after we ship `std::atomic<T>` go in the other direction. And the legacy requirements are pretty entrenched here, too.


Repository:
  rOMP OpenMP

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

https://reviews.llvm.org/D62393