Skip to content

Commit

Permalink
[OpenMP] Fix not respecting -j in additional checks
Browse files Browse the repository at this point in the history
This fixes that the implementation missed to pass a potentially specified
`-j` option to the additionally specified test targets.

It seems that libc is however not respecting this unless a previous `cd`
into the runtimes-build directory is done. This will be in a separate
patch.

Differential Revision: https://reviews.llvm.org/D149756
  • Loading branch information
jplehr committed May 3, 2023
1 parent 8736ff1 commit 8027f90
Showing 1 changed file with 2 additions and 1 deletion.
3 changes: 2 additions & 1 deletion zorg/buildbot/builders/OpenMPBuilder.py
Expand Up @@ -132,9 +132,10 @@ def cleanObjRequested(step):
# When requested run additional lit tests
if add_lit_checks != None:
for add_check in add_lit_checks:
ninja_test_args = ['ninja', WithProperties('-j %s' % jobs)]
f.addStep(LitTestCommand(
name = 'Add check ' + add_check,
command = ['ninja', add_check],
command = [ninja_test_args, add_check],
description = ["Additional check in OpenMP for", add_check,],
env = merged_env,
workdir = f.obj_dir,
Expand Down

0 comments on commit 8027f90

Please sign in to comment.