[PATCH] D86119: [OPENMP50]Allow overlapping mapping in target constrcuts.

Alexey Bataev via Phabricator via cfe-commits cfe-commits at lists.llvm.org
Wed Nov 18 07:13:24 PST 2020


ABataev added a comment.

In D86119#2402661 <https://reviews.llvm.org/D86119#2402661>, @ye-luo wrote:

> 1. Could you separate the reordering related changes to separate patch?

It is impossible. There is a separate patch for removing extra TGT_TARGET_PARAM but it cannot be separated from this patch.

> 2. Could you mention which line in spec 4.5 was the restriction? Even 5.0/5.1 has some restrictions. Need to be clear which one you refer to.

Yes, you right, I'm trying to support something like `map(alloc: S) map(tofrom: S.f)` or `map(S, S.ptr[0])` which was not previously allowed and is a feature of OpenMP 5.0.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D86119



More information about the cfe-commits mailing list