Skip to content

[interop][SwiftToCxx] Use 'SWIFT_INLINE_PRIVATE_HELPER' for getTypeMetadata helper to avoid emitting a reference to it when 'DEBUG' macro is set #69519

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
Nov 1, 2023

Conversation

hyp
Copy link
Contributor

@hyp hyp commented Oct 30, 2023

Fixes #69234

Based on #69473, see last commit only

@hyp hyp added the c++ interop Feature: Interoperability with C++ label Oct 30, 2023
@hyp
Copy link
Contributor Author

hyp commented Oct 30, 2023

@swift-ci please test

…tadata helper to avoid emitting a reference to it when 'DEBUG' macro is set

This is needed as the metadata for C++ types might not be actually emitted and thus we can't always reference it

Fixes swiftlang#69234

rdar://117089662
@hyp hyp force-pushed the eng/get-type-metadata-helper-inline branch from 77184e7 to b1d63d0 Compare October 31, 2023 16:21
@hyp
Copy link
Contributor Author

hyp commented Oct 31, 2023

@swift-ci please test

1 similar comment
@hyp
Copy link
Contributor Author

hyp commented Oct 31, 2023

@swift-ci please test

@hyp
Copy link
Contributor Author

hyp commented Nov 1, 2023

@swift-ci please test macOS platform

@hyp hyp merged commit ea38d57 into swiftlang:main Nov 1, 2023
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.

Swift-to-C++: getTypeMetadata inline helpers that refer to C++ type metadata are annotated as used for debug compilations, causing linker errors
1 participant