Skip to content

[5.6] Turn on emit-module-separately for whole-module builds by default #960

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

Closed
wants to merge 5 commits into from

Conversation

xymus
Copy link
Contributor

@xymus xymus commented Dec 23, 2021

Scheduling a separate emit-module job for whole-module builds, combined with the integrated driver in the build system, will allow to parallelize what used to be essentially sequential builds. This mode unblocks building dependents early by duplicating some work in two processes, it requires more cpu work for a faster wall clock build time. Another advantage of this mode is the resulting swiftmodule is lighter from a classic whole-module build.

Cherry-pick from #922.

rdar://85976762

Scheduling a separate emit-module job for whole-module builds, combined
with the integrated driver in the build system, will allow to
parallelize what used to be essentially sequential builds. This mode
unblocks building dependents early by duplicating some work in two
processes, it requires more cpu work for a faster wall clock build time.
Another advantage of this mode is the resulting swiftmodule is lighter
from a classic whole-module build.

rdar://85976762
@xymus
Copy link
Contributor Author

xymus commented Dec 23, 2021

@swift-ci Please test

@abertelrud
Copy link

The test failure is the same as in #929, which is that there is no downloadable Swift 5.6 toolchain yet, and @swift-ci test doesn't do a full build in this repository.

@xymus
Copy link
Contributor Author

xymus commented Jan 10, 2022

@swift-ci Please test

1 similar comment
@xymus
Copy link
Contributor Author

xymus commented Jan 11, 2022

@swift-ci Please test

@xymus xymus closed this Jan 26, 2022
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