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

Initial support for static fields in JcInterpreter #30

Merged
merged 3 commits into from
Jul 17, 2023
Merged

Conversation

Saloed
Copy link
Collaborator

@Saloed Saloed commented Jun 29, 2023

Initial support for static fields

  • Represent statics with UFieldLValue using unique concrete address for each JcClass.
  • Analyze static initializers on first access.

@Saloed Saloed force-pushed the saloed/statics branch 2 times, most recently from b4b4e72 to 0d23a17 Compare June 30, 2023 19:12
@CaelmBleidd
Copy link
Member

I didn't get, how it works with initialization time. To resolve static values in results we have to have some kind of initial memory for them, and it is different from the one we use for regular fields. How does resolve for static fields work?

@Saloed
Copy link
Collaborator Author

Saloed commented Jul 11, 2023

I didn't get, how it works with initialization time. To resolve static values in results we have to have some kind of initial memory for them, and it is different from the one we use for regular fields. How does resolve for static fields work?

We currently don't have a mode when statics are treated as input values. For now, we run the static initializer once on a first access as if we were analyzing the entrypoint in a fresh environment.

@dvvrd dvvrd merged commit ccc8d8e into main Jul 17, 2023
@sergeypospelov sergeypospelov mentioned this pull request Jul 17, 2023
57 tasks
@Saloed Saloed deleted the saloed/statics branch July 26, 2023 07:54
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.

4 participants