Skip to content

Conversation

@pull
Copy link

@pull pull bot commented Aug 22, 2025

See Commits and Changes for more details.


Created by pull[bot] (v2.0.0-alpha.3)

Can you help keep this open source service alive? 💖 Please sponsor : )

tekknolagi and others added 7 commits August 21, 2025 19:26
#14270)

If the LiveRange looks like (idx, idx), we will currently not allocate a
register. This change allocates a register and then immediately
deallocates it.

Fix Shopify#614
Co-authored-by: Alan Wu <alansi.xingwu@shopify.com>
Co-authored-by: Max Bernstein <ruby@bernsteinbear.com>
Co-authored-by: Aaron Patterson <tenderlove@ruby-lang.org>
Thanks to nobu for pointing this out. This is a YJIT file so shouldn't
have ZJIT stuff in it. ZJIT doesn't support building on BSDs yet.

Fix: 92b218f
Make vm_search_method return a cme instead of a cc

Both of its callers ended up calling `vm_cc_cme` on the result and only
used the cme, so it's probably better to return a cme directly.

This will also make it easier for ZJIT to later inline the
`rb_vm_objtostring` call.
@pull pull bot locked and limited conversation to collaborators Aug 22, 2025
@pull pull bot added the ⤵️ pull label Aug 22, 2025
@pull pull bot merged commit 6fbe2dd into turkdevops:master Aug 22, 2025
1 of 2 checks passed
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

6 participants