Skip to content

[Const extract] Extract static function calls #76251

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
Oct 10, 2024

Conversation

jPaolantonio
Copy link
Contributor

Adds extraction for static function calls.

This fixes a bug in which static functions could be incorrectly extracted as enumerated cases.

Resolves rdar://134265950

@jPaolantonio
Copy link
Contributor Author

@swift-ci test

@jPaolantonio jPaolantonio force-pushed the eng/statics-const-extract branch from 7e84c84 to c2ca7b0 Compare September 4, 2024 20:15
@jPaolantonio
Copy link
Contributor Author

@swift-ci test

@jPaolantonio
Copy link
Contributor Author

@swift-ci test Windows platform

@jPaolantonio jPaolantonio force-pushed the eng/statics-const-extract branch from 426b0e0 to 3a5d21e Compare September 5, 2024 18:42
@jPaolantonio
Copy link
Contributor Author

@swift-ci test

@jPaolantonio jPaolantonio force-pushed the eng/statics-const-extract branch from 81a6dbe to 33e0780 Compare September 26, 2024 19:24
@jPaolantonio jPaolantonio force-pushed the eng/statics-const-extract branch from 33e0780 to d6ecc2a Compare September 30, 2024 18:28
@jPaolantonio
Copy link
Contributor Author

@swift-ci test

Copy link
Contributor

@artemcm artemcm left a comment

Choose a reason for hiding this comment

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

Thanks!

@jPaolantonio jPaolantonio merged commit 5e1de49 into swiftlang:main Oct 10, 2024
5 checks passed
@jPaolantonio jPaolantonio deleted the eng/statics-const-extract branch October 11, 2024 02:20
j-hui pushed a commit to j-hui/swift that referenced this pull request Oct 29, 2024
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.

3 participants