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.

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

Cleanup tier2 debug output #116920

Merged
merged 11 commits into from
Mar 18, 2024
Merged

Conversation

gvanrossum
Copy link
Member

Various tweaks, including a slight refactor of the special cases for _PUSH_FRAME/_POP_FRAME to show the actual operand emitted.

These changes helped me understand what's going on in faster-cpython/ideas#664, but I don't want them mixed in with possible fixes.

Python/optimizer.c Outdated Show resolved Hide resolved
Copy link
Member

@markshannon markshannon left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Looks good.

@gvanrossum gvanrossum merged commit 76d0868 into python:main Mar 18, 2024
53 of 54 checks passed
@gvanrossum gvanrossum deleted the cleanup-tier2-debugg-output branch March 18, 2024 18:08
vstinner pushed a commit to vstinner/cpython that referenced this pull request Mar 20, 2024
Various tweaks, including a slight refactor of the special cases for `_PUSH_FRAME`/`_POP_FRAME` to show the actual operand emitted.
adorilson pushed a commit to adorilson/cpython that referenced this pull request Mar 25, 2024
Various tweaks, including a slight refactor of the special cases for `_PUSH_FRAME`/`_POP_FRAME` to show the actual operand emitted.
diegorusso pushed a commit to diegorusso/cpython that referenced this pull request Apr 17, 2024
Various tweaks, including a slight refactor of the special cases for `_PUSH_FRAME`/`_POP_FRAME` to show the actual operand emitted.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants