Skip to content

ZJIT: Specialize setinstancevariable when ivar is already in shape#15290

Merged
tekknolagi merged 10 commits intoruby:masterfrom
tekknolagi:mb-setivar-in-shape
Nov 25, 2025
Merged

ZJIT: Specialize setinstancevariable when ivar is already in shape#15290
tekknolagi merged 10 commits intoruby:masterfrom
tekknolagi:mb-setivar-in-shape

Conversation

@tekknolagi
Copy link
Contributor

Don't support shape transitions for now.

@tekknolagi tekknolagi marked this pull request as ready for review November 22, 2025 00:23
@matzbot matzbot requested a review from a team November 22, 2025 00:24
This reverts commit 5c0db55a7b88ffbf1b2759afb031dba10343db48.
@tekknolagi tekknolagi enabled auto-merge (squash) November 25, 2025 18:23
Copy link
Member

@eregon eregon left a comment

Choose a reason for hiding this comment

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

LGTM, happy to see StoreField & WriteBarrier are turning up useful for this (as planned).

@tekknolagi tekknolagi linked an issue Nov 25, 2025 that may be closed by this pull request
@tekknolagi tekknolagi merged commit 0eb5305 into ruby:master Nov 25, 2025
88 checks passed
@tekknolagi tekknolagi deleted the mb-setivar-in-shape branch November 25, 2025 18:50
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.

ZJIT: Specialize monomorphic SetIvar

3 participants