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

[Johannes Doerfert via Phabricator via Openmp-commits]

jdoerfert added a comment.

> we for sure need to update the name and maybe commit message.

Assuming the name & commit message are adjusted I'm withdrawing my objections.

I was hoping to look into this issue because I'm *very* unhappy to claim "The cuda8 compiler is too optimistic and makes some incorrect optimizations" without actually understanding what the problem is. Unfortunately, we still do not have a proper CI infrastructure to test things and I seem to not find the time to reproduce the issue on my end. (anyone else interested in creating a reproducer for this? @Hahnfeld maybe?)


Repository:
  rOMP OpenMP

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

https://reviews.llvm.org/D62393