Skip to content

Comments

[pull] master from ruby:master#172

Merged
pull[bot] merged 4 commits intoturkdevops:masterfrom
ruby:master
Aug 14, 2025
Merged

[pull] master from ruby:master#172
pull[bot] merged 4 commits intoturkdevops:masterfrom
ruby:master

Conversation

@pull
Copy link

@pull pull bot commented Aug 14, 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 : )

st0012 and others added 4 commits August 13, 2025 13:03
ZJIT: Fix ObjToString rewrite

Currently, the rewrite for `ObjToString` always replaces it with a
`SendWithoutBlock(to_s)` instruction when the receiver is not a
string literal. This is incorrect because it calls `to_s` on the
receiver even if it's already a string.

This change fixes it by:
- Avoiding the `SendWithoutBlock(to_s)` rewrite
- Implement codegen for `ObjToString`
* ZJIT: Add test and implement display for StringIntern HIR

Co-authored-by: Emily Samp <emily.samp@shopify.com>

* ZJIT: Implement StringIntern codegen

Co-authored-by: Emily Samp <emily.samp@shopify.com>

* ZJIT: Fix StringIntern's return type

---------

Co-authored-by: Emily Samp <emily.samp@shopify.com>
On 32-bit Cygwin at least, it was failing to find that function,
presumably due to it being stdcall.

Signed-off-by: Jeremy Drake <github@jdrake.com>

ruby/resolv@bceafef74b
@pull pull bot locked and limited conversation to collaborators Aug 14, 2025
@pull pull bot added the ⤵️ pull label Aug 14, 2025
@pull pull bot merged commit cb28165 into turkdevops:master Aug 14, 2025
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.

3 participants