Skip to content

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

Merged
artemcm merged 1 commit intoswiftlang:release/5.8from
artemcm:58EnableClangTarget
Jan 10, 2023
Merged

[5.8] Enable '-clang-target' by-default#1251
artemcm merged 1 commit intoswiftlang:release/5.8from
artemcm:58EnableClangTarget

Conversation

@artemcm
Copy link
Copy Markdown
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
Copy Markdown
Contributor Author

artemcm commented Jan 9, 2023

@swift-ci test

@artemcm
Copy link
Copy Markdown
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