Skip to content

ZJIT: Re-introduce allocation fast path - #18241

Merged
eightbitraptor merged 1 commit into
ruby:masterfrom
eightbitraptor:mvh-reintroduce-zjit-alloc-fast-path
Aug 7, 2026
Merged

ZJIT: Re-introduce allocation fast path#18241
eightbitraptor merged 1 commit into
ruby:masterfrom
eightbitraptor:mvh-reintroduce-zjit-alloc-fast-path

Conversation

@eightbitraptor

Copy link
Copy Markdown
Member

Re-enable rb_gc_impl_zjit_new_obj_fastpath, was stubbed to "return false" by the recent rlgc merge. The bump-pointer state moved out of the deleted rb_ractor_newobj_cache_t into the per-objspace rb_heap_t.newobj, so the fastpath and the JIT codegen now index that struct and reach it through ractor->objspace instead of ractor->newobj_cache.

@matzbot
matzbot requested a review from a team August 7, 2026 10:49
@github-actions github-actions Bot added the jit YJIT, ZJIT, ... label Aug 7, 2026
Comment thread zjit.c

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

Please consider #18241 (comment) but otherwise LGTM

@eightbitraptor
eightbitraptor force-pushed the mvh-reintroduce-zjit-alloc-fast-path branch from 351f50a to ea96bd6 Compare August 7, 2026 17:14
Re-enable rb_gc_impl_zjit_new_obj_fastpath, was stubbed to "return
false" by the recent rlgc merge.  The bump-pointer state moved out of
the deleted rb_ractor_newobj_cache_t into the per-objspace
rb_heap_t.newobj, so the fastpath and the JIT codegen now index that
struct and reach it through ractor->objspace instead of
ractor->newobj_cache.
@eightbitraptor
eightbitraptor force-pushed the mvh-reintroduce-zjit-alloc-fast-path branch from ea96bd6 to 6d96390 Compare August 7, 2026 17:14
@eightbitraptor
eightbitraptor enabled auto-merge (rebase) August 7, 2026 17:15
@eightbitraptor
eightbitraptor merged commit 3156d5a into ruby:master Aug 7, 2026
126 of 128 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

jit YJIT, ZJIT, ...

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants