[PATCH] D99350: [OPENMP]Fix PR49649: The introduction of $ref globals is not always valid.

Johannes Doerfert via Phabricator via cfe-commits cfe-commits at lists.llvm.org
Tue Apr 27 21:54:27 PDT 2021


jdoerfert added a comment.

The cast was never the problem but the fact that the `$ref` lives in the global address space while the global value might be in the shared one.
D101030 <https://reviews.llvm.org/D101030> gives a way out here by not creating `$ref` variables if there is no host version of a global. This makes sense because we don't need to copy to the global ever.
So, users can have static globals in shared memory by ensuring they are not host accessible. We probably should emit an error if they try to do global in shared memory
that has a host version.

Long story short, this cast won't help.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D99350



More information about the cfe-commits mailing list