Skip to content

Commit

Permalink
[lit] Push computation of workers into run_tests
Browse files Browse the repository at this point in the history
This also avoids assigning to the option object.
  • Loading branch information
Julian Lettner committed Apr 30, 2020
1 parent 511868d commit 3cd6e02
Showing 1 changed file with 4 additions and 4 deletions.
8 changes: 4 additions & 4 deletions llvm/utils/lit/lit/main.py
Expand Up @@ -83,8 +83,6 @@ def main(builtin_params={}):

filtered_tests = filtered_tests[:opts.max_tests]

opts.workers = min(len(filtered_tests), opts.workers)

start = time.time()
run_tests(filtered_tests, lit_config, opts, len(discovered_tests))
elapsed = time.time() - start
Expand Down Expand Up @@ -190,14 +188,16 @@ def filter_by_shard(tests, run, shards, lit_config):


def run_tests(tests, lit_config, opts, discovered_tests):
workers = min(len(tests), opts.workers)
display = lit.display.create_display(opts, len(tests), discovered_tests,
opts.workers)
workers)

def progress_callback(test):
display.update(test)
if opts.order == 'failing-first':
touch_file(test)

run = lit.run.Run(tests, lit_config, opts.workers, progress_callback,
run = lit.run.Run(tests, lit_config, workers, progress_callback,
opts.max_failures, opts.timeout)

display.print_header()
Expand Down

0 comments on commit 3cd6e02

Please sign in to comment.