[Openmp-commits] [PATCH] D51107: [LIBOMPTARGET] Add support for mapping of lambda captures.

[Alexey Bataev via Phabricator via Openmp-commits]

ABataev added a comment.

In https://reviews.llvm.org/D51107#1223428, @Hahnfeld wrote:

> In https://reviews.llvm.org/D51107#1211075, @ABataev wrote:
>
> > BTW, the main problem here is that there is no mapping between host pointer and device pointer for the firstprivates, just like for the mapped data. We just need to rework the way we allocate the memory for the firstprivate to make `MEMBER_OF` to work.
>
>
> I don't see this path taken in the latest patch. Is there a major drawback of doing this?


I did it using new `tgtArgsPositions`, which handles mappings between the firstprivates and the corresponding target pointers.


Repository:
  rOMP OpenMP

https://reviews.llvm.org/D51107