Implement liveness analysis #229
Open
Conversation
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
kulcsaradam
force-pushed
the
live_analysis
branch
from
29ef8fa to
6660ba7
Compare
kulcsaradam
force-pushed
the
live_analysis
branch
from
6660ba7 to
4ca3dd1
Compare
kulcsaradam
force-pushed
the
live_analysis
branch
from
4ca3dd1 to
ddfeaf8
Compare
zherczeg
requested changes
Mar 21, 2024
kulcsaradam
force-pushed
the
live_analysis
branch
2 times, most recently
from
bfdbb4c to
4bb4a46
Compare
kulcsaradam
force-pushed
the
live_analysis
branch
2 times, most recently
from
9d6a783 to
2b9957b
Compare
kulcsaradam
force-pushed
the
live_analysis
branch
5 times, most recently
from
e0b6926 to
0a169a2
Compare
kulcsaradam
force-pushed
the
live_analysis
branch
3 times, most recently
from
c062e79 to
0bd6931
Compare
kulcsaradam
force-pushed
the
live_analysis
branch
from
0bd6931 to
52cc323
Compare
kulcsaradam
force-pushed
the
live_analysis
branch
3 times, most recently
from
903818c to
8e94ba5
Compare
kulcsaradam
force-pushed
the
live_analysis
branch
4 times, most recently
from
8eb6770 to
b7d1651
Compare
kulcsaradam
force-pushed
the
live_analysis
branch
2 times, most recently
from
1fae2e9 to
3033fb2
Compare
kulcsaradam
force-pushed
the
live_analysis
branch
from
3033fb2 to
dda696b
Compare
kulcsaradam
force-pushed
the
live_analysis
branch
4 times, most recently
from
573c03b to
7928cd9
Compare
kulcsaradam
force-pushed
the
live_analysis
branch
3 times, most recently
from
c1b4afd to
f99de50
Compare
kulcsaradam
changed the title
kulcsaradam
force-pushed
the
live_analysis
branch
from
f99de50 to
8a4974e
Compare
kulcsaradam
force-pushed
the
live_analysis
branch
4 times, most recently
from
07d8e4f to
889b81a
Compare
kulcsaradam
force-pushed
the
live_analysis
branch
4 times, most recently
from
e141fa4 to
cc50def
Compare
kulcsaradam
changed the title
kulcsaradam
force-pushed
the
live_analysis
branch
from
cc50def to
47849d4
Compare
Implement live variable analysis for function parameters, variables and constants. Define the range from start to end offset where a variable is used and allocate space for used variables. Also define which variables need to be cleared before use. Move constant values to a lower offset on the stack if possibles. Signed-off-by: Adam Laszlo Kulcsar <[email protected]>
kulcsaradam
force-pushed
the
live_analysis
branch
from
47849d4 to
4fc330b
Compare
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
With this patch I aim to optimize Walrus stack useage. This patch does not run without errors yet. Currently, there is a variable pushed down the stack for each local variable.
My idea is to analyse the Bytecode and get the maximum amount of variables the code needs at one point and only push down the stack that given number of variables.
The patch is still in draft, I am just curious about things I might have missed or done better! There are definitly parts that can be optimized, which I will try to once I get the patch running properly.