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>

1 files changed, 0 insertions, 24 deletions

diff --git a/include/cps/IntegerHandler.h b/include/cps/IntegerHandler.h
deleted file mode 100644
index 2cd0bbc..0000000
--- a/include/cps/IntegerHandler.h
+++ /dev/null
@@ -1,24 +0,0 @@
-#pragma once
-#include "llvm/IR/IRBuilder.h"
-#include "llvm/IR/LLVMContext.h"
-#include "llvm/IR/Module.h"
-#include <map>
-#include <string>
-
-namespace cps {
-
-class IntegerHandler {
-    llvm::LLVMContext &Context;
-    llvm::IRBuilder<> &Builder;
-    llvm::Module &Module;
-    std::map<std::string, llvm::Value*> &NamedValues;
-
-public:
-    IntegerHandler(llvm::LLVMContext &Ctx, llvm::IRBuilder<> &B, llvm::Module &M, std::map<std::string, llvm::Value*> &NV)
-        : Context(Ctx), Builder(B), Module(M), NamedValues(NV) {}
-
-    void emitDeclare(const std::string &Name);
-    llvm::Value *createLiteral(int64_t Val);
-};
-
-}