Skip to content

[CSSimplify] Delay inout type to pointer conversion until inout is sufficiently resolved #76439

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

Conversation

xedin
Copy link
Contributor

@xedin xedin commented Sep 12, 2024

If left-hand side of any conversion constraint is inout type
and right is a pointer (or optional thereof), delay simplification
until inout is at least partially structurally resolved (cannot
be a type variable or dependent member) because eager simplification
won't record all of the possible conversions.

Allow inferring type of inout from a pointer type
(or optional thereof) but delay the binding set because
it might not be complete and object type of inout could
also be an Array or C-style pointer type.

…s sufficiently resolved

If left-hand side of any conversion constraint is `inout` type
and right is a pointer (or optional thereof), delay simplification
until `inout` is at least partially structurally resolved (cannot
be a type variable or dependent member) because eager simplification
won't record all of the possible conversions.
@xedin
Copy link
Contributor Author

xedin commented Sep 12, 2024

@swift-ci please test

@xedin
Copy link
Contributor Author

xedin commented Sep 12, 2024

@swift-ci please test Windows platform

Allow inferring type of `inout` from a pointer type
(or optional thereof) but delay the binding set because
it might not be complete and object type of `inout` could
also be an Array or C-style pointer type.
@xedin xedin force-pushed the delay-implicit-pointer-conversions-for-unknown-inout branch from 05fbc70 to 279ef7d Compare September 13, 2024 12:08
@xedin
Copy link
Contributor Author

xedin commented Sep 13, 2024

@swift-ci please test

@xedin xedin merged commit 16244b3 into swiftlang:main Sep 13, 2024
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.

2 participants