Skip to content

[pull] master from ruby:master#747

Merged
pull[bot] merged 4 commits intoturkdevops:masterfrom
ruby:master
Feb 4, 2026
Merged

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

Conversation

@pull
Copy link

@pull pull bot commented Feb 4, 2026

See Commits and Changes for more details.


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

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

Earlopain and others added 4 commits February 4, 2026 14:58
Currently I see myself not updating this when a new version is added.

Instead, rewrite it to just work with new versions:
* Try to set the version
* If that doesn't succeed, check if it is lower
* If it isn't lower, it must be higher

We can then use `PM_OPTIONS_VERSION_LATEST` which doesn't need to change.

Also added a very basic test.

ruby/prism@443e9b9959
* ZJIT: Fix codegen for GuardSuperMethodEntry

We need to register the VALUE in the generated code so that it is
visible to the GC. This may be the cause of a crash in the test suite
that Alan found.

* ZJIT: Use LoadField+GuardBitEquals instead of GuardSuperMethodEntry

We have modular instructions. Use them.

* ZJIT: Use LoadField instead of GetBlockHandler

We have modular instructions. Use them.

* ZJIT: Remove unused HIR instructions

We don't use GuardSuperMethodEntry, GuardShape, or GetBlockHandler
anymore.

* ZJIT: Replace GuardNotFrozen with LoadField+GuardNoBitsSet

* ZJIT: Replace GuardNotShared with LoadField+GuardNoBitsSet

* ZJIT: Remove unused HIR instructions

* ZJIT: Remove comment
If self is an immediate, setinstancevariable would have raised.
Therefore we can conclude that self must not be an immediate. This helps
us match YJIT type upgrades (YJIT will upgrade to UnknownHeap or
something similar).
@pull pull bot locked and limited conversation to collaborators Feb 4, 2026
@pull pull bot added the ⤵️ pull label Feb 4, 2026
@pull pull bot merged commit 9167262 into turkdevops:master Feb 4, 2026
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.

2 participants