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

Optimize CodeGen for the executable JAR #43560

Open
wants to merge 200 commits into
base: master
Choose a base branch
from
Open

Optimize CodeGen for the executable JAR #43560

wants to merge 200 commits into from

Conversation

MaryamZi
Copy link
Member

@MaryamZi MaryamZi commented Nov 8, 2024 •
edited
Loading

Purpose

Fix and merge #43013 to master

Fixes #43111

@Thushara-Piyasekara @heshanpadmasiri
Add DeadCodeAnalyzer
bd8f4fd
@Thushara-Piyasekara @heshanpadmasiri
Add dead code detection for functions and anonymous functions
0c05d02
@Thushara-Piyasekara @heshanpadmasiri
Add comments and optimizations
1d7a670
@Thushara-Piyasekara @heshanpadmasiri
Add Dead Code Detection for multiple modules
00c1143
@Thushara-Piyasekara @heshanpadmasiri
Add function invocation persistence through modifying BIR
da0a68b
@Thushara-Piyasekara @heshanpadmasiri
Add deletion of unused top level functions from thin jars
508e7de
@Thushara-Piyasekara @heshanpadmasiri
Remove ModuleData class. Optimize DeadCodeAnalyzer
71a5b06
@Thushara-Piyasekara @heshanpadmasiri
Add comments. Fix certain langlib children not being updated
be5fb8b
@Thushara-Piyasekara @heshanpadmasiri
Add file license signatures and optimizations
2131060
@Thushara-Piyasekara @heshanpadmasiri
Minor optimizations
aad61ff
@Thushara-Piyasekara @heshanpadmasiri
Fix certain child functions not being added to parent symbol
c48745b
@Thushara-Piyasekara @heshanpadmasiri
Fix certain child functions not being added to parent symbol
a32c3f9
@Thushara-Piyasekara @heshanpadmasiri
Optimize code according to best practices and suggestions
cdf3030
@Thushara-Piyasekara @heshanpadmasiri
Refactor CompilerPhaseRunner
498e552
@Thushara-Piyasekara @heshanpadmasiri
Improve Resource function handling logic
4fa6129
@Thushara-Piyasekara @heshanpadmasiri
Refactor licence and variable declarations
85a16bf
@Thushara-Piyasekara @heshanpadmasiri
Refactor file separator
2bb88b1
@Thushara-Piyasekara @heshanpadmasiri
Add BIRDeadNodeAnalyzer
2f57ba8
@Thushara-Piyasekara @heshanpadmasiri
Add detection and deletion of unused Type Definitions
0e17b66
@Thushara-Piyasekara @heshanpadmasiri
Add optimized Thin JAR object generation
158fb6d
@Thushara-Piyasekara @heshanpadmasiri
Add BIR TypeDefAnalyzer
92bbf65
@Thushara-Piyasekara @heshanpadmasiri
Add documentation for TypeDefAnalyzer
0233cfc
@Thushara-Piyasekara @heshanpadmasiri
Add analyzer support for cyclic typeDefs and typeDef function calls
76ee647
@Thushara-Piyasekara @heshanpadmasiri
Refactor the analyzer algorithm
86ff789
@Thushara-Piyasekara @heshanpadmasiri
Add interop dependencies white listing
a66b501
@Thushara-Piyasekara @heshanpadmasiri
Fix TypeDefAnalyzer bugs and refactor code
8961afd
@Thushara-Piyasekara @heshanpadmasiri
Handle null pointer exceptions in BIRNode.getPackageID
2c5dba0
@Thushara-Piyasekara @heshanpadmasiri
Add new entries for interop-dependencies
7eb8ea2
@Thushara-Piyasekara @heshanpadmasiri
Refactor DeadBIRNodeAnalyzer
e208b23
@Thushara-Piyasekara @heshanpadmasiri
Add unused module elimination
22f97d5
@heshanpadmasiri heshanpadmasiri mentioned this pull request Nov 8, 2024
Closed
13 tasks
gimantha and others added 2 commits November 8, 2024 22:29
@gimantha
Merge pull request #43561 from Thushara-Piyasekara/code-gen-optimizer…
01576b2 
…-feature

