KarpelesLab / goro / 26083252686

12%

Build Type

push

github

Committed by MagicalTux

Commit Message

vm: lower unset(\$container[\$key]) natively for simple-local containers

Adds OP_UNSET_DIM + compiler.UnsetArrayDim helper (extracted from
the runArrayAccess.WriteValue value==nil branch). Refuses string-on-
array unset, no-ops on nil containers, otherwise calls
ZArray.OffsetUnset which fires destructors on object elements.

Extends the unset emit to accept array-access-on-simple-local args
alongside simple variables. Nested forms (\$a[\$k1][\$k2]),
object-prop, static-prop, and dyn-name unsets still AST-delegate.

Bodies whose unsets are all supported shapes stay slot-safe.

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

Coverage Stats

53 of 85 new or added lines in 5 files covered. (62.35%)

19 existing lines in 4 files now uncovered.

14470 of 117498 relevant lines covered (12.32%)

2553.51 hits per line