Skip to content

[NameLookup] Opaque Type Collector should skip existential types #63221

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

Conversation

angela-laar
Copy link
Contributor

@angela-laar angela-laar commented Jan 25, 2023

An error in the opaque type collector is causing basic code using existential 'any' to crash with the 'ImplicitSome' feature.

i.e.

  protocol P { }

  var x: any P

Fixes rdar://102698119

@angela-laar angela-laar removed the request for review from slavapestov January 25, 2023 21:02
@angela-laar
Copy link
Contributor Author

@swift-ci please smoke test

1 similar comment
@angela-laar
Copy link
Contributor Author

@swift-ci please smoke test

@angela-laar angela-laar force-pushed the verifier-failure-implicit-some branch from e750c0a to 959d833 Compare January 27, 2023 01:15
@angela-laar angela-laar removed the request for review from AnthonyLatsis January 27, 2023 01:19
@angela-laar angela-laar force-pushed the verifier-failure-implicit-some branch from 959d833 to 286772a Compare January 27, 2023 01:36
@angela-laar
Copy link
Contributor Author

@swift-ci please smoke test

1 similar comment
@angela-laar
Copy link
Contributor Author

@swift-ci please smoke test

@xedin xedin removed their request for review January 30, 2023 18:24
@angela-laar angela-laar force-pushed the verifier-failure-implicit-some branch from 7ce4191 to e0c8a61 Compare February 1, 2023 22:42
@angela-laar
Copy link
Contributor Author

@swift-ci please smoke test

@angela-laar angela-laar force-pushed the verifier-failure-implicit-some branch from e0c8a61 to a24cf19 Compare February 2, 2023 20:56
@angela-laar
Copy link
Contributor Author

@swift-ci please smoke test

1 similar comment
@angela-laar
Copy link
Contributor Author

@swift-ci please smoke test

@angela-laar
Copy link
Contributor Author

@swift-ci please smoke test macOS

@angela-laar
Copy link
Contributor Author

@swift-ci please smoke test linux

@angela-laar angela-laar force-pushed the verifier-failure-implicit-some branch from a24cf19 to cba7389 Compare February 4, 2023 02:13
@angela-laar
Copy link
Contributor Author

@swift-ci please smoke test

@angela-laar angela-laar merged commit 5427191 into swiftlang:main Feb 4, 2023
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