Skip to content

[s390x] Make stage 2 builder use clean builds #450

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
May 20, 2025

Conversation

uweigand
Copy link
Member

Use clean builds for the s390x multistage builder as was done for Linaro in #306. This will enable collapsing build requests to help the builder keep up.

As further speed-up, skip the stage 1 testing, as is already done for most other multistage builders.

Use clean builds for the s390x multistage builder as was done for
Linaro in llvm#306.  This will
enable collapsing build requests to help the builder keep up.

As further speed-up, skip the stage 1 testing, as is already done
for most other multistage builders.
Copy link
Contributor

@DavidSpickett DavidSpickett left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I see -DLLVM_CCACHE_BUILD=ON in the stage 1 configuration, so this shouldn't add much if any overhead, and https://lab.llvm.org/buildbot/#/builders/42 covers the stage 1 testing already.

LGTM.

@DavidSpickett
Copy link
Contributor

#250 was the origin of this. We never did find out why it was not an issue before then, I suspect it should have been an issue, but for a bug elsewhere possibly in Buildbot itself that got fixed at some point.

@uweigand uweigand merged commit 5520fd5 into llvm:main May 20, 2025
2 checks passed
@uweigand uweigand deleted the s390x-stage2-collapse branch May 20, 2025 14:22
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.

2 participants