[Openmp-commits] [PATCH] D25510: Fix for mistake done by https://reviews.llvm.org/D23115

[Andrey Churbanov via Openmp-commits]

AndreyChurbanov added a comment.

I haven't debugged your particular example, probably it should have the problem, you cannot see it without debugger of some memory checker tool.  We saw it on another pattern - when each thread launches tons of tasks so that all above first 256 were serialized in each thread (as the task queue is full) and the test crashes because of lack of memory after a long time execution.

I doubt writing small test case is feasible.  User level code works with and without the fix, as the leak is in the library internals. Compiler-generated code also works in both cases.

We found the bug accidentally on Spec OMP2012 gross tests, but the failure we saw took about an hour of running before the crash on our machine.  On small tests the problem is not visible.


Repository:
  rL LLVM

https://reviews.llvm.org/D25510