Skip to content

Don't build JITs for interpreter speed baseline#486

Draft
XrXr wants to merge 1 commit into
mainfrom
disable-zjit
Draft

Don't build JITs for interpreter speed baseline#486
XrXr wants to merge 1 commit into
mainfrom
disable-zjit

Conversation

@XrXr

@XrXr XrXr commented Apr 7, 2026

Copy link
Copy Markdown
Contributor

We found out that as expected, the interpreter runs slower on some micro benchmarks when the build contains ZJIT. As ZJIT builds optimize for when ZJIT is enabled, it's more fair that we compare --enable-zjit speed against a --disable-yjit --disable-zjit interpreter baseline.

@XrXr
XrXr requested a review from rwstauner April 7, 2026 21:35
@jhawthorn

jhawthorn commented Apr 7, 2026

Copy link
Copy Markdown
Member

Maybe having merged #483 helps

EDIT: no it does not.

@XrXr
XrXr marked this pull request as draft April 7, 2026 21:55
@XrXr XrXr changed the title Also disable ZJIT for no_jit builds Don't build JITs for interpreter speed baseline Apr 7, 2026
We found out that as expected, the interpreter runs slower on some micro
benchmarks when the build contains ZJIT. As ZJIT builds optimize for
when ZJIT is enabled, it's more fair that we compare `--enable-zjit`
speed against a `--disable-yjit --disable-zjit` interpreter baseline.
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.

2 participants