Skip to content

[5.7][IDE] Skip walking serialized non-visible extension decls #59029

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

bnbarham
Copy link
Contributor

@bnbarham bnbarham commented May 23, 2022

Cherry-picks 4a4dded (#59024) to fix a crash when building if index-while-building is enabled and a system module contains an internal extension.


fec7a0b skipped all non-visible
ValueDecls but missed ExtensionDecls, which have the same issue.
Make sure to skip these too.

Resolves rdar://91279771.

fec7a0b skipped all non-visible
`ValueDecls` but missed `ExtensionDecls`, which have the same issue.
Make sure to skip these too.

Resolves rdar://91279771.
@bnbarham bnbarham requested review from xymus and airspeedswift May 23, 2022 18:52
@bnbarham bnbarham requested a review from a team as a code owner May 23, 2022 18:52
@bnbarham bnbarham changed the title [IDE] Skip walking serialized non-visible extension decls [5.7][IDE] Skip walking serialized non-visible extension decls May 23, 2022
@bnbarham bnbarham added the r5.7 label May 23, 2022
@bnbarham
Copy link
Contributor Author

@swift-ci please test

Copy link
Contributor

@xymus xymus left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thanks!

@bnbarham bnbarham merged commit 9de27f6 into swiftlang:release/5.7 May 24, 2022
@bnbarham bnbarham deleted the cherry-index-skip-internal-extensions branch May 24, 2022 23:27
@AnthonyLatsis AnthonyLatsis added 🍒 release cherry pick Flag: Release branch cherry picks swift 5.7 labels Jan 9, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
🍒 release cherry pick Flag: Release branch cherry picks swift 5.7
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants