[PATCH] D28789: [lit] Support sharding testsuites, for parallel execution.

Daniel Dunbar via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Tue Jan 17 11:37:26 PST 2017


ddunbar accepted this revision.
ddunbar added a comment.
This revision is now accepted and ready to land.

LGTM, this seems like a great idea!



================
Comment at: utils/lit/lit/main.py:436
+                            shard_begin+1, shard_end))
+            run.tests = run.tests[shard_begin:shard_end]
+
----------------
Would it be better to shard in a round robin fashion? There is some tendency for tests to be clumped by where they are defined, and where they are defined to be (weakly) correlated with how long they take to run, so that would distribute long running tests across machines, which should help reduce the deviation between total testing time among shards.


https://reviews.llvm.org/D28789





More information about the llvm-commits mailing list