Skip to content

GH-133171: Prevent combinations of --disable-gil and --enable-experimental-jit (for now) - #133179

Merged
brandtbucher merged 4 commits into
python:mainfrom
brandtbucher:nogil-means-nojit
May 2, 2025
Merged

GH-133171: Prevent combinations of --disable-gil and --enable-experimental-jit (for now)#133179
brandtbucher merged 4 commits into
python:mainfrom
brandtbucher:nogil-means-nojit

Conversation

@brandtbucher

@brandtbucherbrandtbucher commented Apr 30, 2025

Copy link
Copy Markdown
Member

...I do not think it means what you think it means.

@brandtbucherbrandtbucher added build The build process and cross-build topic-free-threading topic-JIT labels Apr 30, 2025
@brandtbucherbrandtbucher self-assigned this Apr 30, 2025
@bedevere-appbedevere-appBot mentioned this pull request Apr 30, 2025
7 tasks
@brandtbucherbrandtbucher changed the title GH-133171: Prevent combinations of --disable-gil *and* --enable-experimental-jit (for now)GH-133171: Prevent combinations of --disable-gil and --enable-experimental-jit (for now)Apr 30, 2025
@brandtbucher

Copy link
Copy Markdown
MemberAuthor

Probably gonna conflict with GH-131751. @savannahostrowski, you can land that one first.

Comment threadPCbuild/build.bat
)
)

if "%UseDisableGil%" EQU "true" if "%UseTIER2%" NEQ "" (

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.

Strictly speaking this would be better as target in pythoncore.vcxproj (see the existing _WarnAboutZlib for example), since there's no requirement to use the batch file, and it's very easy to bypass it. (The same goes for the PGO warning above, I don't remember when that was added.)

But it's not a big deal. It's relatively temporary code anyway

Copy link
Copy Markdown
MemberAuthor

Choose a reason for hiding this comment

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

Yeah, this will only be around for 3.14. I could just put an #error in the C source or whatever, but I figure failing early is best.

Besides, it's not like there's anything wrong with enabling both features. It just doesn't work the way you'd expect.

@brandtbucher
brandtbucher merged commit 2da48e3 into python:mainMay 2, 2025
Pranjal095 pushed a commit to Pranjal095/cpython that referenced this pull request Jul 12, 2025
Sign up for freeto join this conversation on GitHub. Already have an account? Sign in to comment

Labels

buildThe build process and cross-buildtopic-free-threadingtopic-JIT

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants

@brandtbucher@zooba