-
Notifications
You must be signed in to change notification settings - Fork 10.3k
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
Sema: Generalize heuristic in favoredOverDisjunction() #76318
Sema: Generalize heuristic in favoredOverDisjunction() #76318
Conversation
return hasConversions(pair->second); | ||
} else if (auto eltTy = ConstraintSystem::isSetType(structTy)) { | ||
return hasConversions(*eltTy); | ||
} |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I'm a bit worried that for Array, Dictionary, Set if we are too aggressive in binding we might lose viable bindings to bridgeable types.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
We'll see what happens :) Validation tests did pass...
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
🤞
e8b1876
to
73aa029
Compare
@swift-ci Please smoke test |
return hasConversions(pair->second); | ||
} else if (auto eltTy = ConstraintSystem::isSetType(structTy)) { | ||
return hasConversions(*eltTy); | ||
} |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
🤞
@swift-ci Please test source compatibility |
This fixes a too complex test case that was originally added in 2507a31.