Skip to content

Conversation

@hamishknight
Copy link
Contributor

Resolving only a structural type meant we weren't checking generic constraints, allowing invalid extensions to get past the type-checker. Change to resolve an interface type.

@hamishknight
Copy link
Contributor Author

@swift-ci please test

@hamishknight
Copy link
Contributor Author

@swift-ci please test source compatibility

@hamishknight
Copy link
Contributor Author

Will need to rebase on #83824 when it lands since this fixes one of its crashers

Resolving only a structural type meant we weren't checking generic
constraints, allowing invalid extensions to get past the type-checker.
Change to resolve an interface type.
@hamishknight
Copy link
Contributor Author

@swift-ci please smoke test

@hamishknight hamishknight enabled auto-merge August 24, 2025 10:50
@hamishknight hamishknight merged commit db8b848 into swiftlang:main Aug 24, 2025
3 checks passed
@hamishknight hamishknight deleted the carousel branch August 24, 2025 22:16
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