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

Shilei Tian via Phabricator via Openmp-commits openmp-commits at lists.llvm.org
Mon Mar 7 07:58:46 PST 2022


tianshilei1992 added inline comments.


================
Comment at: openmp/libomptarget/plugins/cuda/src/rtl.cpp:767
+    // deinitialized.
+    if (std::none_of(InitializedFlags.begin(), InitializedFlags.end(),
+                     [](bool IsInitialized) { return IsInitialized; }))
----------------
tianshilei1992 wrote:
> This `none_of` is useless here because `InitializedFlags[DeviceId]` is reset beforehand.
Oh, `none_of`, nvm.


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