Skip to content

[copy-operator] Add parsing support for copy operator. #1669

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 7 commits into from
May 19, 2023

Conversation

gottesmm
Copy link
Contributor

I followed the same model that we used for consume, namely we need an identifier
to copy. This ensures that if we have a function called copy or a variable
called copy, we do not break source stability.

rdar://101862423

@gottesmm gottesmm requested a review from jckarter May 16, 2023 02:54
@gottesmm gottesmm requested a review from ahoppen as a code owner May 16, 2023 02:54
@gottesmm
Copy link
Contributor Author

Swift PR: swiftlang/swift#65935

@gottesmm
Copy link
Contributor Author

swiftlang/swift#65935

@swift-ci test

gottesmm added 6 commits May 17, 2023 14:11
I followed the same model that we used for consume, namely we need an identifier
to copy. This ensures that if we have a function called copy or a variable
called copy, we do not break source stability.

rdar://101862423
… of move operator and add consume tests.

rdar://109413392
…ndle consume used as a name in case stmt as pattern match.

rdar://109413278
@gottesmm
Copy link
Contributor Author

@ahoppen I decided to also fix consume in this commit as well. I did the same thing as we did with copy/added tests/etc.

@gottesmm
Copy link
Contributor Author

swiftlang/swift#65935

@swift-ci test

@gottesmm
Copy link
Contributor Author

swiftlang/swift#65935

@swift-ci test

@gottesmm
Copy link
Contributor Author

@swift-ci test windows platform

@gottesmm gottesmm merged commit 6645096 into swiftlang:main May 19, 2023
@gottesmm gottesmm deleted the rdar101862423 branch May 19, 2023 21:50
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants