[Lldb-commits] [PATCH] D12651: Add ctrl-c support to parallel dotest.py.
Todd Fiala via lldb-commits
lldb-commits at lists.llvm.org
Fri Sep 4 15:54:44 PDT 2015
tfiala added a comment.
In http://reviews.llvm.org/D12651#240439, @zturner wrote:
> Hmm, yea the problem is worse than I thought. Even if I don't do Ctrl+C at
> all (I just run the test suite like I always have and let it finish) it's
> just leaving a bunch of stale processes at the end without them shutting
> down on their own.
Ok, and it doesn't do that prior to the patch? (i.e. this isn't just a case of the processes slowly falling off at the end?)
There's a point where all the worker processes are joined, which should mean they're done. So either the join is failing to complete (maybe because something in the process isn't wrapping up right), or the join isn't causing them to die.
Is the initial test runner itself finishing up?
More information about the lldb-commits