Skip to content

[6.2][cxx-interop] Only swiftify template instantiations behind typealiases #82105

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

Xazax-hun
Copy link
Contributor

Explanation: C++ template instantiations that are not behind type aliases don't have corresponding Swift names that are both syntactically and semantically valid types. This PR prevents generating swiftified overloads for those types.
Issue: rdar://151422108
Risk: Low, we swiftify functions less often.
Testing: Regression test added.
Original PR: #81973
Reviewer: @hnrklssn

@Xazax-hun Xazax-hun requested a review from hnrklssn June 9, 2025 12:31
@Xazax-hun Xazax-hun requested a review from a team as a code owner June 9, 2025 12:31
@Xazax-hun Xazax-hun added c++ interop Feature: Interoperability with C++ 🍒 release cherry pick Flag: Release branch cherry picks swift 6.2 labels Jun 9, 2025
@Xazax-hun Xazax-hun force-pushed the gaborh/disable-non-aliased-instantiations-on-6.2 branch from 64367ff to 5c2876b Compare June 11, 2025 10:34
…aliases

Explanation: C++ template instantiations that are not behind type
aliases don't have corresponding Swift names that are both syntactically
and semantically valid types. This PR prevents generating swiftified
overloads for those types.
Issue: rdar://151422108
Risk: Low, we swiftify functions less often.
Testing: Regression test added.
Original PR: #81973
Reviewer: @hnrklssn
@Xazax-hun Xazax-hun force-pushed the gaborh/disable-non-aliased-instantiations-on-6.2 branch from 5c2876b to 4590d55 Compare June 12, 2025 18:06
@Xazax-hun
Copy link
Contributor Author

@swift-ci please test

@Xazax-hun Xazax-hun enabled auto-merge June 12, 2025 18:06
@Xazax-hun Xazax-hun merged commit 9bf1d28 into release/6.2 Jun 13, 2025
5 checks passed
@Xazax-hun Xazax-hun deleted the gaborh/disable-non-aliased-instantiations-on-6.2 branch June 13, 2025 03:21
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
c++ interop Feature: Interoperability with C++ 🍒 release cherry pick Flag: Release branch cherry picks swift 6.2
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants