From 1749b2b4a3e55db7c9bed95376f99b7f07932b49 Mon Sep 17 00:00:00 2001
From: _ov4 <ov4@ov4.org>
Date: Sun, 26 Jul 2026 12:15:33 +0000
Subject: refactor: drop per-type micro handlers and the parallel NamedValues
 map

Integer/Real/Boolean/CharHandler carried only five one-line LLVM API
wrappers (now inlined at their call sites in CodeGen.cc) plus dead
emitDeclare methods that violated the documented two-table lock-step
invariant. StringHandler::emitDeclare (also dead) and its NamedValues
reference go too.

With the dead writers gone, NamedValues held no information not already
in Symbols[].Storage, so the parallel map is removed: getNamedValue
reads Symbols, ArrayHandler/FunctionGen take one table, and function
bodies save/restore a single map. Behavior unchanged.

Co-Authored-By: Claude Fable 5 <noreply@anthropic.com>
---
 include/cps/ArrayHandler.h | 2 --
 1 file changed, 2 deletions(-)

(limited to 'include/cps/ArrayHandler.h')

diff --git a/include/cps/ArrayHandler.h b/include/cps/ArrayHandler.h
index e8a57fb..6e2c015 100644
--- a/include/cps/ArrayHandler.h
+++ b/include/cps/ArrayHandler.h
@@ -28,7 +28,6 @@ class ArrayHandler {
     llvm::LLVMContext *TheContext;
     llvm::IRBuilder<> *Builder;
     llvm::Module *TheModule;
-    std::map<std::string, llvm::Value*> *NamedValues;
     std::map<std::string, SymbolInfo> *Symbols;
     TypeSystem &Types;
 
@@ -57,7 +56,6 @@ public:
     ArrayHandler(llvm::LLVMContext &C,
                  llvm::IRBuilder<> &B,
                  llvm::Module &M,
-                 std::map<std::string, llvm::Value*> &NV,
                  std::map<std::string, SymbolInfo> &Sym,
                  TypeSystem &TS,
                  RuntimeCheck &RC);
-- 
cgit v1.3.1-4-g156e