[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#1211423, @RaviNarayanaswamy wrote:

> If the objects which are copied are not compatible between host and target due to use of pointers which have different size in a structure then it is users responsibility.
>  I am more interested in where you allocate memory on device you get back an opaque object of size 64.  The opaque object may be 64bit device pointer  or a high 32bit pointer to device memory and low 32bit offset from that memory or it can be a pointer to a structure which has other information.  The idea having libomptarget is to provide a thin layer between compiler and device specific plugin and we need to support different types of plugin.


Ok, I see. Then ask Georgios to fix this, this problem is not part of this patch.


Repository:
  rOMP OpenMP

https://reviews.llvm.org/D51107