-
Notifications
You must be signed in to change notification settings - Fork 10.4k
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: Small fixes for result builder inference #76749
Conversation
@swift-ci please smoke test Linux |
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.
LGTM! I second Hamish's comment about BraceHasReturnRequest
it would be good to do at least that here.
37c326a
to
66d4cd1
Compare
@swift-ci please smoke test Linux |
66d4cd1
to
02665c5
Compare
@swift-ci please smoke test Linux |
@swift-ci please test source compatibility release |
…look at getter, not storage
…nyFunctionRef` method
A callback enables the caller to control whether to abort the walk. This is handy for `BraceHasExplicitReturnStmtRequest`.
02665c5
to
4a750af
Compare
@swift-ci please smoke test macOS |
…nctionRef` Also rename it to `getExplicitReturnStmts` for clarity and have it take a `SmallVector` out parameter instead as a small optimization and to discourage use of this new method as an alternative to `AnyFunctionRef::bodyHasExplicitReturnStmt`.
4a750af
to
41adfec
Compare
@swift-ci please test source compatibility release |
@swift-ci please smoke test |
Fixes #76362, fixes #76746, fixes #76747.
These are technically source breaking, but the cases in which they break source are quite niche.