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

Jonas Hahnfeld via Phabricator via cfe-commits cfe-commits at lists.llvm.org
Thu Aug 1 09:38:32 PDT 2019


Hahnfeld removed a reviewer: Hahnfeld.
Hahnfeld added inline comments.
This revision is now accepted and ready to land.


================
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);
 
----------------
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.


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