Skip to content

Enable PEP 709 inlined comprehensions #7412

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

Merged
youknowone merged 4 commits into from
Mar 21, 2026
Merged

Enable PEP 709 inlined comprehensions #7412

Show file tree
Hide file tree
Changes from 1 commit
Commits
Show all changes
4 commits
Select commit Hold shift + click to select a range
9be8a94
Enable PEP 709 inlined comprehensions for function-like scopes
youknowone Mar 12, 2026
6723f2f
Add localspluskinds, unify DEREF to localsplus index
youknowone Mar 17, 2026
c18b464
Fix clippy warnings, formatting, restore _opcode_metadata.py
youknowone Mar 17, 2026
b25e850
Exclude async/await comprehensions from PEP 709 inlining in symboltable
youknowone Mar 19, 2026
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
Next Next commit
Enable PEP 709 inlined comprehensions for function-like scopes 
Activate the existing compile_inlined_comprehension() implementation
by fixing 6 bugs that prevented it from working:

- LoadFastAndClear: push NULL (not None) when slot is empty so
  StoreFast can restore empty state after comprehension
- StoreFast: accept NULL from stack for the restore path
- sub_tables.remove(0) replaced with next_sub_table cursor to
  match the pattern used elsewhere in the compiler
- in_inlined_comp flag moved from non-inlined to inlined path
- is_inlined_comprehension_context() now checks comp_inlined flag
  and restricts inlining to function-like scopes
- comp_inlined set only when parent scope uses fastlocals

Symbol table analysis handles conflict detection:
- Nested scopes in comprehension → skip inlining
- Bound name conflicts with parent symbol → skip inlining
- Cross-comprehension reference conflicts → skip inlining
- Splice comprehension sub_tables into parent for nested scope tracking
  • Loading branch information
@youknowone
youknowone committed Mar 20, 2026
commit 9be8a94248e1de40495745fd4156d6b34ae7770c
308 changes: 95 additions & 213 deletions Lib/_opcode_metadata.py
View file
Open in desktop

Some generated files are not rendered by default. Learn more about how customized files appear on GitHub.

Loading