Skip to content

gh-144712 extern _Py_jit_entry in Modules/_testinternalcapi/interpreter.c#144713

Closed
jjhelmus wants to merge 1 commit into
python:mainfrom
jjhelmus:extern_py_jit_entry
Closed

gh-144712 extern _Py_jit_entry in Modules/_testinternalcapi/interpreter.c#144713
jjhelmus wants to merge 1 commit into
python:mainfrom
jjhelmus:extern_py_jit_entry

Conversation

@jjhelmus

@jjhelmus jjhelmus commented Feb 11, 2026

Copy link
Copy Markdown

Removes duplicate symbols in Python/ceval.o and Modules/_testinternalcapi/interpreter.o when the --enable-experimental-jit option is enabled.

@python-cla-bot

python-cla-bot Bot commented Feb 11, 2026

Copy link
Copy Markdown

All commit authors signed the Contributor License Agreement.

CLA signed

@bedevere-app

bedevere-app Bot commented Feb 11, 2026

Copy link
Copy Markdown

Most changes to Python require a NEWS entry. Add one using the blurb_it web app or the blurb command-line tool.

If this change has little impact on Python users, wait for a maintainer to apply the skip news label instead.

@Fidget-Spinner Fidget-Spinner left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

Thanks

@Fidget-Spinner

Copy link
Copy Markdown
Member

Unfortunately the testCAPI doesn't builld anymore.

@jjhelmus

Copy link
Copy Markdown
Author

Unfortunately the testCAPI doesn't builld anymore.

Yes, I think the symbol is needed when testCAPI is built as an dynamic extension module.

@jjhelmus

Copy link
Copy Markdown
Author

Closing this as a different approach is necessary.

@jjhelmus jjhelmus closed this Feb 12, 2026
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.

2 participants