-
-
Notifications
You must be signed in to change notification settings - Fork 32.2k
GH-135379: Top of stack caching for the JIT. #135465
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
Open
markshannon
wants to merge
23
commits into
python:main
Choose a base branch
from
faster-cpython:tier-2-tos-caching
base: main
Could not load branches
Branch not found: {{ refName }}
Loading
Could not load tags
Nothing to show
Loading
Are you sure you want to change the base?
Some commits from the old base branch may be removed from the timeline,
and old review comments may become outdated.
+9,562
−1,335
Open
Changes from 1 commit
Commits
Show all changes
23 commits
Select commit
Hold shift + click to select a range
579b758
Tier 2 TOS caching. Work in progress
markshannon 489e510
Tier 2 TOS caching, working for interpreter.
markshannon f603929
Get JIT working
markshannon cf1d7ab
Fix tool to support 3.11
markshannon efd7a0a
Add news
markshannon bb4e6b9
int arithmetic doesn't escape
markshannon e976b9b
Repair stats
markshannon 11de93e
Add missing type annotation
markshannon 4698695
Pacify mypy
markshannon 33837a7
Add type annotation
markshannon 8bb12ef
Avoid overflow gathering stats
markshannon 920e6de
Reduce spilling
markshannon 45e1abd
Merge branch 'main' into tier-2-tos-caching
markshannon 3d72871
Merge branch 'main' into tier-2-tos-caching
markshannon 0240115
Merge branch 'main' into tier-2-tos-caching
markshannon 2850d72
Improve heuristics for stack caching
markshannon 1c291f1
Merge branch 'main' into tier-2-tos-caching
markshannon ba2331a
Add news
markshannon cbee8d2
Fix uop execution stats
markshannon 40988a0
Address review comments
markshannon 76030e9
Address code review
markshannon 991bbea
Merge branch 'main' into tier-2-tos-caching
markshannon 51d4342
Remove blank line
markshannon File filter
Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
There are no files selected for viewing
6 changes: 6 additions & 0 deletions
6
Misc/NEWS.d/next/Core_and_Builtins/2025-06-20-16-03-59.gh-issue-135379.eDg89T.rst
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
Original file line number | Diff line number | Diff line change |
---|---|---|
@@ -0,0 +1,6 @@ | ||
Implement a limited form of register allocation know as "top of stack | ||
caching" in the JIT. It works by keeping 0-3 of the top items in the stack | ||
in registers. The code generator generates multiple versions of thos uops | ||
that do not escape and are relatively small. During JIT compilation, the | ||
copy that produces the least memory traffic is selected, spilling or | ||
reloading values when needed. | ||
Oops, something went wrong.
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.
Uh oh!
There was an error while loading. Please reload this page.