GV

GeezerVison

The World Wide Web, restored to 800 pixels of glory.

Home|About|Gaming cabinet|Archives|How it works|Starter sites

On this site

  • Sign in
  • ActionsCI/CD and automation
  • CodespacesCloud development environments
  • All Features
  • Documentation
  • Training
  • Certifications
  • Build with Copilot
  • Customize Copilot
  • Integrations
  • Trending repos
  • GitHub Sponsors
  • Developer articles
  • Enterprises

This view

  • Original page
  • New address

GeezerVison is showing github.com — GH-123391: Code generators: Track stack and locals in if statements. by markshannon · Pull Request #123469 · python/cpython

Conversation

@markshannon

@markshannon markshannon commented Aug 29, 2024 •
edited by bedevere-app Bot
Loading

Uh oh!

There was an error while loading. Please reload this page.

Copy link
Copy Markdown
Member
  • Issue: Interpreter code generators need to be able to handle basic flow control #123391

Sorry, something went wrong.

Uh oh!

There was an error while loading. Please reload this page.
All reactions
markshannon added 17 commits August 7, 2024 16:13
@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-app bedevere-app Bot mentioned this pull request Aug 29, 2024
Interpreter code generators need to be able to handle basic flow control #123391
Closed
markshannon added 2 commits August 29, 2024 12:03
@markshannon
Clean up asserts
a2e430a
@markshannon
Update test
95408d3
@markshannon
markshannon force-pushed the spill-before-escaping-calls branch from 5b06484 to 95408d3 Compare August 29, 2024 15:25
@markshannon markshannon added the skip news label Aug 29, 2024
@markshannon
Flush locals as well on error
eb3a645
@markshannon markshannon closed this Sep 23, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

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

Uh oh!

There was an error while loading. Please reload this page.

Assignees

No one assigned

Labels

skip news

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

Uh oh!

There was an error while loading. Please reload this page.

1 participant

@markshannon
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.
You are visitor 01402503 — GeezerVison does not store the pages you look up.
Best viewed with Netscape Navigator 4.7 or Internet Explorer 6 at 800×600. No cookies required.