Skip to content

[cxx-interop] Diagnose ObjC APIs returning SWIFT_SHARED_REFERENCE types #78458

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

fahadnayyar
Copy link
Contributor

@fahadnayyar fahadnayyar commented Jan 7, 2025

Adding diagnostic support (related to SWIFT_RETURNS_(UN)RETAINED annotations) for ObjC APIs returning SWIFT_SHARED_REFERENCE types

rdar://142500663

@fahadnayyar fahadnayyar added the c++ interop Feature: Interoperability with C++ label Jan 7, 2025
@fahadnayyar fahadnayyar force-pushed the returns-retained-objc-diagnostics branch 2 times, most recently from 8db53ac to eba183a Compare January 7, 2025 23:30
@fahadnayyar fahadnayyar force-pushed the returns-retained-objc-diagnostics branch from eba183a to 6f5c98c Compare January 8, 2025 02:12
Copy link
Contributor

@Xazax-hun Xazax-hun left a comment

Choose a reason for hiding this comment

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

Overall, looks good to me. If you can make the tests use -verify that would be nice. But in case you have confidence in the current testing method I am OK with merging this as is (after the last nits are fixed).

@fahadnayyar fahadnayyar force-pushed the returns-retained-objc-diagnostics branch from 6f5c98c to 1c140ce Compare January 8, 2025 19:34
@fahadnayyar fahadnayyar force-pushed the returns-retained-objc-diagnostics branch from 1c140ce to 69332f7 Compare January 8, 2025 19:37
@fahadnayyar
Copy link
Contributor Author

@swift-ci please smoke test

@fahadnayyar fahadnayyar merged commit d2e0f66 into swiftlang:main Jan 8, 2025
3 checks passed
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