[llvm-branch-commits] [llvm] [openmp] [offload] Pool host and shared allocations (PR #214754)

[Joseph Huber via llvm-branch-commits]

================
@@ -1368,8 +1368,12 @@ struct GenericDeviceTy : public DeviceAllocatorTy {
   /// only necessary for unhosted targets like the GPU.
   virtual bool shouldSetupRPCServer() const { return false; }
 
-  /// Pointer to the memory manager or nullptr if not available.
+  /// Pointer to the device memory manager or nullptr if not available.
   MemoryManagerTy *MemoryManager;
+  /// Memory managers for the host and shared allocation kinds or nullptr if not
+  /// available.
+  MemoryManagerTy *HostMemoryManager;
+  MemoryManagerTy *SharedMemoryManager;
----------------
jhuber6 wrote:

Unrelated to the PR, but what's the actual difference between host and shared for us? IIRC 'Shared' is supposed to correspond to migratable pages, but I don't think we ever hooked up SVM. If it's host-accessible VRAM that's an entirely different beast.

https://github.com/llvm/llvm-project/pull/214754