From bc61e661bd55343dac22c08c6c7be8855adc44c9 Mon Sep 17 00:00:00 2001 From: Chris Lattner Date: Sun, 2 Nov 2003 05:57:39 +0000 Subject: [PATCH] Implement transmogriphication of allocation instructions git-svn-id: https://llvm.org/svn/llvm-project/llvm/trunk@9654 91177308-0d34-0410-b5e6-96231b3b80d8 --- .../Scalar/InstructionCombining.cpp | 35 +++++++++++++++++-- 1 file changed, 33 insertions(+), 2 deletions(-) diff --git a/lib/Transforms/Scalar/InstructionCombining.cpp b/lib/Transforms/Scalar/InstructionCombining.cpp index bc1a81967f4..d5da0b3b54a 100644 --- a/lib/Transforms/Scalar/InstructionCombining.cpp +++ b/lib/Transforms/Scalar/InstructionCombining.cpp @@ -34,14 +34,15 @@ //===----------------------------------------------------------------------===// #include "llvm/Transforms/Scalar.h" -#include "llvm/Transforms/Utils/BasicBlockUtils.h" -#include "llvm/Transforms/Utils/Local.h" #include "llvm/Instructions.h" #include "llvm/Pass.h" #include "llvm/Constants.h" #include "llvm/ConstantHandling.h" #include "llvm/DerivedTypes.h" #include "llvm/GlobalVariable.h" +#include "llvm/Target/TargetData.h" +#include "llvm/Transforms/Utils/BasicBlockUtils.h" +#include "llvm/Transforms/Utils/Local.h" #include "llvm/Support/InstIterator.h" #include "llvm/Support/InstVisitor.h" #include "llvm/Support/CallSite.h" @@ -57,6 +58,7 @@ namespace { public InstVisitor { // Worklist of all of the instructions that need to be simplified. std::vector WorkList; + TargetData *TD; void AddUsesToWorkList(Instruction &I) { // The instruction was simplified, add all users of the instruction to @@ -73,6 +75,7 @@ namespace { virtual bool runOnFunction(Function &F); virtual void getAnalysisUsage(AnalysisUsage &AU) const { + AU.addRequired(); AU.setPreservesCFG(); } @@ -1536,6 +1539,33 @@ Instruction *InstCombiner::visitCastInst(CastInst &CI) { } } + // If we are casting a malloc or alloca to a pointer to a type of the same + // size, rewrite the allocation instruction to allocate the "right" type. + // + if (AllocationInst *AI = dyn_cast(Src)) + if (AI->hasOneUse()) + if (const PointerType *PTy = dyn_cast(CI.getType())) { + // Get the type really allocated and the type casted to... + const Type *AllocElTy = AI->getAllocatedType(); + unsigned AllocElTySize = TD->getTypeSize(AllocElTy); + const Type *CastElTy = PTy->getElementType(); + unsigned CastElTySize = TD->getTypeSize(CastElTy); + + // If the allocation is for an even multiple of the cast type size + if (AllocElTySize % CastElTySize == 0) { + Value *Amt = ConstantUInt::get(Type::UIntTy, + AllocElTySize/CastElTySize); + std::string Name = AI->getName(); AI->setName(""); + AllocationInst *New; + if (isa(AI)) + New = new MallocInst(CastElTy, Amt, Name); + else + New = new AllocaInst(CastElTy, Amt, Name); + InsertNewInstBefore(New, CI); + return ReplaceInstUsesWith(CI, New); + } + } + // If the source value is an instruction with only this use, we can attempt to // propagate the cast into the instruction. Also, only handle integral types // for now. @@ -1975,6 +2005,7 @@ void InstCombiner::removeFromWorkList(Instruction *I) { bool InstCombiner::runOnFunction(Function &F) { bool Changed = false; + TD = &getAnalysis(); WorkList.insert(WorkList.end(), inst_begin(F), inst_end(F)); -- 2.34.1