Skip to content

[5.5] Collect Fingerprints for Nested Decls #39451

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
merged 1 commit into from
Sep 25, 2021

Conversation

CodaFi
Copy link
Contributor

@CodaFi CodaFi commented Sep 24, 2021

Cherry pick #39450


Serialization was reporting that it did not have fingerprints for these declarations. When combined with the driver's new type body fingerprint work for extensions, this effectively hid changes to the bodies of these nested types from the driver's incremental build infrastructure.

Make sure we recur into all of the iterable decl contexts when serializing decls to collect as many fingerprints as we can.

rdar://83469936

Serialization was reporting that it did not have fingerprints for these declarations. When combined with the driver's new type body fingerprint work for extensions, this effectively hid changes to the bodies of these nested types from the driver's incremental build infrastructure.

Make sure we recur into all of the iterable decl contexts when serializing decls to collect as many fingerprints as we can.

rdar://83469936
@CodaFi CodaFi added the r5.5 label Sep 24, 2021
@CodaFi CodaFi requested a review from davidungar September 24, 2021 21:41
@CodaFi CodaFi requested a review from a team as a code owner September 24, 2021 21:41
@CodaFi
Copy link
Contributor Author

CodaFi commented Sep 24, 2021

@swift-ci test

Copy link
Contributor

@davidungar davidungar left a comment

Choose a reason for hiding this comment

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

LGTM! Great to get this in to 5.5

@swift-ci
Copy link
Contributor

Build failed
Swift Test OS X Platform
Git Sha - 0a0f838

@CodaFi
Copy link
Contributor Author

CodaFi commented Sep 25, 2021

LLDB test failure is unrelated

@swift-ci test macOS

@CodaFi CodaFi merged commit dcc5591 into swiftlang:release/5.5 Sep 25, 2021
@CodaFi CodaFi deleted the ear-nest-hemmingway branch September 25, 2021 18:39
@AnthonyLatsis AnthonyLatsis added 🍒 release cherry pick Flag: Release branch cherry picks swift 5.5 labels Jan 8, 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.5
Projects
None yet
Development

Successfully merging this pull request may close these issues.

5 participants