Skip to content

Make SwiftValue == support unconditional #73552

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 2 commits into from
May 11, 2024
Merged

Conversation

tbkka
Copy link
Contributor

@tbkka tbkka commented May 9, 2024

PR #71620 made this behavior conditional as a way to help provide binary compatibility for legacy software that might be relying on the old behavior.

So far, it appears the only such problems arose from the SwiftObject behavior changes, not from SwiftValue behavior. So let's optimistically back this out and make the new behavior unconditional.

Resolves rdar://127839540

PR swiftlang#71620 made this behavior conditional as a way to help
provide binary compatibility for legacy software that might
be relying on the old behavior.

So far, it appears the only such problems arose from the
SwiftObject behavior changes, not from SwiftValue behavior.
So let's optimistically back this out and make the new behavior
unconditional.

Resolves rdar://127839540
@tbkka tbkka requested review from mikeash and al45tair as code owners May 9, 2024 21:45
@tbkka
Copy link
Contributor Author

tbkka commented May 9, 2024

@swift-ci Please test

Copy link
Contributor

@mikeash mikeash left a comment

Choose a reason for hiding this comment

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

I salute your bravery.

@tbkka tbkka requested a review from a team as a code owner May 11, 2024 00:11
@tbkka
Copy link
Contributor Author

tbkka commented May 11, 2024

@swift-ci Please test

@tbkka tbkka enabled auto-merge May 11, 2024 00:11
@tbkka tbkka merged commit 54a2007 into swiftlang:main May 11, 2024
4 of 5 checks passed
@tbkka
Copy link
Contributor Author

tbkka commented May 13, 2024

@swift-ci Please test macOS platform

tbkka added a commit to tbkka/swift that referenced this pull request May 13, 2024
Make SwiftValue == support unconditional
@tbkka tbkka deleted the tbkka-127839540 branch August 1, 2024 16:38
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