Skip to content

[Explicit Module Build] Execute module interface verification in a sub-invocation context #71416

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

artemcm
Copy link
Contributor

@artemcm artemcm commented Feb 6, 2024

Otherwise, with -explicit-interface-module-build they do not read out/inherit compiler flags written in the interface itself.

Resolves rdar://122418125

@artemcm artemcm requested a review from tshortli as a code owner February 6, 2024 20:19
@artemcm artemcm force-pushed the ExplicitInterfaceVerifyReadFlagsFix branch 2 times, most recently from 1d19c78 to eb76257 Compare February 6, 2024 20:47
@artemcm
Copy link
Contributor Author

artemcm commented Feb 6, 2024

@swift-ci smoke test

@artemcm artemcm force-pushed the ExplicitInterfaceVerifyReadFlagsFix branch 2 times, most recently from 3947113 to 8760a87 Compare February 6, 2024 21:31
@artemcm
Copy link
Contributor Author

artemcm commented Feb 6, 2024

@swift-ci smoke test

@artemcm artemcm enabled auto-merge February 6, 2024 21:49
@artemcm
Copy link
Contributor Author

artemcm commented Feb 7, 2024

@swift-ci test Windows platform

…b-invocation context

Otherwise, with `-explicit-interface-module-build` they do not read out/inherit compiler flags written in the interface itself.

Resolves rdar://122418125
@artemcm artemcm force-pushed the ExplicitInterfaceVerifyReadFlagsFix branch from 8760a87 to 55cc29f Compare February 7, 2024 16:46
@artemcm
Copy link
Contributor Author

artemcm commented Feb 7, 2024

@swift-ci smoke test

@artemcm artemcm merged commit a1062d0 into swiftlang:main Feb 7, 2024
@artemcm artemcm deleted the ExplicitInterfaceVerifyReadFlagsFix branch February 7, 2024 22:44
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.

4 participants