Skip to content

Add support for opaque result types in structural positions #38392

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
Aug 4, 2021

Conversation

willtunnels
Copy link

Currently, we only support opaque result types as top level level types, e.g. some P is allowed but not (some P)?. This branch adds support for opaque result types in structural type positions.

@willtunnels willtunnels requested a review from slavapestov July 14, 2021 19:47
@willtunnels
Copy link
Author

This is not ready to merge yet, but considering its size it might be worth doing some code review now.

@willtunnels willtunnels marked this pull request as draft July 15, 2021 15:11
@willtunnels willtunnels force-pushed the structural-some-types branch from fc33fce to bced7d6 Compare July 16, 2021 16:37
@willtunnels willtunnels requested a review from xedin July 16, 2021 19:49
@willtunnels willtunnels force-pushed the structural-some-types branch from 45af428 to 7fabab8 Compare July 21, 2021 19:17
Copy link
Contributor

@xedin xedin left a comment

Choose a reason for hiding this comment

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

I have looked at the type-checker/solver related changes and left a couple of comments inline, overall everything makes sense to me.

@willtunnels willtunnels force-pushed the structural-some-types branch from abe5600 to c77d8ba Compare August 2, 2021 22:56
@willtunnels willtunnels requested a review from hborla August 2, 2021 22:57
Copy link
Member

@hborla hborla left a comment

Choose a reason for hiding this comment

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

Constraint system changes look good!

@willtunnels
Copy link
Author

@swift-ci please smoke test

@willtunnels
Copy link
Author

@swift-ci please smoke test linux

@willtunnels willtunnels marked this pull request as ready for review August 4, 2021 00:29
@willtunnels willtunnels merged commit 319b3e6 into swiftlang:main Aug 4, 2021
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.

4 participants