[PATCH] D84235: [lit][xunit output] Don't include tests skipped due to sharding

Julian Lettner via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Wed Jul 22 11:56:53 PDT 2020


yln added a comment.

Thanks for working on this! I forgot to consider the interaction with sharding.



================
Comment at: llvm/utils/lit/lit/main.py:105
     for report in opts.reports:
-        report.write_results(discovered_tests, elapsed)
+        report.write_results(discovered_tests, elapsed, opts.runShard)
 
----------------
Can we filter the list of tests here (if sharding is active)? Other report types are also affected by this.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D84235





More information about the llvm-commits mailing list