[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
Mon Jul 29 08:41:13 PDT 2019


gtbercea added a comment.

In D65341#1604562 <https://reviews.llvm.org/D65341#1604562>, @kkwli0 wrote:

> In D65341#1604561 <https://reviews.llvm.org/D65341#1604561>, @gtbercea wrote:
>
> > In D65341#1604440 <https://reviews.llvm.org/D65341#1604440>, @ABataev wrote:
> >
> > > In D65341#1604411 <https://reviews.llvm.org/D65341#1604411>, @Hahnfeld wrote:
> > >
> > > > There's already D55892 <https://reviews.llvm.org/D55892> with a better set of tests, including `target enter data` / `target exit data`.
> > >
> > >
> > > Better to merge those two patches into one.
> >
> >
> > How would you like me to proceed? It looks like the other patch has been sitting there approved for many many months.
>
>
> Yes, the patch has been waiting for the runtime implementation first and then commit.


The runtime implementation is here: D65340 <https://reviews.llvm.org/D65340>


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