Skip to content

[SR-15038][Sema] Use locator to propagate information of array, set, dictonary element on check cast constraint #38884

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

LucianoPAlmeida
Copy link
Contributor

Only TMF_ApplyingFix is not enough to propagate information of (set, array, dictionary) element type (which should not record extraneous cast fixes) through unresolved constraint when the element is a type variable that can't still be solved. So to fix this changes the approach to track that via locator element.

Resolves SR-15038.

@LucianoPAlmeida LucianoPAlmeida requested a review from xedin August 14, 2021 18:05
@LucianoPAlmeida
Copy link
Contributor Author

@swift-ci Please smoke test

@LucianoPAlmeida
Copy link
Contributor Author

@swift-ci Please smoke test Linux Platform

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.

Makes sense, I don't think TMF_ApplyingFix was the right answer here anyway, that said, I think we can avoid having to use special locator.

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.

Look good, thank you! One more note - please update commit message to reflect updated logic and reference SR in there too for posterity. (No need to re-request review for that :))

@LucianoPAlmeida
Copy link
Contributor Author

One more note - please update commit message to reflect updated logic and reference SR in there too for posterity. (No need to re-request review for that :))

Sure, I'll adjust it. Thank you for the review =]

@xedin
Copy link
Contributor

xedin commented Aug 17, 2021

Thank you for the fix!

…ray, set, dictionary) generic elt in CheckedCast constraint and not record extraneous cast fixes for it
@LucianoPAlmeida
Copy link
Contributor Author

@swift-ci Please smoke test

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