Fix deletion of `buildPkg` functions invoked only by `testablePkg`
@Thushara-Piyasekara
Update integration test output for windows
ce0dc94
@Thushara-Piyasekara Thushara-Piyasekara mentioned this pull request Nov 9, 2024
Merged
13 tasks
MaryamZi and others added 4 commits November 9, 2024 08:17
@MaryamZi
Merge pull request #43562 from Thushara-Piyasekara/code-gen-optimizer…
af28a18 
…-feature

Update `OptimizedExecutableTestingTest` output for windows
@Thushara-Piyasekara
Fix testable pkg invocation data handling logic
21cd4ae
@MaryamZi
Merge pull request #43566 from Thushara-Piyasekara/code-gen-optimizer…
095db58 
…-feature

Fix testable pkg `InvocationData` handling logic for Dead Code Elimination
@Thushara-Piyasekara
Fix checkstyle issue
399303f
@Thushara-Piyasekara Thushara-Piyasekara mentioned this pull request Nov 11, 2024
Merged
13 tasks
@MaryamZi
Merge pull request #43567 from Thushara-Piyasekara/code-gen-optimizer…
76598e7 
…-feature

Fix checkstyle issue in feature branch `feature-codegen-optimizer`
@MaryamZi MaryamZi closed this Nov 12, 2024
@MaryamZi MaryamZi reopened this Nov 12, 2024
MaryamZi and others added 14 commits November 13, 2024 08:19
@MaryamZi
Merge pull request #43575 from ballerina-platform/master
092cc69 
Sync branches
@Thushara-Piyasekara
Fix optimized thin jar caching for bal test
38881b7
@Thushara-Piyasekara
Fix review suggestions
4227209
@MaryamZi
Merge pull request #43582 from Thushara-Piyasekara/code-gen-optimizer…
bfe8fe6 
…-feature

Fix thin jar caching when testing the optimized executable
@MaryamZi
Resolve conflicts and merge branch 'master' of https://github.com/bal…
e8bde9a 
…lerina-platform/ballerina-lang into feature-codegen-optimizer
@MaryamZi
Update workflows
4134b08
@MaryamZi
Merge pull request #43610 from MaryamZi/update-workflows
f2dd1af 
Update workflows
@MaryamZi
Resolve conflicts and merge branch 'feature-codegen-optimizer' of htt…
52ff1c6 
…ps://github.com/ballerina-platform/ballerina-lang into feature-codegen-optimizer
@MaryamZi
Merge branch 'master' of https://github.com/ballerina-platform/baller…
e539bb8 
…ina-lang into feature-codegen-optimizer
@MaryamZi
Update tests for strand annotation
f909a51
@MaryamZi
Fix tests
98cb3c7
@MaryamZi
Resolve conflicts and merge branch 'master' of https://github.com/bal…
3c776af 
…lerina-platform/ballerina-lang into feature-codegen-optimizer
@MaryamZi
Fix a windows test failure
b60c011
@MaryamZi
Merge pull request #43609 from MaryamZi/feature-codegen-optimizer
b341719 
Sync with master
@sonarqubecloud SonarQubeCloud
Copy link

Quality Gate Failed Quality Gate failed

Failed conditions
0.0% Coverage on New Code (required ≥ 80%)
C Reliability Rating on New Code (required ≥ A)

See analysis details on SonarQube Cloud

Catch issues before they fail your Quality Gate with our IDE extension SonarQube for IDE

@MaryamZi MaryamZi changed the title [Temporary] Implement dead code elimination Optimize CodeGen for executable jar Nov 22, 2024
@MaryamZi MaryamZi changed the title Optimize CodeGen for executable jar Optimize CodeGen for the executable JAR Nov 22, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@gimantha gimantha Awaiting requested review from gimantha gimantha is a code owner

@KavinduZoysa KavinduZoysa Awaiting requested review from KavinduZoysa KavinduZoysa is a code owner

@warunalakshitha warunalakshitha Awaiting requested review from warunalakshitha warunalakshitha is a code owner

@azinneera azinneera Awaiting requested review from azinneera azinneera is a code owner

@sameerajayasoma sameerajayasoma Awaiting requested review from sameerajayasoma sameerajayasoma is a code owner

@hasithaa hasithaa Awaiting requested review from hasithaa hasithaa is a code owner

At least 1 approving review is required 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.

[Improvement]: Implement dead code elimination
4 participants
@MaryamZi @Thushara-Piyasekara @heshanpadmasiri @gimantha