From 4df7c22b99349dcf2175d69679eafa175d49aa9f Mon Sep 17 00:00:00 2001 From: _ov4 Date: Sun, 26 Jul 2026 12:02:32 +0000 Subject: refactor: batch 1 - correctness and error contract - MID/RIGHT/LEFT integer args now coerced at call sites (RIGHT(s, n/2) no longer emits invalid IR); drop emitLeft's private conversion - FOR loop direction decided at run time via select on STEP sign; descending FOR works, STEP evaluated once per iteration - ArrayHandler: extract emitCheckedIndices/loadElement; index count is validated before emission, fixing a compiler-crash (vector OOB UB) on OUTPUT with too many indices - Error contract: Lexer/Parser/CodeGen track errors; all silent failure paths now print Error: diagnostics with line numbers; unterminated char literal no longer truncates the rest of the program; module is verified with verifyModule; driver exits non-zero on any error Co-Authored-By: Claude Fable 5 --- include/cps/ArithmeticHandler.h | 5 +++-- 1 file changed, 3 insertions(+), 2 deletions(-) (limited to 'include/cps/ArithmeticHandler.h') diff --git a/include/cps/ArithmeticHandler.h b/include/cps/ArithmeticHandler.h index 8d6ea27..ca6266d 100644 --- a/include/cps/ArithmeticHandler.h +++ b/include/cps/ArithmeticHandler.h @@ -8,10 +8,11 @@ namespace cps { class ArithmeticHandler { llvm::LLVMContext &Context; llvm::IRBuilder<> &Builder; + bool &HadError; public: - ArithmeticHandler(llvm::LLVMContext &Ctx, llvm::IRBuilder<> &B) - : Context(Ctx), Builder(B) {} + ArithmeticHandler(llvm::LLVMContext &Ctx, llvm::IRBuilder<> &B, bool &HadErrorFlag) + : Context(Ctx), Builder(B), HadError(HadErrorFlag) {} llvm::Value *emitBinaryOp(int Op, llvm::Value *LHS, llvm::Value *RHS, int Line); }; -- cgit v1.3.1-4-g156e