[PATCH] D65341: [OpenMP] Add support for close map modifier in Clang

Gheorghe-Teodor Bercea via Phabricator via cfe-commits cfe-commits at lists.llvm.org
Thu Aug 1 11:05:38 PDT 2019


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


================
Comment at: lib/CodeGen/CGOpenMPRuntime.cpp:7695
               Flags &= ~(OMP_MAP_TO | OMP_MAP_FROM | OMP_MAP_ALWAYS |
-                         OMP_MAP_DELETE);
+                         OMP_MAP_DELETE | OMP_MAP_CLOSE);
 
----------------
Hahnfeld wrote:
> gtbercea wrote:
> > ABataev wrote:
> > > Why?
> > If the pointee has been mapped as TO/FROM/etc already no need to map it TO/FROM/etc again.
> I'm not sure if there's a test for this. I can't verify right now, so I trust you that it's covered.
I added the test as per Alexey's previous request.


Repository:
  rC Clang

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

https://reviews.llvm.org/D65341





More information about the cfe-commits mailing list