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

[Johannes Doerfert via Phabricator via Openmp-commits]

jdoerfert added a comment.

In D81054#2102212 <https://reviews.llvm.org/D81054#2102212>, @ye-luo wrote:

> Having a allocation size limit may work. Below MAXSIZE, got to manager. Above MAXSIZE, go directly.
>  Power user can do even MAXSIZE=0 to fully skip the manager if they want.


I like that idea. 
By default I want some memory management for people and the smaller the allocations the more important it is. That said, opting out should always be an option.

@tianshilei1992 please also make sure that the runtime shutdown routines will free this memory.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D81054