Skip to content

Jimple to JavaObjects to change minimaltestsuite #1162

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
Draft
wants to merge 25 commits into
base: develop
Choose a base branch
from
Draft

Jimple to JavaObjects to change minimaltestsuite #1162

wants to merge 25 commits into from

Conversation

@sahilagichani14
Copy link
Collaborator

@sahilagichani14 sahilagichani14 commented Jan 20, 2025

No description provided.

@sahilagichani14 sahilagichani14 linked an issue Jan 20, 2025 that may be closed by this pull request
[Feature]: JimpleJavaObjectPrinter #1056
Open
3 tasks
@sahilagichani14 sahilagichani14 marked this pull request as draft January 20, 2025 05:13
swissiety
swissiety requested changes Jan 20, 2025
View reviewed changes
Copy link
Collaborator

@swissiety swissiety left a comment

Choose a reason for hiding this comment

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

Just to be clear we are on the same page: the goal is to transform a StmtGraph to a String containing Java Sourcecode. And if we run that Javacode we should get the equivalent StmtGraph (again)

@sahilagichani14
Copy link
Collaborator Author

sahilagichani14 commented Jan 20, 2025

Just to be clear we are on the same page: the goal is to transform a StmtGraph to a String containing Java Sourcecode. And if we run that Javacode we should get the equivalent StmtGraph (again)

So the starting point is StmtGraph and not Jimple String?

@swissiety
Copy link
Collaborator

swissiety commented Jan 20, 2025

yes and please make use/implement a StmtVisitor to print the details of the respective Stmts. And in those methods of the StmtVisitor you could make use of that JavaPoet library if it helps you to keep track of the different Java variables of the generated source code. Maybe a Map<Value, JavaVariableNameString> of already generated Jimple objects would suffice as well, so you could generate each Stmt and initialize its Operands or reuse the already created ones.

sahilagichani14 and others added 17 commits February 7, 2025 10:52
@sahilagichani14
StmtValueVisitor, JavaCodeClassSpec, JavaCodeMethodSpec
4c2da10
@sahilagichani14
Merge branch 'refs/heads/develop' into 1056-feature-jimplejavaobjectp…
eabcca1 
…rinter
@sahilagichani14
Merge branch 'refs/heads/develop' into 1056-feature-jimplejavaobjectp…
b6aef49 
…rinter
@sahilagichani14
added valGenStr, valVarName, stmtGenStr, stmtVarName maps and added v…
cb535c6 
…ariable referencing
@sahilagichani14
build stmtgraph objects
4cde925
@sahilagichani14
add another test case from minimaltestsuite java6
fe5218f
@sahilagichani14
Merge branch 'refs/heads/develop' into 1056-feature-jimplejavaobjectp…
0fef9a8 
…rinter
@sahilagichani14
return void implement
a00410c
@sahilagichani14
refactor and moved tc to bytecode frontend as cyclic ref
a61a12c
@sahilagichani14
Merge branch 'develop' into 1056-feature-jimplejavaobjectprinter
6001270
@sahilagichani14
add logger for stmt & value visitor, implement InMemoryClassLoader & …
12f35c3 
…exec javaCodeObjects
@sahilagichani14
added license headers
64d564a
@sahilagichani14
Merge branch 'develop' into 1056-feature-jimplejavaobjectprinter
28208b1
@sahilagichani14
Merge branch 'develop' into 1056-feature-jimplejavaobjectprinter
3bdf3b4
@sahilagichani14
Merge branch 'develop' into 1056-feature-jimplejavaobjectprinter
0dd69f9
@sahilagichani14
moved stmtgraph srccodebuilder & runsrccodestr functionality to java.…
7e7bb1c 
…core
@sahilagichani14
fixed tc2
c3a8088
@sahilagichani14 sahilagichani14 marked this pull request as ready for review March 15, 2025 23:01
@sahilagichani14 sahilagichani14 marked this pull request as draft March 17, 2025 09:39
sahilagichani14
sahilagichani14 commented Mar 17, 2025
View reviewed changes
sootup.java.core/src/main/java/sootup/java/core/buildsrccode/JavaSrcCodeBuilder.java Outdated
List<Stmt> bodyStmts = body.getStmts();
if (!bodyStmts.isEmpty()) {
Iterator<Stmt> stmtIterator = bodyStmts.iterator();
// TODO: how to add edges or blocks when jumping statements?
Copy link
Collaborator Author

@sahilagichani14 sahilagichani14 Mar 17, 2025

Choose a reason for hiding this comment

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

@swissiety can we have a call to discuss this part?

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@stschott stschott Awaiting requested review from stschott

@swissiety swissiety Awaiting requested review from swissiety

Requested changes must be addressed to merge this pull request.

Assignees

No one assigned

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

[Feature]: JimpleJavaObjectPrinter

3 participants

@sahilagichani14 @swissiety