KarpelesLab / goro / 26061116615

13%

Build Type

push

github

Committed by MagicalTux

Commit Message

vm: lower unset() for simple-local args natively

Adds OP_UNSET_LOCAL that fires the destructor on the slot value,
clears the slot, and (for slot-unsafe bodies) calls OffsetUnset so
external observers see the removal. The emitter picks the native
path only when every unset arg is a plain \$name; array-access,
object-prop, and static-prop forms keep the existing AST path.

CallDestructorIfNeeded is exported so the VM can reuse the
recursive array-walker + destructor-chain that runnableUnset.Run
already implements — no duplication.

Bodies that only use unset() on simple locals are now slot-safe.

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

Coverage Stats

39 of 53 new or added lines in 5 files covered. (73.58%)

12 existing lines in 4 files now uncovered.

14318 of 116654 relevant lines covered (12.27%)

2598.98 hits per line