Skip to content

Enable '-clang-target' by-default #1250

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
Jan 10, 2023
Merged

Conversation

artemcm
Copy link
Contributor

@artemcm artemcm commented Jan 9, 2023

This will result in the driver passing down '-clang-target' flag to the frontend, which will then ensure that all downstream module (Swift and Clang) compilation compiler sub-invocations are consistent with respect to the target triple used by their underlying instances of Clang. Resulting in all Clang module dependencies being built against the main module's target triple.

@artemcm
Copy link
Contributor Author

artemcm commented Jan 9, 2023

@swift-ci test

This will result in the driver passing down '-clang-target' flag to the frontend, which will then ensure that all downstream module (Swift and Clang) compilation compiler sub-invocations are consistent with respect to the target triple used by their underlying instances of Clang. Resulting in all Clang module dependencies being built against the main module's target triple.
@artemcm artemcm force-pushed the EnableClangTarget branch from 3031fa8 to 63bd1d9 Compare January 9, 2023 22:16
@artemcm
Copy link
Contributor Author

artemcm commented Jan 9, 2023

@swift-ci test

@artemcm artemcm requested a review from nkcsgexi January 10, 2023 17:38
@artemcm artemcm marked this pull request as ready for review January 10, 2023 17:38
@artemcm artemcm requested a review from xymus January 10, 2023 17:38
@artemcm
Copy link
Contributor Author

artemcm commented Jan 10, 2023

Cross-repo tests in swiftlang/swift#60581 are also green.

Copy link
Contributor

@nkcsgexi nkcsgexi left a comment

Choose a reason for hiding this comment

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

So excited to see this happening by default!

@artemcm artemcm merged commit d1dfa2d into swiftlang:main Jan 10, 2023
@artemcm artemcm deleted the EnableClangTarget branch January 10, 2023 17:56
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