Skip to content

Conversation

@vvereschaka
Copy link
Contributor

The latest changes for 'flang' and 'flang-rt' projects cause the build host overload and freeze the system for a long time. Reduce resource usage for the build to avoid it.

Note: restore the changes after the problem will get solved.

@vvereschaka vvereschaka marked this pull request as draft April 2, 2025 21:21
Copy link
Contributor

@gkistanova gkistanova left a comment

Choose a reason for hiding this comment

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

LGTM.

…builder-7 worker.

The latest changes for 'flang' and 'flang-rt' projects cause the build host overload
and freeze the system for a long time. Reduce resource usage for the build to avoid it.

Note: restore the changes after the problem will get solved.
@vvereschaka vvereschaka force-pushed the update-as-builder-7-config branch from a7593b4 to d0f2953 Compare April 3, 2025 00:31
@vvereschaka
Copy link
Contributor Author

I repalced UnifiedTreeBuilder.getCmakeWithNinjaBuildFactory call with UnifiedTreeBuilder.getCmakeExBuildFactory. The first one does not support the jobs argument.

@vvereschaka vvereschaka marked this pull request as ready for review April 3, 2025 02:39
@vvereschaka vvereschaka requested a review from gkistanova April 3, 2025 02:39
Copy link
Contributor

@gkistanova gkistanova left a comment

Choose a reason for hiding this comment

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

The first one supports jobs as a build property. But this one seems fine too.

@vvereschaka vvereschaka merged commit d94235a into llvm:main Apr 3, 2025
2 checks passed
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