From be755279f37c02c50355780ea71b0b23d566a985 Mon Sep 17 00:00:00 2001 From: Misha Brukman Date: Mon, 17 Nov 2003 20:40:07 +0000 Subject: Emit the MachineConstantPool constants in one contiguous memory `pool'. llvm-svn: 10060 --- llvm/lib/ExecutionEngine/JIT/Emitter.cpp | 25 ++++++++++++++++++++----- 1 file changed, 20 insertions(+), 5 deletions(-) (limited to 'llvm/lib/ExecutionEngine/JIT/Emitter.cpp') diff --git a/llvm/lib/ExecutionEngine/JIT/Emitter.cpp b/llvm/lib/ExecutionEngine/JIT/Emitter.cpp index 32d0651f226..be60b239a3e 100644 --- a/llvm/lib/ExecutionEngine/JIT/Emitter.cpp +++ b/llvm/lib/ExecutionEngine/JIT/Emitter.cpp @@ -187,13 +187,28 @@ void Emitter::finishFunction(MachineFunction &F) { void Emitter::emitConstantPool(MachineConstantPool *MCP) { const std::vector &Constants = MCP->getConstants(); + if (Constants.size() == 0) return; + + std::vector ConstantSizes; + unsigned TotalSize = 0; + // Calculate how much space we will need for all the constants for (unsigned i = 0, e = Constants.size(); i != e; ++i) { - // For now we just allocate some memory on the heap, this can be - // dramatically improved. const Type *Ty = ((Value*)Constants[i])->getType(); - void *Addr = malloc(TheVM->getTargetData().getTypeSize(Ty)); - TheVM->InitializeMemory(Constants[i], Addr); - ConstantPoolAddresses.push_back(Addr); + unsigned TySize = TheVM->getTargetData().getTypeSize(Ty); + ConstantSizes.push_back(TySize); + TotalSize += TySize; + } + // Allocate a 'pool' of memory just once + void *ConstPool = malloc(TotalSize); + if (!ConstPool) { + perror("malloc"); + abort(); + } + // Initialize each slot in the 'pool' appropriately + for (unsigned i = 0, e = Constants.size(); i != e; ++i) { + TheVM->InitializeMemory(Constants[i], ConstPool); + ConstantPoolAddresses.push_back(ConstPool); + ConstPool = (void*) ((intptr_t)ConstPool + ConstantSizes[i]); } } -- cgit v1.2.3