[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_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.
I added the test as per Alexey's previous request.
CHANGES SINCE LAST ACTION
More information about the cfe-commits