Skip to content

Package CMO: Skip deserialization error checks for same-module decls. #79157

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
Feb 6, 2025

Conversation

elsh
Copy link
Contributor

@elsh elsh commented Feb 5, 2025

IterableDeclContext::checkDeserializeMemberErrorInPackage recursively checks if
decls and their member decls are deserialized correctly into another module.
This PR adds a check to make sure the inspected decls are from another module,
and provides an opt-in flag to fail fast on deserialization failure if found.

rdar://143830240

@elsh
Copy link
Contributor Author

elsh commented Feb 5, 2025

@swift-ci smoke test

IterableDeclContext::checkDeserializeMemberErrorInPackage recursively checks if
decls and their member decls are deserialized correctly into another module.
This PR adds a check to make sure the inspected decls are from another module,
and provides an opt-in flag to fail fast on deserialization failure if found.

rdar://143830240
@elsh elsh force-pushed the elsh/pcmo-bypass-res-check-mods branch from 3ef5b01 to f314d45 Compare February 5, 2025 22:27
@elsh elsh force-pushed the elsh/pcmo-bypass-res-check-mods branch from f314d45 to ee75183 Compare February 5, 2025 22:29
@elsh
Copy link
Contributor Author

elsh commented Feb 5, 2025

@swift-ci smoke test

@elsh elsh merged commit 098f2a9 into main Feb 6, 2025
3 checks passed
@elsh elsh deleted the elsh/pcmo-bypass-res-check-mods branch February 6, 2025 23:38
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