From 1749b2b4a3e55db7c9bed95376f99b7f07932b49 Mon Sep 17 00:00:00 2001 From: _ov4 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 --- lib/CodeGen/IntegerHandler.cc | 16 ---------------- 1 file changed, 16 deletions(-) delete mode 100644 lib/CodeGen/IntegerHandler.cc (limited to 'lib/CodeGen/IntegerHandler.cc') diff --git a/lib/CodeGen/IntegerHandler.cc b/lib/CodeGen/IntegerHandler.cc deleted file mode 100644 index 8161593..0000000 --- a/lib/CodeGen/IntegerHandler.cc +++ /dev/null @@ -1,16 +0,0 @@ -#include "cps/IntegerHandler.h" - -using namespace llvm; -using namespace cps; - -void IntegerHandler::emitDeclare(const std::string &Name) { - Function *TheFunction = Builder.GetInsertBlock()->getParent(); - IRBuilder<> TmpB(&TheFunction->getEntryBlock(), TheFunction->getEntryBlock().begin()); - AllocaInst *Alloca = TmpB.CreateAlloca(Type::getInt64Ty(Context), nullptr, Name); - Builder.CreateStore(ConstantInt::get(Context, APInt(64, 0)), Alloca); - NamedValues[Name] = Alloca; -} - -Value *IntegerHandler::createLiteral(int64_t Val) { - return ConstantInt::get(Context, APInt(64, Val, true)); -} -- cgit v1.3.1-4-g156e