-
-
Notifications
You must be signed in to change notification settings - Fork 32k
gh-132732: Automatically constant evaluate pure operations #132733
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
Fidget-Spinner
wants to merge
32
commits into
python:main
Choose a base branch
from
Fidget-Spinner:pure
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.
+600
−280
Open
Changes from all commits
Commits
Show all changes
32 commits
Select commit
Hold shift + click to select a range
1ffbb6b
Automatically constant evaluate pure operations
Fidget-Spinner 691084d
📜🤖 Added by blurb_it.
blurb-it[bot] b89e4dc
Fix tests
Fidget-Spinner 0959918
Merge branch 'pure' of github.com:Fidget-Spinner/cpython into pure
Fidget-Spinner 2541683
Merge remote-tracking branch 'upstream/main' into pure
Fidg
8000
et-Spinner d5b2208
Apply review suggestions
Fidget-Spinner 71ced86
reduce diff
Fidget-Spinner a10d5a1
Merge remote-tracking branch 'upstream/main' into pure
Fidget-Spinner d22f165
Update pycore_opcode_metadata.h
Fidget-Spinner 8ae38c7
Apply changes from code review
Fidget-Spinner 712a810
Merge remote-tracking branch 'upstream/main' into pure
Fidget-Spinner dc2d922
Address review, add test
Fidget-Spinner f3f2a69
Add more tests
Fidget-Spinner 53ce10f
Fix tests
Fidget-Spinner 17634a8
Push fix noticed by Mark and Brandt
Fidget-Spinner 4937c2f
Merge remote-tracking branch 'upstream/main' into pure
Fidget-Spinner ae08b79
Merge remote-tracking branch 'upstream/main' into pure
Fidget-Spinner c0c6600
remove pure from _POP_CALL_TWO_LOAD_CONST_INLINE_BORROW
Fidget-Spinner 6bdd3f9
Merge remote-tracking branch 'upstream/main' into pure
Fidget-Spinner de8e170
use upstream changes for stackref
Fidget-Spinner c2f8e22
remove unused comment
Fidget-Spinner ac7e343
Address review
Fidget-Spinner 05b822f
fix test
Fidget-Spinner b4c2e93
fix negative refcount
Fidget-Spinner d229f57
Merge remote-tracking branch 'upstream/main' into pure
Fidget-Spinner c4aae6c
Merge remote-tracking branch 'upstream/main' into pure
Fidget-Spinner 8552182
Use `REPLACE_OPCODE_IF_EVALUTES_PURE`
Fidget-Spinner 703dfc9
Fix test, move is_abstract to subclass attribute
Fidget-Spinner b278734
fix linter/mypy
Fidget-Spinner 73a8b00
remove whitespace
Fidget-Spinner 4116a31
Remove PyDict_Type
Fidget-Spinner 548b67c
add bool type
Fidget-Spinner 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
Some generated files are not rendered by default. Learn more about how customized files appear on GitHub.
Oops, something went wrong.
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
Some generated files are not rendered by default. Learn more about how customized files appear on GitHub.
Oops, something went wrong.
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
1 change: 1 addition & 0 deletions
1
Misc/NEWS.d/next/Core_and_Builtins/2025-04-19-16-22-47.gh-issue-132732.jgqhlF.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 @@ | ||
Automatically constant evaluate bytecode operations marked as pure in the JIT optimizer. | ||
Oops, something went wrong.
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.