Skip to content

build-script: Throttle the number of parallel tests run acording to -j. #25798

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Merged
merged 1 commit into from
Jun 27, 2019

Conversation

adrian-prantl
Copy link
Contributor

This patch propagates the number of parallel jobs from build-script to
llvm-lit. Due to the way how build-script-impl is implemented, changing this
number needs a reconfigure for the Swift tests and can be done on the fly for
LLDB tests, since build-script-impl directly invokes llvm-lit in the latter
case.

rdar://problem/52062631

This patch propagates the number of parallel jobs from build-script to
llvm-lit. Due to the way how build-script-impl is implemented, changing this
number needs a reconfigure for the Swift tests and can be done on the fly for
LLDB tests, since build-script-impl directly invokes llvm-lit in the latter
case.

rdar://problem/52062631
@adrian-prantl adrian-prantl requested a review from shahmishal June 26, 2019 22:19
@adrian-prantl
Copy link
Contributor Author

@swift-ci test and merge

@swift-ci swift-ci merged commit 0c2f0c4 into swiftlang:swift-5.1-branch Jun 27, 2019
@shahmishal
Copy link
Member

it still used 32 cores - 15:55:33 -- Testing: 12001 tests, 32 threads --

@adrian-prantl
Copy link
Contributor Author

On which bot did you see that?

https://ci.swift.org/view/swift-5.1-branch/job/oss-swift-5.1-incremental-RA-linux-ubuntu-16_04/1287/consoleFull#console-section-7

-- Testing: 11985 tests, 16 threads --

@adrian-prantl
Copy link
Contributor Author

@adrian-prantl
Copy link
Contributor Author

Note that the change needs a --reconfigure to go into effect.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants