[PATCH] D47210: [lit] Fix the `--max-time` flag feature which was completely broken.

Reid Kleckner via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Tue May 22 11:25:05 PDT 2018


rnk added inline comments.


================
Comment at: utils/lit/lit/run.py:127
+    def execute_tests_sequentially_in_process(self, max_time):
+        global child_lit_config
+        deadline = None
----------------
If you are running the tests in process, there is no need to use this global child_lit_config. It is only needed so that mulitprocessing can pickle the data and set it up in the child process. We should be able to update self.lit_confi.maxIndividualTest time directly, and it will take effect.


Repository:
  rL LLVM

https://reviews.llvm.org/D47210





More information about the llvm-commits mailing list