[Openmp-commits] [PATCH] D85246: [OpenACC] Fix `omp target update` for array extension

George Rokos via Phabricator via Openmp-commits openmp-commits at lists.llvm.org
Tue Aug 4 15:06:39 PDT 2020


grokos accepted this revision.
grokos added a comment.
This revision is now accepted and ready to land.

I suppose the same must be applied to all cases where `present` is used. Without a modification like this, `present` just confirms that the starting address of the object is already mapped, although a size may be specified in the map clause.

  #pragma omp target map(present: p[0:10]) // present will succeed even if only p[0] is mapped and p[1]-p[9] are not

So the question here is: should `present` apply to the size as well as the begin address? I would say yes.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D85246



More information about the Openmp-commits mailing list