Skip to content
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

Serialization: Skip invalid @available attributes during lazy serialization #80215

Merged

Conversation

tshortli
Copy link
Contributor

When performing lazy module serialization, we may be making the first attempt to turn an AvailableAttr into a SemanticAvailableAttr. If it turns out the attribute is invalid at that point, we need to skip it instead of assuming that the attribute will always be valid there.

Resolves rdar://147539902.

…lization.

When performing lazy module serialization, we may be making the first attempt
to turn an `AvailableAttr` into a `SemanticAvailableAttr`. If it turns out the
attribute is invalid at that point, we need to skip it instead of assuming
that the attribute will always be valid there.

Resolves rdar://147539902.
@tshortli tshortli requested a review from xymus as a code owner March 21, 2025 22:46
@tshortli
Copy link
Contributor Author

@swift-ci please smoke test

@tshortli tshortli enabled auto-merge March 21, 2025 22:56
@tshortli
Copy link
Contributor Author

@swift-ci please smoke test macOS

1 similar comment
@tshortli
Copy link
Contributor Author

@swift-ci please smoke test macOS

@tshortli tshortli merged commit dd9431a into swiftlang:main Mar 23, 2025
3 checks passed
@tshortli tshortli deleted the lazy-serialization-available-attribute-crash branch March 23, 2025 07:08
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