Skip to content

[CSSimplify] Fix invalid diagnostics emitted for a pack reference outside of pack expansion #67489

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 3 commits into from
Jul 25, 2023

Conversation

xedin
Copy link
Contributor

@xedin xedin commented Jul 24, 2023

.rawValue and assignment diagnostics didn't handle holes properly which led to
the solver recording incorrect fixes.

Resolves: rdar://110847476

xedin added 3 commits July 24, 2023 11:15
…resentable conformance

Placeholder types just like type variables are set up to return
a conformance ref which, in this case, would mean that we'd produce
an invalid diagnostic.
…or destination are a hole

If either source or destination type is a hole conversion should
just be ignored because there is no r-value or other mismatch that
could be diagnosed in this case.
@xedin xedin requested a review from hborla as a code owner July 24, 2023 20:28
@xedin
Copy link
Contributor Author

xedin commented Jul 24, 2023

@swift-ci please test

@xedin xedin merged commit cf5531b into swiftlang:main Jul 25, 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.

1 participant