Skip to content

Conversation

@xedin
Copy link
Contributor

@xedin xedin commented Oct 30, 2025

…ession

The change continues what #85211 started.

Copy, Borrow, {Any, Optional}Try, VarargExpansion expressions have to perform the same transformation - resolve the type and use it to coerce their sub-expression. This change introduces a single method to do just that instead of copying the same code around.

…ession

Copy, Borrow, {Any, Optional}Try, VarargExpansion expressions
have to perform the same transformation - resolve the type and
use it to coerce their sub-expression. This change introduces a
single method to do just that instead of copying the same code
around.
@xedin xedin requested a review from hborla as a code owner October 30, 2025 15:55
Copy link
Contributor

@hamishknight hamishknight left a comment

Choose a reason for hiding this comment

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

Nice!

Copy link
Contributor

@slavapestov slavapestov left a comment

Choose a reason for hiding this comment

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

Beautiful!

@slavapestov
Copy link
Contributor

@swift-ci Please smoke test

@slavapestov
Copy link
Contributor

@swift-ci Please test source compatibility

@xedin
Copy link
Contributor Author

xedin commented Oct 31, 2025

The source compatibility failures are unrelated and reproducible on main as well.

@xedin xedin merged commit 94949f3 into swiftlang:main Oct 31, 2025
3 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