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

[Johannes Doerfert via Phabricator via Openmp-commits]

jdoerfert added a comment.

In D81054#2213237 <https://reviews.llvm.org/D81054#2213237>, @JonChesterfield wrote:

> OK, cool. If we're open to changing the implementation later this is fine by me.

Always!

> An instance per host thread is likely to be better than all the internal locks.

That is one of the things we can profile and change, no objection if it turns out problematic.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D81054