[PATCH] D99073: [lit] Reliable progress indicator and ETA

Roman Lebedev via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Mon Mar 22 11:34:29 PDT 2021


lebedev.ri added inline comments.


================
Comment at: llvm/utils/lit/lit/display.py:9-10
+    num_tests = len(tests)
+    of_total = (' of %d' % total_tests) if (num_tests != total_tests) else ''
+    header = '-- Testing: %d%s tests, %d workers --' % (num_tests, of_total, workers)
 
----------------
yln wrote:
> Could now use string interpolation (because we require Python 3.6).
I think i'll prefer to leave that as-is.
If wanted, a cleanup en masse could be performed later.


Repository:
  rG LLVM Github Monorepo

CHANGES SINCE LAST ACTION
  https://reviews.llvm.org/D99073/new/

https://reviews.llvm.org/D99073



More information about the llvm-commits mailing list