[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

Johannes Doerfert via Phabricator via Openmp-commits openmp-commits at lists.llvm.org
Thu Jan 7 12:51:36 PST 2021


jdoerfert added a comment.

One nit, works for me otherwise. Anyone else?



================
Comment at: openmp/libomptarget/src/MemoryManager.cpp:101
+      deleteOnDevice(Itr->second.Ptr);
+    }
 }
----------------
Early exit or braces please, former preferred.


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