KarpelesLab / goro / 25336784689

13%

push

github

vm: native ?? (null coalesce) for variable LHS

Adds OP_JMP_IF_NOT_NULL_PEEK and an emitter case for the
T_COALESCE operator. PHP's ?? has "set and non-null" semantics
on the LHS — undefined variables don't warn — so the emitter
loads a variable LHS via OP_LOAD_LOCAL (the no-warn form)
rather than OP_LOAD_LOCAL_OR_WARN.

Currently only variable LHS is lowered; array/property/
method-call LHS still falls back to AST since their AST runners
have writeContext-style suppression that isn't trivial to
replicate.

Tests cover defined / undefined / null / zero / empty-string
LHS plus chained `?? ?? ?? default`.

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

23 of 32 new or added lines in 2 files covered. (71.88%)

9107 of 114285 relevant lines covered (7.97%)

892.01 hits per line