[Openmp-commits] [PATCH] D81054: [OpenMP] Introduce target memory manager

[Ye Luo via Phabricator via Openmp-commits]

ye-luo added a comment.

I think this optimization can be an option but not replacing the existing scheme directly allocate/free memory.
Application may request device memory outside openmp and use vendor native programming model or libraries.
Having libomptarget holding large memory doesn't make sense.
You may consider using the pool only for very small allocation requests <1M.
It is application's responsibility to take care of large memory allocation.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D81054