[Openmp-commits] [PATCH] D81054: [OpenMP] Introduce target memory manager

Thu Aug 20 18:51:20 PDT 2020

tianshilei1992 added a comment.

In D81054#2229637 <https://reviews.llvm.org/D81054#2229637>, @JonChesterfield wrote:

> As a heads up, I'm told this breaks amdgpu tests. @ronlieb is looking at the merge from upstream, don't have any more details at this time. The basic idea of wrapping device alloc seems likely to be sound for all targets so I'd guess we've run into a bug in this patch.

Yeah, issuing a bug would be nice because at least I could get a reproducer. ;-) BTW, all data race mentioned by others were guarded by lock actually.

Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D81054