Skip to content
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

[Constraint system] More or less rewrite the constraint propagation p… #8725

Merged
merged 1 commit into from Apr 12, 2017
Merged

[Constraint system] More or less rewrite the constraint propagation p… #8725

merged 1 commit into from Apr 12, 2017

Conversation

rudkx
Copy link
Member

@rudkx rudkx commented Apr 12, 2017

…ass.

This is still disabled by default, and as with the previous
implementation does not pass all tests if enabled.

Rather than looking for applicable function constraints and then looking
at the bind overload disjunctions associated with those, instead
directly collect all bind overload disjunctions and revise the enabled
elements of those disjunctions by attempting to bind them simultaneously
with the elements of other disjunctions that are linked to this one
through applicable function constraints.

…ass.

This is still disabled by default, and as with the previous
implementation does not pass all tests if enabled.

Rather than looking for applicable function constraints and then looking
at the bind overload disjunctions associated with those, instead
directly collect all bind overload disjunctions and revise the enabled
elements of those disjunctions by attempting to bind them simultaneously
with the elements of other disjunctions that are linked to this one
through applicable function constraints.
@rudkx
Copy link
Member Author

rudkx commented Apr 12, 2017

@swift-ci Please smoke test

@rudkx rudkx merged commit 024338a into apple:master Apr 12, 2017
@rudkx rudkx deleted the propagate-constraints branch April 12, 2017 21:46
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.

None yet

1 participant