[Lldb-commits] [PATCH] D98179: [lit] Sort test start times based on prior test timing data
David Zarzycki via Phabricator via lldb-commits
lldb-commits at lists.llvm.org
Fri Mar 12 03:16:09 PST 2021
davezarzycki updated this revision to Diff 330194.
davezarzycki added a comment.
I've made all of the requested changes to date. Two notes:
1. One cannot simply omit the first argument to `split()` if the second is provided. As I just learned, python will complain. I switched the first parameter to `None` as a compromise.
2. For the record, I disagree with the use of the word "run" over "start" but this isn't something that I care enough about to resist changing the patch. Strictly speaking, `lit` only controls when tests start, not anything about how they run. With enough cores, tests will complete in a fairly predictable order despite any effort to start tests in a different order (or randomly).
rG LLVM Github Monorepo
CHANGES SINCE LAST ACTION
-------------- next part --------------
A non-text attachment was scrubbed...
Size: 15302 bytes
Desc: not available
More information about the lldb-commits