-
Notifications
You must be signed in to change notification settings - Fork 10.5k
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
Support std::unique_ptr
.
#65577
Conversation
@swift-ci please test |
import StdUniquePtr | ||
#if os(Linux) | ||
import CxxStdlib | ||
// FIXME: import CxxStdlib.string once libstdc++ is split into submodules. |
There was a problem hiding this comment.
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?
There was a problem hiding this comment.
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.
@swift-ci please test |
@swift-ci please test windows |
Shall we guard move only types on libc++ for the time being or try to fix the libstdc++ issue? |
@swift-ci please test |
@swift-ci please smoke test |
Currently the synthesized members consume but that's OK for now. At least it's correct.