[Openmp-commits] [PATCH] D116315: [OpenMP][CUDA] Add resource pool for CUevent

[Shilei Tian via Phabricator via Openmp-commits]

tianshilei1992 added inline comments.


================
Comment at: openmp/libomptarget/plugins/cuda/src/rtl.cpp:487
         EnvTeamThreadLimit(-1), RequiresFlags(OMP_REQ_UNDEFINED),
-        DynamicMemorySize(0) {
+        DynamicMemorySize(0), EventPool(AllocatorTy<CUevent>(), 8) {
 
----------------
ye-luo wrote:
> Need to be 0 instead of 8. Otherwise resize() happens before cuInit().
Oh, that is the reason I used `unique_ptr` before.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D116315