Skip to content

Support std::unique_ptr. #65577

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

Closed
wants to merge 4 commits into from
Closed

Conversation

zoecarver
Copy link
Contributor

Currently the synthesized members consume but that's OK for now. At least it's correct.

@zoecarver zoecarver requested review from hyp and egorzhdan as code owners May 2, 2023 06:16
@zoecarver
Copy link
Contributor Author

@swift-ci please test

import StdUniquePtr
#if os(Linux)
import CxxStdlib
// FIXME: import CxxStdlib.string once libstdc++ is split into submodules.
Copy link
Contributor

Choose a reason for hiding this comment

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

That's not going to happen. Can you just import CxxStdlib?

Copy link
Contributor

Choose a reason for hiding this comment

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

We have a bunch of existing tests that do this, I'll update them to just import CxxStdlib.

@zoecarver
Copy link
Contributor Author

@swift-ci please test

@hyp hyp added the c++ interop Feature: Interoperability with C++ label May 2, 2023
@kavon kavon self-requested a review May 2, 2023 19:03
@zoecarver
Copy link
Contributor Author

@swift-ci please test windows

@zoecarver
Copy link
Contributor Author

Shall we guard move only types on libc++ for the time being or try to fix the libstdc++ issue?

@zoecarver
Copy link
Contributor Author

@swift-ci please test

@zoecarver
Copy link
Contributor Author

@swift-ci please smoke test

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