Skip to content

[Explicit Module Builds] Remove dead/incorrect code for handling header inputs of binary module dependencies #1653

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
Jul 8, 2024

Conversation

artemcm
Copy link
Contributor

@artemcm artemcm commented Jul 8, 2024

As of swiftlang/swift#72067, we instead serialize .h inputs directly into binary .swiftmodule files, because their clients may not be able to use the dependnecies' corresponding .pch files due to a compilation context mismatch. The clients of such binary modules then consume the serialized .h files directly, and compile them, implicitly, using explicit module dependencies collected during scan of such header files.

Resolves rdar://131261765

…er inputs of binary module dependencies

As of swiftlang/swift#72067, we instead serialize `.h` inputs directly into binary `.swiftmodule` files, because their clients may not be able to use the dependnecies' corresponding `.pch` files due to a compilation context mismatch. The clients of such binary modules then consume the serialized `.h` files directly, and compile them, implicitly, using explicit module dependencies collected during scan of such header files.

Resolves rdar://131261765
@artemcm artemcm requested a review from cachemeifyoucan July 8, 2024 17:35
Copy link
Contributor

@cachemeifyoucan cachemeifyoucan left a comment

Choose a reason for hiding this comment

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

LGTM

@artemcm
Copy link
Contributor Author

artemcm commented Jul 8, 2024

@swift-ci test

@artemcm artemcm merged commit d7de803 into swiftlang:main Jul 8, 2024
3 checks passed
@artemcm artemcm deleted the RemoveDeadBinaryDepPCHCode branch July 8, 2024 23:12
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