PR Review — feat: Add support for various integer and floating-point operations in the interpreter

Status: Needs Changes (Compilation Error)

This PR adds comprehensive support for integer and floating-point bitwise/arithmetic operations, tail call optimization (RETURN_CALL), and a MAP_KEYS instruction. The implementation is well-scoped with good test coverage. However, the code fails to compile due to missing updates to interp.go.

Blocking Issue: Missing newCompiler() Call Updates

Problem:
The main branch has changed newCompiler() signature from newCompiler() (no args) to newCompiler(int) (requires int arg) in jit_stub.go. However, this PR does not update the call sites in interp.go at lines 619 and 657.

CI Error:

interp/interp.go:619:20: not enough arguments in call to newCompiler
	have ()
	want (int)
interp/interp.go:657:19: not enough arguments in call to newCompiler
	have ()
	want (int)

Required Fix:
Update both call sites in interp.go:

• Line 619: compiler, err := newCompiler() → compiler, err := newCompiler(<arg>)
• Line 657: compiler, err := newCompiler() → compiler, err := newCompiler(<arg>)

Where <arg> should be determined based on the jit_stub.go signature change. This is likely the function address or a similar context value (e.g., addr parameter if available, or 0).

Review Summary

Scope: ✅ Well-defined, minimal changes focused on instruction additions
Tests: ✅ Comprehensive new tests for all new operations
Implementation Quality: ✅ Code follows existing patterns, correct stack handling
Merge Readiness: ❌ Blocked by compilation error

Minimal Path to Merge

1. Identify what value should be passed to newCompiler(int) (check the recent jit_stub.go commit)
2. Update the two call sites in interp.go (lines 619, 657)
3. Verify CI passes
4. Ready to merge

Once the compilation issue is fixed, this PR should merge cleanly.

Generated by Claude Code