Skip to content

[ConstraintSolver] NFC: Add option to control early termination of shrinking phase #11085

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 1 commit into from
Jul 21, 2017

Conversation

xedin
Copy link
Contributor

@xedin xedin commented Jul 20, 2017

Currently we have a number of unsolved disjunctions hard-coded to 5,
which breaks some existing code by terminating shrinking too early.
This patch makes it a command-line option so users have control over
what that threshold can be.

Resolves: rdar://problem/33433595

…rinking phase

Currently we have a number of unsolved disjunctions hard-coded to 5,
which breaks some existing code by terminating shrinking too early.
This patch makes it a command-line option so users have control over
what that threshold can be.

Resolves: rdar://problem/33433595
@xedin xedin requested a review from rudkx July 20, 2017 19:48
@xedin
Copy link
Contributor Author

xedin commented Jul 20, 2017

@swift-ci please smoke test

@rudkx
Copy link
Contributor

rudkx commented Jul 20, 2017

@swift-ci Please test source compatibility

Copy link
Contributor

@rudkx rudkx left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM. Out of an abundance of caution I started the source compatibility suite as well.

@xedin xedin merged commit c45e85f into swiftlang:master Jul 21, 2017
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