[Openmp-commits] [libcxx] [llvm] [openmp] [lit] Add a flag to disable lit time tests (PR #98270)
Vincent Lee via Openmp-commits
openmp-commits at lists.llvm.org
Sat Jul 13 00:01:51 PDT 2024
================
@@ -124,7 +124,7 @@ def main(builtin_params={}):
run_tests(selected_tests, lit_config, opts, len(discovered_tests))
elapsed = time.time() - start
- if not opts.skip_test_recording or opts.time_tests:
+ if not opts.skip_test_time_recording or opts.time_tests:
----------------
thevinster wrote:
The original idea was to prevent the case where both `--skip-test-time-recording` and `--time-tests` are both passed. If it didn't overwrite, then we'd run into the situation where we wouldn't time the tests, but attempt to print the histogram.
Now that I think about it, the `--time-tests` parameter is redundant since you cannot enable both `--time-tests` and `--skip-test-time-recording` as they are mutually exclusive args.
https://github.com/llvm/llvm-project/pull/98270
More information about the Openmp-commits
mailing list