[Openmp-commits] [PATCH] D104418: [PoC][WIP][OpenMP][Offloading] Fixed data race in libomptarget caused by async data movement

[Joachim Protze via Phabricator via Openmp-commits]

protze.joachim added a comment.

In D104418#2825481 <https://reviews.llvm.org/D104418#2825481>, @tianshilei1992 wrote:

> Correct, but in the body of the task, `BlockA[0]` and `BlockB[0]` are only read. It's valid to mark them as `in` only.

What I mean is the write to the corresponding variable implied by the map. 
Do you agree that the following would be invalid and needs inout for target enter data?

  #pragma omp target enter data nowait map(to: BlockA[:BS * BS]) depend(in: BlockA[0])
  #pragma omp target nowait depend(in: BlockA[0])
  {}


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D104418