KarpelesLab / goro / 25336630631

13%

push

github

vm: native object instantiation, property read, method call

Adds OP_NEW_OBJECT, OP_OBJECT_GET, OP_OBJECT_CALL.

Coverage:
- new ClassName(args…)
  Routes through phpobj.NewZObject which already handles
  abstract / interface / enum errors and __construct
  dispatch.
- $obj->prop  (read only)
  Dispatches to ZObject.ObjectGet which already handles
  __get, visibility, lazy init, virtual properties, etc.
- $obj->method(args…)
  Routes through ZObject.GetMethod + ctx.CallZVal so
  visibility checks and __call dispatch happen the same
  way as via the AST.

Out of scope (still falls back to AST):
- new $cls(...)            (dynamic class name)
- new class { … }          (anonymous class)
- $obj->\$name             (dynamic property — encoded with
                            varName="\$name" by the parser)
- $obj->\$name(…)          (dynamic method)
- $obj?->...               (nullsafe)
- $obj->prop = v           (property write)
- Foo::bar / static fields
- Spread / named arguments

Notable subtlety: runObjectVar/runObjectFunc encode dynamic
names by setting varName/op to a "$"-prefixed string (the
AST resolves the variable at run time). The emitter checks
for that prefix and fails to ErrUnsupported so the AST runs
the dynamic resolution.

Bench impact: not measurably different — Fibonacci/Arithmetic
hot paths don't use objects. The win is correctness coverage:
~70% of phpt class tests now run under VM mode.

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

115 of 233 new or added lines in 5 files covered. (49.36%)

9031 of 114253 relevant lines covered (7.9%)

839.46 hits per line