Skip to content

Fix GenericSpecializer for addressable parameters. #80577

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 5 commits into from
Apr 8, 2025

Conversation

atrick
Copy link
Contributor

@atrick atrick commented Apr 7, 2025

Addressable parameters must remain indirect.

Incidentally also fixes an obvious latent bug in which all specialization was disabled if any metatypes could not be specialized.

Fixes rdar://145687827 (Crash of inline-stored Span properties with optimizations)

@atrick atrick marked this pull request as draft April 7, 2025 08:00
@atrick
Copy link
Contributor Author

atrick commented Apr 7, 2025

@swift-ci test

atrick added 4 commits April 7, 2025 14:42
Addressable parameters must remain indirect.

Incidentally also fixes an obvious latent bug in which all specialization was
disabled if any metatypes could not be specialized.

Fixes rdar://145687827 (Crash of inline-stored Span properties with optimizations)
@atrick atrick force-pushed the addressable-escape branch from 0a02f71 to ef7c053 Compare April 7, 2025 21:42
@atrick atrick marked this pull request as ready for review April 7, 2025 21:42
@atrick
Copy link
Contributor Author

atrick commented Apr 7, 2025

@swift-ci smoke test

@atrick atrick enabled auto-merge April 7, 2025 22:32
@atrick atrick force-pushed the addressable-escape branch from ef7c053 to 6073d59 Compare April 8, 2025 01:19
@atrick
Copy link
Contributor Author

atrick commented Apr 8, 2025

@swift-ci test

@atrick atrick force-pushed the addressable-escape branch from 6073d59 to 923de73 Compare April 8, 2025 03:22
@atrick
Copy link
Contributor Author

atrick commented Apr 8, 2025

@swift-ci test

@atrick atrick force-pushed the addressable-escape branch from 923de73 to 5914369 Compare April 8, 2025 07:00
@atrick
Copy link
Contributor Author

atrick commented Apr 8, 2025

@swift-ci test

@atrick
Copy link
Contributor Author

atrick commented Apr 8, 2025

@swift-ci smoke test macOS

@atrick atrick merged commit a1fc2bc into swiftlang:main Apr 8, 2025
4 of 5 checks passed
@atrick atrick deleted the addressable-escape branch April 8, 2025 20:40
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