KarpelesLab / goro / 25324078757

13%

push

github

vm: route binops through compiler.EvalBinop for full PHP semantics

Adds compiler.EvalBinop — a public dispatcher that takes
pre-evaluated ZVals and applies the same numeric-coercion,
array-union, GMP-overload, type-error, and warning logic as
runOperator.Run does inline.

The VM's opAdd / opSub / opMul / opDiv / opMod / opPow /
opBitAnd / opBitOr / opBitXor / opShl / opShr now route
through EvalBinop instead of calling OperatorMath /
OperatorMathLogic directly. Same semantics as the AST: an
array+array `+` produces a union, GMP objects dispatch via
HandleDoOperation, non-numeric strings raise TypeError, etc.

Comparison and concat ops keep their direct paths since
OperatorCompare and OperatorAppend already handle the full
edge-case set.

No phpt regressions.

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

58 of 115 new or added lines in 2 files covered. (50.43%)

6642 of 113561 relevant lines covered (5.85%)

652.2 hits per line