[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_CLOSE);
> 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.
CHANGES SINCE LAST ACTION
More information about the cfe-commits