Skip to content

swift_build_sdk_interfaces.py: builds compiled modules from an SDK #25432

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

Conversation

jrose-apple
Copy link
Contributor

...by crawling it and looking for module interfaces. This is the tool Apple is using for the "prebuilt-modules" directory in the toolchain in Xcode.app, which avoids even a first-use cost for importing modules from Apple SDKs as long as the SDKs haven't changed. We've been landing changes to support this model on the compiler side, so the tool to generate the "prebuilt" compiled modules belongs in the compiler repo too. (In theory this could be used with open-source toolchains as well, or for alternate SDKs.)

@jrose-apple
Copy link
Contributor Author

@swift-ci Please smoke test

Copy link
Contributor

@bcardosolopes bcardosolopes left a comment

Choose a reason for hiding this comment

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

LGTM

@jrose-apple jrose-apple force-pushed the swift_build_sdk_interfaces.py branch from 3447829 to 1883338 Compare June 14, 2019 18:04
@jrose-apple
Copy link
Contributor Author

@swift-ci Please test

@swift-ci
Copy link
Contributor

Build failed
Swift Test Linux Platform
Git Sha - 3447829ad15236e4261ba994dc6711794c18a08d

@swift-ci
Copy link
Contributor

Build failed
Swift Test OS X Platform
Git Sha - 3447829ad15236e4261ba994dc6711794c18a08d

@jrose-apple
Copy link
Contributor Author

ClockKit issue on watchOS. :-(

@swift-ci Please smoke test macOS

...by crawling it and looking for module interfaces. This is the tool
Apple is using for the "prebuilt-modules" directory in the toolchain
in Xcode.app, which avoids even a first-use cost for importing modules
from Apple SDKs as long as the SDKs haven't changed. We've been
landing changes to support this model on the compiler side, so the
tool to generate the "prebuilt" compiled modules belongs in the
compiler repo too. (In theory this could be used with open-source
toolchains as well, or for alternate SDKs.)
@jrose-apple jrose-apple force-pushed the swift_build_sdk_interfaces.py branch from 1883338 to b2c1035 Compare June 14, 2019 22:19
@jrose-apple
Copy link
Contributor Author

@swift-ci Please smoke test

@jrose-apple jrose-apple merged commit 43bb5e4 into swiftlang:master Jun 17, 2019
@jrose-apple jrose-apple deleted the swift_build_sdk_interfaces.py branch June 17, 2019 17:29
@compnerd
Copy link
Member

@jrose-apple
Copy link
Contributor Author

*sigh* They're all path separators…

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.

5 participants