[Openmp-commits] [PATCH] D84381: [OpenMP] Wait for kernel prior to memory deallocation

[Shilei Tian via Phabricator via Openmp-commits]

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

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

> Does it mean the D2H will always run synchronously after this change?
>  Does it also mean that target_data_end should be split into data transfer and data free parts?


That is good point. There is a critical issue in this patch.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D84381