KarpelesLab / goro / 26137959604

12%

Build Type

push

github

Committed by MagicalTux

Commit Message

vm: lower \$obj->prop = v native (stmt context, static name, non-nullsafe)

emitOperator's '=' branch for an objectVarNode LHS now emits via
the existing OP_OBJECT_SET path (recv + value + opcode) when the
LHS is non-nullsafe and has a static name. The downstream
ZObject.ObjectSet already handles typed properties, hooks, and
asymmetric visibility — no new helper extraction needed.

IsSlotSafe relaxed: plain '=' to a static-name non-nullsafe
objectVar no longer forces slot-unsafe (op handler doesn't touch
caller locals). Compound (+=, .=) and inc/dec still AST-delegate
via emitCompoundAssign and stay slot-unsafe.

Expr-context property assigns still AST-delegate — the assignment
expression's value needs to remain on the stack, and the VM lacks
a SWAP opcode to keep value+receiver in the right order.

Co-Authored-By: Claude Opus 4.7 <noreply@anthropic.com>

Coverage Stats

21 of 23 new or added lines in 2 files covered. (91.3%)

1 existing line in 1 file now uncovered.

14547 of 117586 relevant lines covered (12.37%)

2552.87 hits per line