aboutsummaryrefslogtreecommitdiffstats
diff options
context:
space:
mode:
authorBjörn Gustavsson <[email protected]>2016-05-10 13:15:04 +0200
committerBjörn Gustavsson <[email protected]>2016-05-13 10:52:24 +0200
commit6d66e21dd6b1b891f917e86902edaea3ed86a6eb (patch)
treeb1482694f3061c3bb13a43725c7a8f8cb3ea0218
parentb956a9ec915f4ec0ab3d0136ea32d8f5f159ae37 (diff)
downloadotp-6d66e21dd6b1b891f917e86902edaea3ed86a6eb.tar.gz
otp-6d66e21dd6b1b891f917e86902edaea3ed86a6eb.tar.bz2
otp-6d66e21dd6b1b891f917e86902edaea3ed86a6eb.zip
test_lib: Correct calculation of number of processes
Correct calculation of the number of parallel processes to use when cover is used. It was supposed not to exceed 4 (according to the comment), but it was calculated as the number of schedulers. On my computer, having 8 schedulers, this change made the compiler test suite with cover run almost twice as fast.
-rw-r--r--lib/compiler/test/test_lib.erl3
1 files changed, 2 insertions, 1 deletions
diff --git a/lib/compiler/test/test_lib.erl b/lib/compiler/test/test_lib.erl
index d141d86021..a9236df414 100644
--- a/lib/compiler/test/test_lib.erl
+++ b/lib/compiler/test/test_lib.erl
@@ -106,8 +106,9 @@ p_run(Test, List) ->
%% slightly faster than using 3. Using more than
%% 4 would not buy us much and could actually be
%% slower.
- max(S, 4)
+ min(S, 4)
end,
+ io:format("p_run: ~p parallel processes\n", [N]),
p_run_loop(Test, List, N, [], 0, 0).
p_run_loop(_, [], _, [], Errors, Ws) ->