-
Notifications
You must be signed in to change notification settings - Fork 10.4k
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
swift_build_sdk_interfaces.py: builds compiled modules from an SDK #25432
Conversation
042a327
to
3447829
Compare
@swift-ci Please smoke test |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM
3447829
to
1883338
Compare
@swift-ci Please test |
Build failed |
Build failed |
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.)
1883338
to
b2c1035
Compare
@swift-ci Please smoke test |
*sigh* They're all path separators… |
...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.)