Skip to content

Navigation Menu

Sign in
Appearance settings

Search code, repositories, users, issues, pull requests...

Provide feedback

We read every piece of feedback, and take your input very seriously.

Saved searches

Use saved searches to filter your results more quickly
Appearance settings

gh-133672: Allow LOAD_FAST to be optimized to LOAD_FAST_BORROW #133721

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.

Sign up for GitHub

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
wants to merge 10 commits into
base: main
Choose a base branch
Loading
from
Open

gh-133672: Allow LOAD_FAST to be optimized to LOAD_FAST_BORROW #133721

Changes from 1 commit
Commits
Show all changes
10 commits
Select commit Hold shift + click to select a range
f038e7f
Allow LOAD_FAST to be optimized to LOAD_FAST_BORROW when the loaded
ljfp May 8, 2025
8f38622
📜🤖 Added by blurb_it.
blurb-it[bot] May 9, 2025
f97137c
Merge branch 'main' into fix-issue-133672
ljfp May 9, 2025
9df82ce
Merge branch 'python:main' into fix-issue-133672
ljfp May 14, 2025
2f0ffdd
Improve CFG optimization with new LOAD_FAST_BORROW safety checks.
ljfp May 14, 2025
695a928
Update test_peepholer.py to reflect the new changes made to flowgraph.c
ljfp May 14, 2025
c4b2e95
Merge branch 'main' into fix-issue-133672
ljfp May 14, 2025
a4300a5
Fix linter issues.
ljfp May 14, 2025
8cba63a
Improved documentation for is_borrow_safe and check_borrow_safety_glo…
ljfp May 14, 2025
53f92d4
Merge branch 'main' into fix-issue-133672
ljfp May 21, 2025
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
Prev Previous commit
Next Next commit
Fix linter issues.
  • Loading branch information
@ljfp
ljfp committed May 14, 2025
commit a4300a5e6c21b6555cfba4ceb4c5803c49dd8294
4 changes: 2 additions & 2 deletions 4 Python/flowgraph.c
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -3268,7 +3268,7 @@ optimize_load_fast(cfg_builder *g)
break;
}
}

if (depth_from_tos_at_block_end != -1) {
can_borrow = check_borrow_safety_globally(block, depth_from_tos_at_block_end, local_idx, g);
} else {
Expand Down Expand Up @@ -3305,7 +3305,7 @@ optimize_load_fast(cfg_builder *g)
can_borrow = false;
}
}

if (unconsumed_in_block && !found_any_on_stack) {
can_borrow = false;
}
Expand Down
Loading
Morty Proxy This is a proxified and sanitized view of the page, visit original site.