[PATCH] D122251: [lit] Use sharding for GoogleTest format

Yuanfang Chen via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Fri Apr 1 15:21:36 PDT 2022


ychen marked an inline comment as done.
ychen added inline comments.


================
Comment at: llvm/utils/lit/lit/main.py:112
 
+    post_process_gtest_results(selected_tests, discovered_tests)
+
----------------
yln wrote:
> Can we move this function out into to a gtest-specific file and make it return new (filtered & changed) lists leaving the passed-in ones intact.  Something like:
> ```
> selected_tests, discovered_tests = gtest.post_process_results(selected_tests, discovered_tests)
> ```
> Thanks!
Yep, done.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D122251



More information about the llvm-commits mailing list