Skip to content

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
@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