Skip to content

Add support for JIT binaries#9

Merged
pablogsal merged 4 commits into
python:mainfrom
mdboom:add-jit
Jun 28, 2025
Merged

Add support for JIT binaries#9
pablogsal merged 4 commits into
python:mainfrom
mdboom:add-jit

Conversation

@mdboom

@mdboom mdboom commented Jun 27, 2025

Copy link
Copy Markdown
Contributor

Adds support for building binaries with the JIT.

Cc: @brandtbucher

@brandtbucher brandtbucher 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.

I'm not familiar with this codebase, but the configuration seems fine to me (and I'd like to see JIT memory tracking)!

Do we also need to ensure somewhere that LLVM is installed?

Comment thread backend/scripts/populate_binaries.py Outdated
"flags": ["--enable-experimental-jit=yes"],
"description": "Just-In-Time compilation enabled.",
"color": "#fc03df",
"icon": "zap",

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.

Is this icon supposed to be unique? Looks like both PGO and nogil are using the same zap icon.

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.

Nah, is just the icon that is used when displayed here: https://memory.python.org/binaries

it can be changed later in the admin panel

mdboom and others added 2 commits June 27, 2025 13:40
Co-authored-by: Brandt Bucher <brandtbucher@gmail.com>
@mdboom

mdboom commented Jun 27, 2025

Copy link
Copy Markdown
Contributor Author

Do we also need to ensure somewhere that LLVM is installed?

Good point. I've added that (cribbed from https://github.com/python/cpython/blob/main/.github/workflows/jit.yml)

"flags": ["--enable-experimental-jit=yes"],
"description": "Just-In-Time compilation enabled.",
"color": "#fc03df",
"icon": "zap",

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.

Nah, is just the icon that is used when displayed here: https://memory.python.org/binaries

it can be changed later in the admin panel

@pablogsal pablogsal merged commit 71dd85d into python:main Jun 28, 2025
@pablogsal

Copy link
Copy Markdown
Member

Thanks a lot @mdboom !

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants