[Openmp-commits] [PATCH] D111954: [OpenMP][Plugin] Introduce resouce manager

[Ye Luo via Phabricator via Openmp-commits]

ye-luo added inline comments.


================
Comment at: openmp/libomptarget/plugins/cuda/src/rtl.cpp:244
+  /// A reference to the corresponding allocator.
+  AllocatorTy<T> &Allocator;
+
----------------
How about making this a member rather than a reference and remove StreamAllocator vector?
ResourcePoolTy(AllocatorTy<T> &&A, size_t Size = 0)


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D111954