Skip to content

[4.2] [Serialization] Mark decls that can never be cross-referenced #17302

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

Conversation

jrose-apple
Copy link
Contributor

Cherry-pick of #17223 to the 4.2 branch. Reviewed by @DougGregor.

SR-4851, plus possibly other SourceKit crashes in ModuleFile::readConformance.

…ang#17223)

This allows us to filter them out in cases that would otherwise be
ambiguous. The particular prompting situation looks a lot like the
test case: a protocol, plus a forward-declared class with the same
name. (Normally we ignore forward-declared classes, but SourceKit's
module interface generation feature makes dummy ClassDecls for them
instead.)

https://bugs.swift.org/browse/SR-4851
(cherry picked from commit 05d0a2d)
@jrose-apple
Copy link
Contributor Author

@swift-ci Please test

@jrose-apple jrose-apple merged commit 76e5166 into swiftlang:swift-4.2-branch Jun 18, 2018
@jrose-apple jrose-apple deleted the 4.2-overly-forward branch June 18, 2018 23:25
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.

1 participant