[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