Skip to content

[5.8] Enable '-clang-target' by-default #1251

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

Conversation

artemcm
Copy link
Contributor

@artemcm artemcm commented Jan 9, 2023

Cherry-pick of #1250

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.

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

@artemcm
Copy link
Contributor Author

artemcm commented Jan 10, 2023

@swift-ci test macOS platform

@artemcm artemcm marked this pull request as ready for review January 10, 2023 17:56
@artemcm artemcm requested a review from nkcsgexi January 10, 2023 17:56
@artemcm artemcm merged commit 8e4108b into swiftlang:release/5.8 Jan 10, 2023
@artemcm artemcm deleted the 58EnableClangTarget branch January 10, 2023 18:02
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants