[PATCH] D67978: [OpenMP 5.0] Fix user-defined mapper lookup in sema for arrays

Tue Sep 24 14:23:02 PDT 2019

ABataev added a comment.

In D67978#1681333 <https://reviews.llvm.org/D67978#1681333>, @lildmh wrote:

> In D67978#1681319 <https://reviews.llvm.org/D67978#1681319>, @ABataev wrote:
>
> > In D67978#1681318 <https://reviews.llvm.org/D67978#1681318>, @lildmh wrote:
> >
> > > Without this patch, it cannot recognize array with mapper, for instance, `#pragma omp target map(mapper(a),to: arr[0:2])` won't work without this patch.
> >
> >
> > What if we have a mapper for the array type?
>
>
> Without this patch, it won't be able to find the mapper in this case, and Clang assumes no mapper. This patch fixes it, so the compiler can find the mapper in this case.

It means, that you just ignore mappers for the array types, right?

Repository:
  rC Clang

CHANGES SINCE LAST ACTION
  https://reviews.llvm.org/D67978/new/

https://reviews.llvm.org/D67978