[PATCH] D149815: [OpenMP][Flang][Semantics] Add semantics support for USE_DEVICE_ADDR clause on OMP TARGET DATA directive.
Raghu via Phabricator via llvm-commits
llvm-commits at lists.llvm.org
Mon May 8 09:54:38 PDT 2023
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
More information about the llvm-commits
mailing list