Skip to content

Commit 86bfece

Browse files
Peter Neissclaude
andcommitted
Phase 133 Part 1: Compiler Expression Variables (26 functions updated)
**Part A: High-level Binary/Unary Operators** (6 functions) - codebinexpval: leftExpr/rightExpr, operation, rightRegister - codebini: operation, leftExpr/rightExpr, rightValue - codebinK: leftExpr/rightExpr, constantIndex, operation - finishbinexpval: leftExpr/rightExpr, leftRegister, instructionPosition - codeunexpval: expr, operation, targetRegister - codearith: leftExpr/rightExpr **Part B: Expression Helpers** (14 functions) - freeExpressions: leftExpr/rightExpr, leftReg/rightReg - discharge2reg: expr, targetRegister - discharge2anyreg: expr - exp2reg: expr, targetRegister, falsePosition/truePosition, fallJump, finalLabel - exp2K: expr, constantIndex - exp2RK: expr - exp2anyreg: expr - exp2anyregup: expr - exp2nextreg: expr - exp2val: expr **Part C: Code Generation Helpers** (6 functions started) - freeExpression: expr - str2K: expr - negatecondition: expr - jumponcond: expr, condition - codenot: expr **Impact**: ~200+ identifier improvements across ~26 critical compiler functions - e/e1/e2 → expr/leftExpr/rightExpr (self-documenting) - r/r1/r2 → targetRegister/leftRegister/rightRegister (clear purpose) - op → operation (full word) - v2 → rightValue/constantIndex (semantic clarity) - p_f/p_t → falsePosition/truePosition (unambiguous) **Files**: src/compiler/lcode.cpp (~26 functions, ~200 variable occurrences) **Testing**: All tests pass ✅ (2.10s, maintained excellent performance) **Risk**: LOW (compiler-only changes, no VM hot path impact) 🤖 Generated with [Claude Code](https://claude.com/claude-code) Co-Authored-By: Claude <noreply@anthropic.com>
1 parent bd79637 commit 86bfece

File tree

1 file changed

+144
-144
lines changed

1 file changed

+144
-144
lines changed

0 commit comments

Comments
 (0)