-
Notifications
You must be signed in to change notification settings - Fork 10.5k
Add support for _resultDependsOn #70263
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
Conversation
8d05b50
to
5b0a991
Compare
Please test with following pull request: @swift-ci Please test |
Please test with following pull request: @swift-ci Please test linux platform |
@@ -1,4 +1,4 @@ | |||
// RUN: %target-typecheck-verify-swift -enable-builtin-module -enable-experimental-feature NonEscapableTypes | |||
// RUN: %target-typecheck-verify-swift -enable-builtin-module -enable-experimental-feature NonEscapableTypes -disable-experimental-parser-round-trip |
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 shouldn’t disable the new parser in these test cases. The new parser should support parsing of this source file as well.
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.
rdar://118125715 needs to be fixed before I can enable the new parser for this test.
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-syntax PR has a test for cases that are supported with the new parser - swiftlang/swift-syntax#2386
These attributes are used to establish lifetime dependence between argument and the result. Add them under NonEscapableTypes experimental feature
5b0a991
to
f6afc00
Compare
Merge conflict in ModuleFormat.h Testing again |
Please test with following pull request: @swift-ci Please smoke test |
1 similar comment
Please test with following pull request: @swift-ci Please smoke test |
No description provided.