[Openmp-commits] [PATCH] D94256: [OpenMP] Fixed the issue that memory manager calls plugin interface to release buffered memory even after the plugin has been destroyed
Shilei Tian via Phabricator via Openmp-commits
openmp-commits at lists.llvm.org
Thu Jan 7 12:48:52 PST 2021
tianshilei1992 marked an inline comment as done.
tianshilei1992 added inline comments.
================
Comment at: openmp/libomptarget/src/MemoryManager.cpp:115
+ return OFFLOAD_SUCCESS;
+ }
+
----------------
jdoerfert wrote:
> Can't we do this in the destructor `MemoryManagerTy::~MemoryManagerTy()` ?
Nice point! Did that.
Repository:
rG LLVM Github Monorepo
CHANGES SINCE LAST ACTION
https://reviews.llvm.org/D94256/new/
https://reviews.llvm.org/D94256
More information about the Openmp-commits
mailing list