Skip to content

[cxx-interop] enable support for move-only types #69790

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 24 commits into from
Dec 6, 2023

Conversation

hyp
Copy link
Contributor

@hyp hyp commented Nov 10, 2023

Still working on more testing and bug fixing. Some features require -enable-experimental-feature NoncopyableGenerics.

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

hyp commented Nov 10, 2023

@swift-ci please test

@hyp hyp force-pushed the eng/move-only-is-back branch from aff6a6b to 86e5556 Compare November 16, 2023 20:56
@hyp
Copy link
Contributor Author

hyp commented Nov 16, 2023

@swift-ci please test

@hyp
Copy link
Contributor Author

hyp commented Nov 17, 2023

@swift-ci please test windows platform

@hyp
Copy link
Contributor Author

hyp commented Nov 27, 2023

@swift-ci please test

@hyp
Copy link
Contributor Author

hyp commented Nov 27, 2023

@swift-ci please test

@hyp
Copy link
Contributor Author

hyp commented Nov 27, 2023

The pointee now works, but the use of subscripts is going to be blocked by: #70047

@hyp
Copy link
Contributor Author

hyp commented Nov 29, 2023

@swift-ci please test

@hyp
Copy link
Contributor Author

hyp commented Nov 29, 2023

@swift-ci please test

@hyp
Copy link
Contributor Author

hyp commented Nov 30, 2023

@swift-ci please test

@hyp hyp marked this pull request as ready for review November 30, 2023 04:43
@hyp
Copy link
Contributor Author

hyp commented Nov 30, 2023

@swift-ci please test

1 similar comment
@hyp
Copy link
Contributor Author

hyp commented Nov 30, 2023

@swift-ci please test

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.

Looks great, thanks!

@hyp hyp force-pushed the eng/move-only-is-back branch from eb63b0d to c029ed6 Compare December 1, 2023 23:28
@hyp
Copy link
Contributor Author

hyp commented Dec 1, 2023

@swift-ci please test

@hyp
Copy link
Contributor Author

hyp commented Dec 3, 2023

@swift-ci please test

@hyp
Copy link
Contributor Author

hyp commented Dec 4, 2023

@swift-ci please test

1 similar comment
@hyp
Copy link
Contributor Author

hyp commented Dec 5, 2023

@swift-ci please test

@hyp hyp force-pushed the eng/move-only-is-back branch from cf2f682 to ec931c8 Compare December 5, 2023 21:42
@hyp
Copy link
Contributor Author

hyp commented Dec 5, 2023

@swift-ci please test

@hyp hyp force-pushed the eng/move-only-is-back branch from e5b8445 to 623d3d2 Compare December 5, 2023 22:16
@hyp
Copy link
Contributor Author

hyp commented Dec 5, 2023

@swift-ci please test

@hyp
Copy link
Contributor Author

hyp commented Dec 5, 2023

@swift-ci please test source compatibility

@hyp hyp merged commit c7345d0 into swiftlang:main Dec 6, 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.

3 participants