[Openmp-commits] [PATCH] D132676: [OpenMP][Fix] Track all threads that may delete an entry

Guilherme Valarini via Phabricator via Openmp-commits openmp-commits at lists.llvm.org
Tue Jan 17 08:49:53 PST 2023


gValarini added a comment.

In D132676#4052127 <https://reviews.llvm.org/D132676#4052127>, @jdoerfert wrote:

> Ready to merge?

I believe it is.

There is just that one block of code you commented about that tries to manage the deallocation of nested custom mappers that I still cannot wrap my head around. But I believe we would need to revise how we manage the reference counting of those since they are probably being left allocated by the end of the program (https://github.com/llvm/llvm-project/issues/60106).

Do you think we can merge the patch and take care of that later?


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D132676



More information about the Openmp-commits mailing list