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

gh-142982 Implement _SEND_NON_GEN specialization by alonme · Pull Request #143977 · python/cpython · GitHub

/ cpython Public

gh-142982 Implement _SEND_NON_GEN specialization - #143977

Closed
alonme wants to merge 3 commits into
python:mainfrom
alonme:specialize-send-non-gen-142982
Closed

gh-142982 Implement _SEND_NON_GEN specialization#143977
alonme wants to merge 3 commits into
python:mainfrom
alonme:specialize-send-non-gen-142982

Conversation

alonme commented Jan 17, 2026
edited by bedevere-app Bot
Loading

Copy link
Copy Markdown
Contributor

This adds a generic specialization for SEND for non generator and Coro cases

@Fidget-Spinner @markshannon
How can i run the Pystats and benchmark suites?

Fidget-Spinner 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

I'm thinking of whether this is worth it, as we're running low on opcodes now. So I'll leave Mark to review.

Comment thread Python/optimizer_bytecodes.c Outdated

alonme commented Jan 19, 2026

Copy link
Copy Markdown
Contributor Author

I'm thinking of whether this is worth it, as we're running low on opcodes now. So I'll leave Mark to review.

@Fidget-Spinner - does the same apply to CALL_ALLOC_AND_ENTER_INIT_SLOT which you also mentioned here ?

Would it make sense to run benchmarks to measure the effect of this change, to better inform the decision?

Copy link
Copy Markdown
Member

This has been superseded by #148963

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.

4 participants


Back | FazBrowse Home | New Git URL