Skip to content

[cxx-interop] Mark un-specialized class templates as unavailable in Swift. #60786

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 5 commits into from
Oct 26, 2022

Conversation

zoecarver
Copy link
Contributor

They don't really work, so let's not "support" them yet.

@zoecarver
Copy link
Contributor Author

Draft because I haven't fixed the tests yet.

@zoecarver zoecarver added the c++ interop Feature: Interoperability with C++ label Aug 25, 2022
@zoecarver zoecarver force-pushed the unavailable-attr-class-templates branch from ab84b15 to f049d52 Compare September 27, 2022 01:41
@zoecarver zoecarver marked this pull request as ready for review September 27, 2022 01:41
@zoecarver
Copy link
Contributor Author

@swift-ci please test

@zoecarver
Copy link
Contributor Author

@hyp @egorzhdan this is ready for review now.

@zoecarver zoecarver changed the title [wip][cxx-interop] Mark un-specialized class templates as unavailable in Swift. [cxx-interop] Mark un-specialized class templates as unavailable in Swift. Sep 27, 2022
Copy link
Contributor

@egorzhdan egorzhdan left a comment

Choose a reason for hiding this comment

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

LGTM.
You'll probably need something like using ArrayRefOfDiagnosticArgument = ArrayRef<DiagnosticArgument> to make sure SwiftCompilerSources still compile.

@zoecarver zoecarver force-pushed the unavailable-attr-class-templates branch from f049d52 to dbf4864 Compare October 20, 2022 01:11
@zoecarver
Copy link
Contributor Author

@swift-ci please test

1 similar comment
@zoecarver
Copy link
Contributor Author

@swift-ci please test

@@ -1,9 +1,9 @@
// RUN: %empty-directory(%t)
Copy link
Contributor Author

Choose a reason for hiding this comment

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

@hyp I'll let you decide if you want to land this PR before fixing this test and the next one. Sorry I did such a terrible job "fixing" these. I couldn't always understand the error, so maybe we can work on them offline.

@zoecarver
Copy link
Contributor Author

@swift-ci please test

@zoecarver
Copy link
Contributor Author

@swift-ci please test

@zoecarver zoecarver merged commit ac131df into swiftlang:main Oct 26, 2022
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++
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants