Thanks to visit codestin.com
Credit goes to github.com

Skip to content

[stdlib] Allow metatype comparisons to work with outdated compilers #80859

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 1 commit into from
May 2, 2025

Conversation

lorentey
Copy link
Member

Add a new language feature to avoid the stdlib’s swiftinterface becoming unintelligible to older compilers due to the generalization of Builtin.is_same_metatype.

Restore the original version of the == operator so that metatypes can continue to be compared even if the generalized version ends up getting omitted.

rdar://149396721

@lorentey lorentey requested review from a team, DougGregor and hborla as code owners April 16, 2025 18:18
@lorentey
Copy link
Member Author

I haven't even tried building this locally, but let's roll the dice

@swift-ci test

@lorentey lorentey requested review from glessard, Azoy and tshortli and removed request for DougGregor April 16, 2025 18:20
Add a new language feature to avoid the stdlib’s swiftinterface becoming unintelligible to outdated compiler builds due to the generalization of Builtin.is_same_metatype.

rdar://149396721
@lorentey lorentey force-pushed the is_same_metatype_condfail branch from ad5a81f to 0c406b8 Compare April 16, 2025 19:46
@lorentey
Copy link
Member Author

Well that was silly; apologies for my brain hiccup. The amended version should be closer to what we need.

@lorentey
Copy link
Member Author

@swift-ci test

@lorentey lorentey changed the title [stdlib] Reinstate original versions of metatype comparisons [stdlib] Allow metatype comparisons to work with outdated compilers Apr 16, 2025
@lorentey
Copy link
Member Author

@swift-ci smoke test linux

@lorentey lorentey enabled auto-merge April 22, 2025 17:00
@lorentey
Copy link
Member Author

lorentey commented May 2, 2025

@swift-ci test

@lorentey lorentey merged commit 74efc7d into swiftlang:main May 2, 2025
4 of 5 checks passed
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.

3 participants