Skip to content

[cxx-interop] Do not require the LifetimeDependence feature in _SwiftifyImport #79911

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
Mar 12, 2025

Conversation

Xazax-hun
Copy link
Contributor

We use experimental features to let people know that the construct is subject to change and users should not rely on this unless they are willing to rewrite the uses of this feature later. However, in compiler generated code everything should be fair game, we will update the compiler when these features change. This is a requirement to be able to turn safe wrapper generation on by default.

@Xazax-hun Xazax-hun added the c++ interop Feature: Interoperability with C++ label Mar 11, 2025
@Xazax-hun Xazax-hun requested review from egorzhdan and j-hui March 11, 2025 15:10
Copy link
Member

@DougGregor DougGregor left a comment

Choose a reason for hiding this comment

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

LGTM

@Xazax-hun Xazax-hun enabled auto-merge March 11, 2025 16:18
@Xazax-hun Xazax-hun force-pushed the gaborh/enable-lifetime-in-macro-generated branch from 4d04421 to ff8ad7e Compare March 11, 2025 16:20
…ifyImport

We use experimental features to let people know that the construct is
subject to change and users should not rely on this unless they are
willing to rewrite the uses of this feature later. However, in compiler
generated code everything should be fair game, we will update the
compiler when these features change. This is a requirement to be able to
turn safe wrapper generation on by default.
@Xazax-hun Xazax-hun force-pushed the gaborh/enable-lifetime-in-macro-generated branch from ff8ad7e to 22e2276 Compare March 11, 2025 17:39
@Xazax-hun
Copy link
Contributor Author

@swift-ci please smoke test

@Xazax-hun
Copy link
Contributor Author

@swift-ci please smoke test macos

@Xazax-hun Xazax-hun merged commit bc6c766 into main Mar 12, 2025
3 checks passed
@Xazax-hun Xazax-hun deleted the gaborh/enable-lifetime-in-macro-generated branch March 12, 2025 00:59
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.

2 participants