[Lldb-commits] [PATCH] D12651: Add ctrl-c support to parallel dotest.py.

[Todd Fiala via lldb-commits]

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?


http://reviews.llvm.org/D12651