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

gh-139038: Credit more people for JIT in 3.15 by Fidget-Spinner · Pull Request #148359 · python/cpython · GitHub

  1. python
  2. cpython

Repository navigation

gh-139038: Credit more people for JIT in 3.15 - #148359

Merged
Fidget-Spinner merged 3 commits into
python:mainfrom
Fidget-Spinner:credit-jit-people
Apr 10, 2026
ConversationCommits (3)ChecksFiles changed
Merged

gh-139038: Credit more people for JIT in 3.15#148359
Fidget-Spinner merged 3 commits into
python:mainfrom
Fidget-Spinner:credit-jit-people

Conversation

Fidget-Spinner commented Apr 10, 2026 •
edited by github-actions Bot
Loading

Copy link
Copy Markdown
Member

cc @kumaraditya303 @NekoAsakura @cocolato @Sacul0457

@NekoAsakura please let me know what name you would prefer. Thank you!

📚 Documentation preview 📚: https://cpython-previews--148359.org.readthedocs.build/

bedevere-app Bot added docs Documentation in the Doc dir skip news labels Apr 10, 2026
github-project-automation Bot moved this to Todo in Docs PRs Apr 10, 2026
bedevere-app Bot mentioned this pull request Apr 10, 2026
Open
4 of 19 tasks
savannahostrowski approved these changes Apr 10, 2026
View reviewed changes

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

Pending preferred name inclusion, of course! But this is awesome!

NekoAsakura commented Apr 10, 2026

Copy link
Copy Markdown
Member

It's Zhongtian Zheng. Cheers ❤️

StanFromIreland reviewed Apr 10, 2026
View reviewed changes
Comment thread Doc/whatsnew/3.15.rst Outdated
StanFromIreland reviewed Apr 10, 2026
View reviewed changes
Comment thread Doc/whatsnew/3.15.rst Outdated
sharktide reviewed Apr 10, 2026
View reviewed changes

sharktide left a comment •
edited
Loading

Copy link
Copy Markdown
Contributor

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 left a couple of suggestions that make the docs more explicit and clear. They may not be needed, so I’ll leave this up to you whichever version you like better

Comment thread Doc/whatsnew/3.15.rst
experience lower memory usage for generated machine code and more efficient
machine code versus the old JIT.

machine code versus 3.14.

sharktide Apr 10, 2026

Copy link
Copy Markdown
Contributor

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
Suggested change
machine code versus 3.14.
machine code versus the machine code generated by the 3.14 JIT compiler.

Same here

Fidget-Spinner Apr 10, 2026

Copy link
Copy Markdown
Member Author

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 think this is already implied. It would also be dishonest of us to compare to something else, so I hope people think the same.

Comment thread Doc/whatsnew/3.15.rst
(Contributed by Ken Jin, Donghee Na, Zheao Li, Hai Zhu, Savannah Ostrowski,
Reiden Ong, Noam Cohen, Tomas Roun, PuQing, Cajetan Rodrigues, and Sacul in :gh:`134584`.)

The JIT optimizer now supports significantly more operations than in 3.14.

sharktide Apr 10, 2026

Copy link
Copy Markdown
Contributor

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
Suggested change
The JIT optimizer now supports significantly more operations than in 3.14.
The JIT optimizer now supports significantly more operations than the JIT optimizer in 3.14

Small suggestion for clarity

Comment thread Doc/whatsnew/3.15.rst Outdated
(Contributed by Ken Jin and Savannah Ostrowski in :gh:`132732`.)

The JIT avoids :term:`reference count`\ s where possible. This generally
:term:`reference count`\ s are avoided where safe to do so. This generally

sharktide Apr 10, 2026

Copy link
Copy Markdown
Contributor

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
Suggested change
:term:`reference count`\ s are avoided where safe to do so. This generally
:term:`reference count`\ s are avoided whenever it is safe to do so. This generally

Fidget-Spinner Apr 10, 2026

Copy link
Copy Markdown
Member Author

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

Thank you.

Fidget-Spinner commented Apr 10, 2026

Copy link
Copy Markdown
Member Author

Thanks for the review @StanFromIreland

StanFromIreland reviewed Apr 10, 2026
View reviewed changes
Comment thread Doc/whatsnew/3.15.rst Outdated
Fidget-Spinner merged commit c4d9512 into python:main Apr 10, 2026
30 checks passed
github-project-automation Bot moved this from Todo to Done in Docs PRs Apr 10, 2026

Fidget-Spinner commented Apr 10, 2026

Copy link
Copy Markdown
Member Author

JFYI: CPython web docs are rebuilt roughly every 5 mins now. So watch out for your names on https://docs.python.org/dev/whatsnew/3.15.html#upgraded-jit-compiler !

Fidget-Spinner deleted the credit-jit-people branch April 10, 2026 19:03
ljfp pushed a commit to ljfp/cpython that referenced this pull request Apr 25, 2026
pythongh-139038: Credit more people for JIT in 3.15 (pythonGH-148359)
ca8437c
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
Show hidden characters
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

StanFromIreland StanFromIreland left review comments
savannahostrowski savannahostrowski approved these changes
AA-Turner Awaiting requested review from AA-Turner AA-Turner is a code owner
+1 more reviewer
sharktide sharktide left review comments
Reviewers whose approvals may not affect merge requirements

Assignees

No one assigned

Labels

docs Documentation in the Doc dir skip news

Projects

Docs PRs
Status: Done

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

5 participants

Back | FazBrowse Home | New Git URL