Skip to content

[JVM] Remove unnecessary GETSTATIC in @JvmStatic function #5458

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.

Sign up for GitHub

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

Jump to bottom
Closed
scaventz wants to merge 1 commit into from
Closed

[JVM] Remove unnecessary GETSTATIC in @JvmStatic function #5458

scaventz wants to merge 1 commit into from

Conversation

@scaventz
Copy link
Contributor

@scaventz scaventz commented Jun 3, 2025

Remove unnecessary GETSTATIC when @JvmStatic functions call each other

^KT-77438 fixed

@scaventz
[JVM] Remove unnecessary GETSTATIC in @JvmStatic function
7f2ad6b 
Remove unnecessary GETSTATIC when @JvmStatic functions call each other

^KT-77438 fixed
@scaventz scaventz marked this pull request as ready for review June 4, 2025 01:45
@scaventz scaventz requested review from a team as code owners June 4, 2025 01:45
ilmirus
ilmirus approved these changes Jun 12, 2025
View reviewed changes
Copy link
Member

@ilmirus ilmirus left a comment

Choose a reason for hiding this comment

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

Thank you! Merged to master manually.

@ilmirus ilmirus closed this Jun 12, 2025
@scaventz scaventz deleted the KT-77438-uneccessary-getstatic branch June 12, 2025 06:15
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@ilmirus ilmirus ilmirus approved these changes

@klikh klikh Awaiting requested review from klikh klikh was automatically assigned from JetBrains/kotlin

@yanex yanex Awaiting requested review from yanex yanex was automatically assigned from JetBrains/kotlin-analysis-api

@ilgonmic ilgonmic Awaiting requested review from ilgonmic ilgonmic was automatically assigned from JetBrains/kotlin-wasm

@koperagen koperagen Awaiting requested review from koperagen koperagen was automatically assigned from JetBrains/kotlin-for-data-science

@udalov udalov Awaiting requested review from udalov udalov is a code owner automatically assigned from JetBrains/kotlin-jvm

@dzharkov dzharkov Awaiting requested review from dzharkov dzharkov is a code owner automatically assigned from JetBrains/kotlin-frontend

Assignees

No one assigned

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

2 participants

@scaventz @ilmirus