KarpelesLab / goro / 25354718639

13%

push

github

vm: \$this binding for closures + stack reset on try unwind

ResolveCallable now returns the implicit \$this alongside
the callable: closures with bound \$this expose it via
GetThis(), and array-method callables \[obj, "method"]
pass the receiver as \$this. OP_CALL_INDIRECT forwards it
to ctx.CallZVal as optionalThis so the callee body sees
the right binding (fixes arrow-function \$this access).

TryHandler now records the simulated stack depth at try
entry. When the dispatcher unwinds to a catch body, it
truncates the VM stack back to that depth — partially
evaluated arguments from the failing expression
(e.g. throwing midway through a 60-arg function call,
bug34062) are dropped so the catch body starts cleanly.

phpt under GORO_VM=1: 14 regressions (was 16).

Co-Authored-By: Claude Opus 4.7 (1M context) <noreply@anthropic.com>

21 of 40 new or added lines in 4 files covered. (52.5%)

10296 of 114829 relevant lines covered (8.97%)

1063.51 hits per line