Skip to content

GH-123391: Code generators: Track stack and locals in if statements.#123469

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
Closed
markshannon wants to merge 20 commits into from
Closed

GH-123391: Code generators: Track stack and locals in if statements. #123469

markshannon wants to merge 20 commits into from

Conversation

@markshannon
Copy link
Member

@markshannonmarkshannon commented Aug 29, 2024•
edited by bedevere-app bot
Loading

@markshannon
Add copy and == support to Stack class
75bda28
@markshannon
Merge branch 'main' into stack-copy-and-merge
c7e9102
@markshannon
blacken stack.py
d331371
@markshannon
Cases generator: Track reachability and divergent stacks in if statem…
4673d8a
…ents.
@markshannon
Fix type errors and rename ahead to look_ahead
132df06
@markshannon
Track state of output variables
b7f71d4
@markshannon
Handle stack and output locals togather as Storage class
b97dea9
@markshannon
Track locals as well as stack on differing paths
9838a05
@markshannon
Use 'PEP 7' in syntax error, to make it clear that this is not a C sy…
1f829be
…ntax error, just a tool limitation.
@markshannon
Push peeks back to stack in optimizer code gen
d2e5f12
@markshannon
Merge branch 'main' into stack-copy-and-merge
2753014
@markshannon
Update test
1754fc4
@markshannon
Cleanup whitespace
98f9720
@markshannon
Add tests for PEP 7 parsing and escaping call in condition
0284b3f
@markshannon
Remove merge artifact
03bea71
@markshannon
Remove test for escaping calls in conditions
ca2f457
@markshannon
Spill before escaping calls. Initial attempt
57de61f
@bedevere-appbedevere-appbot mentioned this pull request Aug 29, 2024
Closed
@markshannonmarkshannonforce-pushed the spill-before-escaping-calls branch from 5b06484 to 95408d3CompareAugust 29, 2024 15:25
Sign up for freeto join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@tomasr8tomasr8Awaiting requested review from tomasr8tomasr8 will be requested when the pull request is marked ready for reviewtomasr8 is a code owner

@Fidget-SpinnerFidget-SpinnerAwaiting requested review from Fidget-SpinnerFidget-Spinner will be requested when the pull request is marked ready for reviewFidget-Spinner is a code owner

@savannahostrowskisavannahostrowskiAwaiting requested review from savannahostrowskisavannahostrowski will be requested when the pull request is marked ready for reviewsavannahostrowski is a code owner

Assignees

No one assigned

Labels

skip news

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

1 participant

@markshannon