[PATCH] D149815: [OpenMP][Flang][Semantics] Add semantics support for USE_DEVICE_ADDR clause on OMP TARGET DATA directive.

[Raghu via Phabricator via llvm-commits]

raghavendhra marked an inline comment as done.
raghavendhra added inline comments.


================
Comment at: flang/lib/Semantics/resolve-directives.cpp:520
+      Symbol::Flag::OmpCopyIn, Symbol::Flag::OmpUseDevicePtr,
+      Symbol::Flag::OmpUseDeviceAddr};
 
----------------
clementval wrote:
> What is the plan here? I mean do you plan to use these flags in the lowering somehow or do you just replicate what was done for other clauses?
I added UseDeviceAddr flag because of the following description from OpenMP 5.1 spec

"If a list item that appears in a use_device_addr clause has corresponding storage in the device data environment, references to the list item in the associated structured block are converted into references to the corresponding list item"

Plan is to have UseDeviceAddr flag set so that we can use it in lowering. Also, replicating the behavior of private clause.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D149815