-
Notifications
You must be signed in to change notification settings - Fork 10.5k
[ASTPrinter/CodeCompletion] Stop printing base type when possible #28165
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
38fc71e
to
9566c50
Compare
@swift-ci Please smoke test |
1 similar comment
@swift-ci Please smoke 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.
For code-completion this LGTM. Can this impact swiftinterface generation? If so, it would be good to have a test case.
9566c50
to
8366779
Compare
Does this fix SR-9868 as well? |
8366779
to
318bff6
Compare
318bff6
to
6da2ca7
Compare
rdar://problem/47871647
@swift-ci Please smoke test |
Thanks for the heads up! It does. Added a test case for it |
rdar://problem/57033931
rdar://problem/47871647
https://bugs.swift.org/browse/SR-9868