Skip to content

Package CMO: Enable serializing decls imported with @_spiOnly or package import. #79035

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
Jan 30, 2025

Conversation

elsh
Copy link
Contributor

@elsh elsh commented Jan 30, 2025

Starting in Swift 6.0, package access level and @_spiOnly attribute have been increasingly used in import statements. However, existing import filtering prevented serialization of package APIs that included such decls, leading to a significant drop in overall serialization. This PR removes these restrictive filters, and also allows decls from SDK or system modules to be included in serialization.

Resolves rdar://130788606

@elsh
Copy link
Contributor Author

elsh commented Jan 30, 2025

@swift-ci test

…ackage import`.

Starting in Swift 6.0, `package` access level and `@_spiOnly` attribute have been increasingly used in import statements.
However, existing import filtering prevented serialization of package APIs that included such decls, leading to a
significant drop in overall serialization. This PR removes these restrictive filters, and allows decls from SDK or system
modules to be included in serialization.

rdar://130788606
@elsh elsh force-pushed the elsh/pcmo-imports branch from 944946c to 67594f1 Compare January 30, 2025 07:25
@elsh elsh changed the title Package CMO: Allow serializing decls imported with @_spiOnly or package import. Package CMO: Enable serializing decls imported with @_spiOnly or package import. Jan 30, 2025
@elsh
Copy link
Contributor Author

elsh commented Jan 30, 2025

@swift-ci smoke test

Copy link
Contributor

@eeckstein eeckstein left a comment

Choose a reason for hiding this comment

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

lgtm

@elsh elsh merged commit 2de333c into main Jan 30, 2025
3 checks passed
@elsh elsh deleted the elsh/pcmo-imports branch January 30, 2025 21:15
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.

2 participants