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

[Jon Chesterfield via Phabricator via Openmp-commits]

JonChesterfield added a comment.

Iirc this is used for a server warp to tell workers what to do. So the worker needs to do each load to learn that it needs to do something else or stop execution.

Caching the loaded value (because it is no longer volatile) would be my guess at how this change would break things, so caching it manually would seem to have the same problem.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D106310