KarpelesLab / goro / 25773415063

13%

Build Type

push

github

Committed by MagicalTux

Commit Message

compiler: defer VM compile of named top-level functions

Closures inside compileFunctionWithName were VM-compiled inline,
before the parent's RegisterLazyFunc pre-pass ran. That meant
emit-time queries about sibling top-level functions (e.g.
FunctionTakesByRef for a forward-referenced \`function inc(&\$x)\`)
always missed and the call site emitted a value-passing native
call, silently dropping the ref binding.

Mark named top-level functions for deferred VM compile and run
TryBuildVMClosureBody for them in compileInner after the lazy
pre-pass. Anonymous closures and class methods still compile
inline.

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

Coverage Stats

14 of 16 new or added lines in 2 files covered. (87.5%)

2 existing lines in 2 files now uncovered.

14021 of 115536 relevant lines covered (12.14%)

2049.54 hits per line