Skip to content

gh-142913: Remove JIT interpreter for interpreter generator tests - #143944

Merged
Fidget-Spinner merged 3 commits into
python:mainfrom
Fidget-Spinner:remove_jit
Jan 17, 2026
Merged

gh-142913: Remove JIT interpreter for interpreter generator tests#143944
Fidget-Spinner merged 3 commits into
python:mainfrom
Fidget-Spinner:remove_jit

Conversation

@Fidget-Spinner

@Fidget-SpinnerFidget-Spinner commented Jan 16, 2026

Copy link
Copy Markdown
Member

@mpagempage left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Looks like a few of the builds are failing because of the unused function.

@bedevere-app

Copy link
Copy Markdown

When you're done making the requested changes, leave the comment: I have made the requested changes; please review again.

@DinoVDinoV left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM!


int Test_EvalFrame_Resumes, Test_EvalFrame_Loads;

#ifdef _Py_TIER2

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I think this should go into the new ceval.h to avoid duplication

@Fidget-Spinner
Fidget-Spinner merged commit 949b5ec into python:mainJan 17, 2026
48 of 49 checks passed
@Fidget-Spinner
Fidget-Spinner deleted the remove_jit branch January 17, 2026 00:34
thunder-coding pushed a commit to thunder-coding/cpython that referenced this pull request Feb 15, 2026
Sign up for freeto join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants

@Fidget-Spinner@DinoV@mpage