[Openmp-commits] [PATCH] D104555: [OpenMP][Offloading] Fix data race in data mapping by using two locks

[Shilei Tian via Phabricator via Openmp-commits]

tianshilei1992 added a comment.

In D104555#2874750 <https://reviews.llvm.org/D104555#2874750>, @grokos wrote:

> What about other instances where we have access to the `HostDataToTargetMap` followed be data transfers, e.g. when removing a mapping and copy data back to the host? Shouldn't we apply the same logic there?

I didn't come up with a case that can cause data race with D2H because we have a mandatory synchronization at the end of target region.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D104555