Skip to content

Conversation

mpage
Copy link
Contributor

@mpage mpage commented Oct 31, 2024

The cases generator inserts code to save and restore the stack pointer around statements that contain escaping calls. To find the beginning of such statements, we walk backwards from the escaping call until we encounter a token that is treated as a statement terminator. This set of terminators should include preprocessor directives.

mpage added 2 commits October 30, 2024 16:17
…alls

The cases generator inserts code to save and restore the stack pointer around
statements that contain escaping calls. To find the beginning of such statements,
we would walk backwards from the escaping call until we encountered a token that
was treated as a statement terminator. This set of terminators should include
preprocessor directives.
@mpage mpage requested review from Yhg1s and colesbury October 31, 2024 00:58
@mpage mpage marked this pull request as ready for review October 31, 2024 00:58
@mpage mpage requested a review from markshannon as a code owner October 31, 2024 00:58
@mpage mpage merged commit 821759d into python:main Nov 1, 2024
48 checks passed
@mpage mpage deleted the gh-126211-fix-cases-gen branch November 1, 2024 15:53
picnixz pushed a commit to picnixz/cpython that referenced this pull request Dec 8, 2024
…aining escaping calls (python#126213)

The cases generator inserts code to save and restore the stack pointer around
statements that contain escaping calls. To find the beginning of such statements,
we would walk backwards from the escaping call until we encountered a token that
was treated as a statement terminator. This set of terminators should include
preprocessor directives.
ebonnal pushed a commit to ebonnal/cpython that referenced this pull request Jan 12, 2025
…aining escaping calls (python#126213)

The cases generator inserts code to save and restore the stack pointer around
statements that contain escaping calls. To find the beginning of such statements,
we would walk backwards from the escaping call until we encountered a token that
was treated as a statement terminator. This set of terminators should include
preprocessor directives.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants