[Openmp-commits] [PATCH] D81054: [OpenMP] Introduce target memory manager
Johannes Doerfert via Phabricator via Openmp-commits
openmp-commits at lists.llvm.org
Thu Jun 18 16:59:07 PDT 2020
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.
rG LLVM Github Monorepo
CHANGES SINCE LAST ACTION
More information about the Openmp-commits