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.

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

[Improvement]: Optimize the LHS temporary variables in BIR for map creation #40805

Closed
HindujaB opened this issue Jun 22, 2023 · 0 comments · Fixed by #40579
Closed

[Improvement]: Optimize the LHS temporary variables in BIR for map creation #40805

HindujaB opened this issue Jun 22, 2023 · 0 comments · Fixed by #40579
Assignees
@HindujaB
Labels
Area/BIR Compiler BIR related issues #Compiler Reason/Other None of the other reasons. Team/jBallerina All the issues related to BIR, JVM backend code generation and runtime Type/Improvement

Comments

@HindujaB
Copy link
Contributor

Description

$subject

Describe your problem(s)

We optimize the use of LHS temporary variables at the BIROptimizer to re-use the loaded variables. But this wasn't applied to the initial values passed to the NewMap and NewArray instructions. Therefore these variables do not get re-used. We need to do the same for the initial values as well.

Describe your solution(s)

No response

Related area

-> Runtime

Related issue(s) (optional)

#37351

Suggested label(s) (optional)

No response

Suggested assignee(s) (optional)

No response
@HindujaB HindujaB self-assigned this Jun 22, 2023
@HindujaB HindujaB added Team/jBallerina All the issues related to BIR, JVM backend code generation and runtime Area/BIR Compiler BIR related issues #Compiler labels Jun 22, 2023
@HindujaB HindujaB moved this to In Progress in Ballerina Team Main Board Jun 22, 2023
@HindujaB HindujaB mentioned this issue Jul 4, 2023
Merged
13 tasks
@HindujaB HindujaB moved this from In Progress to On Hold in Ballerina Team Main Board Jul 19, 2023
@github-project-automation github-project-automation bot moved this from On Hold to Done in Ballerina Team Main Board Dec 6, 2023
@chiranSachintha chiranSachintha added the Reason/Other None of the other reasons. label Dec 6, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@HindujaB HindujaB

Labels
Area/BIR Compiler BIR related issues #Compiler Reason/Other None of the other reasons. Team/jBallerina All the issues related to BIR, JVM backend code generation and runtime Type/Improvement
Projects
Ballerina Team Main Board
Archived in project
Milestone
No milestone
Development

Successfully merging a pull request may close this issue.

Support record defaultable fields with closure values for runtime HindujaB/ballerina-lang
2 participants
@HindujaB @chiranSachintha