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

[Gheorghe-Teodor Bercea via Phabricator via cfe-commits]

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