FazBrowse GitHub Viewer | Trending |
URL:
| Home
Tools: [Download Repo ZIP]   [Original HTTPS Page]

gh-131798: JIT optimizer: Support custom binary op and property frames by Fidget-Spinner · Pull Request #143735 · python/cpython · GitHub

Repository navigation

gh-131798: JIT optimizer: Support custom binary op and property frames - #143735

Merged
Fidget-Spinner merged 3 commits into
python:mainfrom
Fidget-Spinner:last_few_push_frames
Jan 13, 2026
Merged

Fidget-Spinner merged 3 commits into
python:mainfrom
Fidget-Spinner:last_few_push_frames

Conversation

Fidget-Spinner commented Jan 12, 2026 •
edited
Loading

Copy link
Copy Markdown
Member

BINARY_OP_SUBSCR_INIT_FRAME is the opcode for subscripting custom __getitem__.

LOAD_ATTR_PROPERTY is the opcode for loading an attribute decorated with @property.

I see a solid 0.5-1% speedup from this PR alone on Sam's fastmark (pyperformance subset).

Fidget-Spinner changed the title gh-131798: JIT: Support custom binary op and property frames gh-131798: JIT optimizer: Support custom binary op and property frames Jan 12, 2026
savannahostrowski self-requested a review January 13, 2026 04:25
Comment thread Lib/test/test_capi/test_opt.py Outdated
Comment thread Lib/test/test_capi/test_opt.py Outdated

bedevere-app Bot commented Jan 13, 2026

Copy link
Copy Markdown

When you're done making the requested changes, leave the comment: I have made the requested changes; please review again.

savannahostrowski left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

Choose a reason Spam Abuse Off Topic Outdated Duplicate Resolved Low Quality

LGTM!

Fidget-Spinner merged commit 2873c31 into python:main Jan 13, 2026
65 of 66 checks passed
Fidget-Spinner deleted the last_few_push_frames branch January 13, 2026 19:11
thunder-coding pushed a commit to thunder-coding/cpython that referenced this pull request Feb 15, 2026
This file contains hidden or 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
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


Back | FazBrowse Home | New Git URL