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

[Jon Chesterfield via Phabricator via Openmp-commits]

JonChesterfield added a comment.

In D81054#2484343 <https://reviews.llvm.org/D81054#2484343>, @tianshilei1992 wrote:

> I’m going to put the issue on the top of my list.

Nice! Thank you.

I was thinking of adding a function to the plugin api, 'enable_memory_manager' or similar, which defaults to false. It seems the amd internal branch currently has an `#if 0` around the entry point to avoid checking an environment variable, but I'd really like to get rid of that local patch.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D81054