KarpelesLab / goro / 26002611573

13%

Build Type

push

github

Committed by MagicalTux

Commit Message

vm: lower Cls::CONST class-const fetch natively

Extracted EvalClassDynConst(ctx, classVal, nameVal) from
runClassDynConst.Run. Added OP_CLASS_DYN_CONST which pops the
const-name value, the class-source value, and pushes the resolved
const. Both AST runner and VM share the helper — visibility,
interface/parent walking, CompileDelayed self-reference detection,
and the \`::class\` special case all live in the helper unchanged.

The class-source expression is compiled normally (literal class name
becomes OP_LOAD_CONST, dynamic eval gets emitted as a subexpression).
The name expression similarly — most uses are literal const names.

Most common class-const fetch form is now native; the runClassStaticObjRef,
runClassStaticVarRef, runClassStaticDynVarRef forms (static prop, dyn
static prop) still AST-delegate.

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

Coverage Stats

5 of 34 new or added lines in 4 files covered. (14.71%)

2 existing lines in 1 file now uncovered.

14200 of 116354 relevant lines covered (12.2%)

2558.17 hits per line