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

Easy migration to CompileStatic annotation #568

Merged
merged 1 commit into from
Jun 24, 2022

Conversation

rougsig
Copy link
Collaborator

@rougsig rougsig commented Jun 23, 2022

If groovy compiles without CompileStatic - code will be compiled in dynamic way,
e.g. All type checks will be in runtime. Erased types in bytecode.
With CompileStatic - compiler will save all typed in bytecode.
If all types saved, we can do static analysis of bytecode with java 8 capability.
More details can be found here: #565 (comment)

@rougsig rougsig force-pushed the compile-static branch 4 times, most recently from 45ef29f to e3d43a6 Compare June 23, 2022 20:45
If groovy compiles without CompileStatic - code will be compiled in dynamic way,
e.g. All type checks will be in runtime. Erased types in bytecode.
With CompileStatic - compiler will save all typed in bytecode.
If all types saved, we can do static analysis of bytecode with java 8 capability.
More details can be found here: google#565 (comment)
@ejona86 ejona86 requested a review from YifeiZhuang June 23, 2022 21:24
@ejona86 ejona86 merged commit 20d4d51 into google:master Jun 24, 2022
@rougsig rougsig deleted the compile-static branch September 29, 2022 20:02
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