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

[Todd Fiala via lldb-commits]

tfiala updated this revision to Diff 34114.
tfiala added a comment.

This change adds the threading-module-based parallel test runner strategy.  It is not selected by default.  It is pool-based.

The threading-pool and multiprocessing-pool strategies are 19.8% slower than the newer hand-wrapped ctrl-c-supporting, hand-wrapped multiprocessing pool approach.  I'm going to keep the multiprocessing strategy turned on for everything except Windows NT.

I may later try to hand-wrap the pooling on the threading side as well, since it looks like the slowdown may actually be the Pool provided by multiprocessing.pool (which we were always using, so the multiprocessing test runner strategy represents a potential 20% win for same-class Linux boxes vs. the previous approach currently at top of tree).  These numbers were on a 24-core Linux box using 24 processes/threads (the defaults for the box).

If I get this working on a Windows box with the current defaults, I'll check it in.


http://reviews.llvm.org/D12651

Files:
  test/dosep.py
  test/dotest.py
  test/dotest_args.py

-------------- next part --------------
A non-text attachment was scrubbed...
Name: D12651.34114.patch
Type: text/x-patch
Size: 21465 bytes
Desc: not available
URL: <http://lists.llvm.org/pipermail/lldb-commits/attachments/20150905/2e8d5fd8/attachment-0001.bin>