lildmh added a comment. In D67978#1681388 <https://reviews.llvm.org/D67978#1681388>, @ABataev wrote:
> 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? Yes, it ignored mappers before, and this patch fixes it. Repository: rC Clang CHANGES SINCE LAST ACTION https://reviews.llvm.org/D67978/new/ https://reviews.llvm.org/D67978 _______________________________________________ cfe-commits mailing list cfe-commits@lists.llvm.org https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
