[Openmp-commits] [PATCH] D120089: [OpenMP] Explicitly deinitialize device resources

Shilei Tian via Phabricator via Openmp-commits openmp-commits at lists.llvm.org
Wed Mar 9 06:00:23 PST 2022


tianshilei1992 added inline comments.


================
Comment at: openmp/libomptarget/plugins/cuda/src/rtl.cpp:760
+    // Close module
+    if (CUmodule &M = Modules[DeviceId])
+      checkResult(cuModuleUnload(M), "Error returned from cuModuleUnload\n");
----------------
`Modules` are not indexed by device ID. We need `std::vector<std::vector<CUmodule>>` instead.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D120089



More information about the Openmp-commits mailing list