X-Git-Url: http://demsky.eecs.uci.edu/git/?a=blobdiff_plain;f=lib%2FTransforms%2FLevelRaise.cpp;h=9cbe3b813eba1620bd070563ea545a309dcbe76a;hb=7e6e44139429d1b8003426b29f14126b89591262;hp=1defda4d8453ab006116b9fed4a110e69d9f6aa5;hpb=3fb2ddd368828fbb73ebef98c5b88dc30822dca9;p=oota-llvm.git diff --git a/lib/Transforms/LevelRaise.cpp b/lib/Transforms/LevelRaise.cpp index 1defda4d845..9cbe3b813eb 100644 --- a/lib/Transforms/LevelRaise.cpp +++ b/lib/Transforms/LevelRaise.cpp @@ -1,4 +1,11 @@ -//===- LevelRaise.cpp - Code to change LLVM to higher level -----------------=// +//===- LevelRaise.cpp - Code to change LLVM to higher level ---------------===// +// +// The LLVM Compiler Infrastructure +// +// This file was developed by the LLVM research group and is distributed under +// the University of Illinois Open Source License. See LICENSE.TXT for details. +// +//===----------------------------------------------------------------------===// // // This file implements the 'raising' part of the LevelChange API. This is // useful because, in general, it makes the LLVM code terser and easier to @@ -6,26 +13,45 @@ // //===----------------------------------------------------------------------===// -#include "llvm/Transforms/LevelChange.h" +#include "llvm/Transforms/Scalar.h" #include "llvm/Transforms/Utils/Local.h" #include "TransformInternals.h" -#include "llvm/iOther.h" -#include "llvm/iMemory.h" +#include "llvm/Instructions.h" #include "llvm/Pass.h" -#include "llvm/ConstantHandling.h" -#include "llvm/Analysis/Expressions.h" #include "llvm/Transforms/Utils/BasicBlockUtils.h" -#include "Support/STLExtras.h" -#include "Support/StatisticReporter.h" +#include "llvm/Support/CommandLine.h" +#include "llvm/Support/Debug.h" +#include "llvm/ADT/Statistic.h" +#include "llvm/ADT/STLExtras.h" #include -using std::cerr; +#include +using namespace llvm; + +// StartInst - This enables the -raise-start-inst=foo option to cause the level +// raising pass to start at instruction "foo", which is immensely useful for +// debugging! +// +static cl::opt +StartInst("raise-start-inst", cl::Hidden, cl::value_desc("inst name"), + cl::desc("Start raise pass at the instruction with the specified name")); -static Statistic<> NumLoadStorePeepholes("raise\t\t- Number of load/store peepholes"); -static Statistic<> NumGEPInstFormed("raise\t\t- Number of other getelementptr's formed"); -static Statistic<> NumExprTreesConv("raise\t\t- Number of expression trees converted"); -static Statistic<> NumCastOfCast("raise\t\t- Number of cast-of-self removed"); -static Statistic<> NumDCEorCP("raise\t\t- Number of insts DCE'd or constprop'd"); +static Statistic<> +NumLoadStorePeepholes("raise", "Number of load/store peepholes"); +static Statistic<> +NumGEPInstFormed("raise", "Number of other getelementptr's formed"); + +static Statistic<> +NumExprTreesConv("raise", "Number of expression trees converted"); + +static Statistic<> +NumCastOfCast("raise", "Number of cast-of-self removed"); + +static Statistic<> +NumDCEorCP("raise", "Number of insts DCEd or constprop'd"); + +static Statistic<> +NumVarargCallChanges("raise", "Number of vararg call peepholes"); #define PRINT_PEEPHOLE(ID, NUM, I) \ DEBUG(std::cerr << "Inst P/H " << ID << "[" << NUM << "] " << I) @@ -40,141 +66,43 @@ static Statistic<> NumDCEorCP("raise\t\t- Number of insts DCE'd or constprop'd") do { PRINT_PEEPHOLE(ID, 0, I1); PRINT_PEEPHOLE(ID, 1, I2); \ PRINT_PEEPHOLE(ID, 2, I3); PRINT_PEEPHOLE(ID, 3, I4); } while (0) +namespace { + struct RPR : public FunctionPass { + virtual bool runOnFunction(Function &F); -// isReinterpretingCast - Return true if the cast instruction specified will -// cause the operand to be "reinterpreted". A value is reinterpreted if the -// cast instruction would cause the underlying bits to change. -// -static inline bool isReinterpretingCast(const CastInst *CI) { - return!CI->getOperand(0)->getType()->isLosslesslyConvertableTo(CI->getType()); -} - - -// Peephole optimize the following instructions: -// %t1 = cast ? to x * -// %t2 = add x * %SP, %t1 ;; Constant must be 2nd operand -// -// Into: %t3 = getelementptr {<...>} * %SP, -// %t2 = cast * %t3 to {<...>}* -// -static bool HandleCastToPointer(BasicBlock::iterator BI, - const PointerType *DestPTy) { - CastInst &CI = cast(*BI); - if (CI.use_empty()) return false; - - // Scan all of the uses, looking for any uses that are not add - // instructions. If we have non-adds, do not make this transformation. - // - for (Value::use_iterator I = CI.use_begin(), E = CI.use_end(); - I != E; ++I) { - if (BinaryOperator *BO = dyn_cast(*I)) { - if (BO->getOpcode() != Instruction::Add || - // Avoid add sbyte* %X, %X cases... - BO->getOperand(0) == BO->getOperand(1)) - return false; - } else { - return false; + virtual void getAnalysisUsage(AnalysisUsage &AU) const { + AU.setPreservesCFG(); + AU.addRequired(); } - } - - std::vector Indices; - Value *Src = CI.getOperand(0); - const Type *Result = ConvertableToGEP(DestPTy, Src, Indices, &BI); - if (Result == 0) return false; // Not convertable... - - PRINT_PEEPHOLE2("cast-add-to-gep:in", Src, CI); - - // If we have a getelementptr capability... transform all of the - // add instruction uses into getelementptr's. - while (!CI.use_empty()) { - BinaryOperator *I = cast(*CI.use_begin()); - assert(I->getOpcode() == Instruction::Add && I->getNumOperands() == 2 && - "Use is not a valid add instruction!"); - - // Get the value added to the cast result pointer... - Value *OtherPtr = I->getOperand((I->getOperand(0) == &CI) ? 1 : 0); - - Instruction *GEP = new GetElementPtrInst(OtherPtr, Indices, I->getName()); - PRINT_PEEPHOLE1("cast-add-to-gep:i", I); - - if (GEP->getType() == I->getType()) { - // Replace the old add instruction with the shiny new GEP inst - ReplaceInstWithInst(I, GEP); - } else { - // If the type produced by the gep instruction differs from the original - // add instruction type, insert a cast now. - // - - // Insert the GEP instruction before the old add instruction... - I->getParent()->getInstList().insert(I, GEP); - PRINT_PEEPHOLE1("cast-add-to-gep:o", GEP); - GEP = new CastInst(GEP, I->getType()); - - // Replace the old add instruction with the shiny new GEP inst - ReplaceInstWithInst(I, GEP); - } + private: + bool DoRaisePass(Function &F); + bool PeepholeOptimize(BasicBlock *BB, BasicBlock::iterator &BI); + }; - PRINT_PEEPHOLE1("cast-add-to-gep:o", GEP); - } - return true; + RegisterOpt X("raise", "Raise Pointer References"); } -// Peephole optimize the following instructions: -// %t1 = cast ulong to {<...>} * -// %t2 = add {<...>} * %SP, %t1 ;; Constant must be 2nd operand -// -// or -// %t1 = cast {<...>}* %SP to int* -// %t5 = cast ulong to int* -// %t2 = add int* %t1, %t5 ;; int is same size as field -// -// Into: %t3 = getelementptr {<...>} * %SP, -// %t2 = cast * %t3 to {<...>}* -// -static bool PeepholeOptimizeAddCast(BasicBlock *BB, BasicBlock::iterator &BI, - Value *AddOp1, CastInst *AddOp2) { - const CompositeType *CompTy; - Value *OffsetVal = AddOp2->getOperand(0); - Value *SrcPtr; // Of type pointer to struct... - - if ((CompTy = getPointedToComposite(AddOp1->getType()))) { - SrcPtr = AddOp1; // Handle the first case... - } else if (CastInst *AddOp1c = dyn_cast(AddOp1)) { - SrcPtr = AddOp1c->getOperand(0); // Handle the second case... - CompTy = getPointedToComposite(SrcPtr->getType()); - } - - // Only proceed if we have detected all of our conditions successfully... - if (!CompTy || !SrcPtr || !OffsetVal->getType()->isIntegral()) - return false; - - std::vector Indices; - if (!ConvertableToGEP(SrcPtr->getType(), OffsetVal, Indices, &BI)) - return false; // Not convertable... perhaps next time - if (getPointedToComposite(AddOp1->getType())) { // case 1 - PRINT_PEEPHOLE2("add-to-gep1:in", AddOp2, *BI); - } else { - PRINT_PEEPHOLE3("add-to-gep2:in", AddOp1, AddOp2, *BI); - } +FunctionPass *llvm::createRaisePointerReferencesPass() { + return new RPR(); +} - GetElementPtrInst *GEP = new GetElementPtrInst(SrcPtr, Indices, - AddOp2->getName()); - BI = ++BB->getInstList().insert(BI, GEP); - Instruction *NCI = new CastInst(GEP, AddOp1->getType()); - ReplaceInstWithInst(BB->getInstList(), BI, NCI); - PRINT_PEEPHOLE2("add-to-gep:out", GEP, NCI); - return true; +// isReinterpretingCast - Return true if the cast instruction specified will +// cause the operand to be "reinterpreted". A value is reinterpreted if the +// cast instruction would cause the underlying bits to change. +// +static inline bool isReinterpretingCast(const CastInst *CI) { + return!CI->getOperand(0)->getType()->isLosslesslyConvertibleTo(CI->getType()); } -static bool PeepholeOptimize(BasicBlock *BB, BasicBlock::iterator &BI) { +bool RPR::PeepholeOptimize(BasicBlock *BB, BasicBlock::iterator &BI) { Instruction *I = BI; + const TargetData &TD = getAnalysis(); if (CastInst *CI = dyn_cast(I)) { Value *Src = CI->getOperand(0); - Instruction *SrcI = dyn_cast(Src); // Nonnull if instr source const Type *DestTy = CI->getType(); // Peephole optimize the following instruction: @@ -183,12 +111,12 @@ static bool PeepholeOptimize(BasicBlock *BB, BasicBlock::iterator &BI) { // Into: // if (DestTy == Src->getType()) { // Check for a cast to same type as src!! - PRINT_PEEPHOLE1("cast-of-self-ty", CI); + PRINT_PEEPHOLE1("cast-of-self-ty", *CI); CI->replaceAllUsesWith(Src); if (!Src->hasName() && CI->hasName()) { std::string Name = CI->getName(); CI->setName(""); - Src->setName(Name, BB->getParent()->getSymbolTable()); + Src->setName(Name); } // DCE the instruction now, to avoid having the iterative version of DCE @@ -209,18 +137,23 @@ static bool PeepholeOptimize(BasicBlock *BB, BasicBlock::iterator &BI) { // destination type of the cast... // ConvertedTypes[CI] = CI->getType(); // Make sure the cast doesn't change - if (ExpressionConvertableToType(Src, DestTy, ConvertedTypes)) { - PRINT_PEEPHOLE3("CAST-SRC-EXPR-CONV:in ", Src, CI, BB->getParent()); - - DEBUG(cerr << "\nCONVERTING SRC EXPR TYPE:\n"); - ValueMapCache ValueMap; - Value *E = ConvertExpressionToType(Src, DestTy, ValueMap); - if (Constant *CPV = dyn_cast(E)) - CI->replaceAllUsesWith(CPV); + if (ExpressionConvertibleToType(Src, DestTy, ConvertedTypes, TD)) { + PRINT_PEEPHOLE3("CAST-SRC-EXPR-CONV:in ", *Src, *CI, *BB->getParent()); + + DEBUG(std::cerr << "\nCONVERTING SRC EXPR TYPE:\n"); + { // ValueMap must be destroyed before function verified! + ValueMapCache ValueMap; + Value *E = ConvertExpressionToType(Src, DestTy, ValueMap, TD); + + if (Constant *CPV = dyn_cast(E)) + CI->replaceAllUsesWith(CPV); + + PRINT_PEEPHOLE1("CAST-SRC-EXPR-CONV:out", *E); + DEBUG(std::cerr << "DONE CONVERTING SRC EXPR TYPE: \n" + << *BB->getParent()); + } BI = BB->begin(); // Rescan basic block. BI might be invalidated. - PRINT_PEEPHOLE1("CAST-SRC-EXPR-CONV:out", E); - DEBUG(cerr << "DONE CONVERTING SRC EXPR TYPE: \n" << BB->getParent()); ++NumExprTreesConv; return true; } @@ -229,34 +162,28 @@ static bool PeepholeOptimize(BasicBlock *BB, BasicBlock::iterator &BI) { // source type of the cast... // ConvertedTypes.clear(); - ConvertedTypes[Src] = Src->getType(); // Make sure the source doesn't change type - if (ValueConvertableToType(CI, Src->getType(), ConvertedTypes)) { - PRINT_PEEPHOLE3("CAST-DEST-EXPR-CONV:in ", Src, CI, BB->getParent()); + // Make sure the source doesn't change type + ConvertedTypes[Src] = Src->getType(); + if (ValueConvertibleToType(CI, Src->getType(), ConvertedTypes, TD)) { + //PRINT_PEEPHOLE3("CAST-DEST-EXPR-CONV:in ", *Src, *CI, + // *BB->getParent()); + + DEBUG(std::cerr << "\nCONVERTING EXPR TYPE:\n"); + { // ValueMap must be destroyed before function verified! + ValueMapCache ValueMap; + ConvertValueToNewType(CI, Src, ValueMap, TD); // This will delete CI! + } - DEBUG(cerr << "\nCONVERTING EXPR TYPE:\n"); - ValueMapCache ValueMap; - ConvertValueToNewType(CI, Src, ValueMap); // This will delete CI! + PRINT_PEEPHOLE1("CAST-DEST-EXPR-CONV:out", *Src); + DEBUG(std::cerr << "DONE CONVERTING EXPR TYPE: \n\n" << + *BB->getParent()); BI = BB->begin(); // Rescan basic block. BI might be invalidated. - PRINT_PEEPHOLE1("CAST-DEST-EXPR-CONV:out", Src); - DEBUG(cerr << "DONE CONVERTING EXPR TYPE: \n\n" << BB->getParent()); ++NumExprTreesConv; return true; } } - // Otherwise find out it this cast is a cast to a pointer type, which is - // then added to some other pointer, then loaded or stored through. If - // so, convert the add into a getelementptr instruction... - // - if (const PointerType *DestPTy = dyn_cast(DestTy)) { - if (HandleCastToPointer(BI, DestPTy)) { - BI = BB->begin(); // Rescan basic block. BI might be invalidated. - ++NumGEPInstFormed; - return true; - } - } - // Check to see if we are casting from a structure pointer to a pointer to // the first element of the structure... to avoid munching other peepholes, // we only let this happen if there are no add uses of the cast. @@ -283,7 +210,7 @@ static bool PeepholeOptimize(BasicBlock *BB, BasicBlock::iterator &BI) { } // If it doesn't have an add use, check to see if the dest type is - // losslessly convertable to one of the types in the start of the struct + // losslessly convertible to one of the types in the start of the struct // type. // if (!HasAddUse) { @@ -294,48 +221,49 @@ static bool PeepholeOptimize(BasicBlock *BB, BasicBlock::iterator &BI) { // Build the index vector, full of all zeros std::vector Indices; - Indices.push_back(ConstantUInt::get(Type::UIntTy, 0)); + + Indices.push_back(Constant::getNullValue(Type::UIntTy)); while (CurCTy && !isa(CurCTy)) { if (const StructType *CurSTy = dyn_cast(CurCTy)) { // Check for a zero element struct type... if we have one, bail. - if (CurSTy->getElementTypes().size() == 0) break; - + if (CurSTy->getNumElements() == 0) break; + // Grab the first element of the struct type, which must lie at // offset zero in the struct. // - ElTy = CurSTy->getElementTypes()[0]; + ElTy = CurSTy->getElementType(0); } else { - ElTy = cast(CurCTy)->getElementType(); + ElTy = cast(CurCTy)->getElementType(); } // Insert a zero to index through this type... - Indices.push_back(ConstantUInt::get(CurCTy->getIndexType(), 0)); + Indices.push_back(Constant::getNullValue(Type::UIntTy)); // Did we find what we're looking for? - if (ElTy->isLosslesslyConvertableTo(DestPointedTy)) break; - + if (ElTy->isLosslesslyConvertibleTo(DestPointedTy)) break; + // Nope, go a level deeper. ++Depth; CurCTy = dyn_cast(ElTy); ElTy = 0; } - + // Did we find what we were looking for? If so, do the transformation if (ElTy) { - PRINT_PEEPHOLE1("cast-for-first:in", CI); + PRINT_PEEPHOLE1("cast-for-first:in", *CI); + + std::string Name = CI->getName(); CI->setName(""); // Insert the new T cast instruction... stealing old T's name GetElementPtrInst *GEP = new GetElementPtrInst(Src, Indices, - CI->getName()); - CI->setName(""); - BI = ++BB->getInstList().insert(BI, GEP); + Name, BI); // Make the old cast instruction reference the new GEP instead of // the old src value. // CI->setOperand(0, GEP); - - PRINT_PEEPHOLE2("cast-for-first:out", GEP, CI); + + PRINT_PEEPHOLE2("cast-for-first:out", *GEP, *CI); ++NumGEPInstFormed; return true; } @@ -345,12 +273,12 @@ static bool PeepholeOptimize(BasicBlock *BB, BasicBlock::iterator &BI) { } else if (StoreInst *SI = dyn_cast(I)) { Value *Val = SI->getOperand(0); Value *Pointer = SI->getPointerOperand(); - + // Peephole optimize the following instructions: - // %t = cast * %P to * ;; If T1 is losslessly convertable to T2 + // %t = cast * %P to * ;; If T1 is losslessly convertible to T2 // store %V, * %t // - // Into: + // Into: // %t = cast %V to // store %t2, * %P // @@ -362,21 +290,19 @@ static bool PeepholeOptimize(BasicBlock *BB, BasicBlock::iterator &BI) { if (CastInst *CI = dyn_cast(Pointer)) if (Value *CastSrc = CI->getOperand(0)) // CSPT = CastSrcPointerType if (const PointerType *CSPT = dyn_cast(CastSrc->getType())) - // convertable types? - if (Val->getType()->isLosslesslyConvertableTo(CSPT->getElementType()) && - !SI->hasIndices()) { // No subscripts yet! - PRINT_PEEPHOLE3("st-src-cast:in ", Pointer, Val, SI); + // convertible types? + if (Val->getType()->isLosslesslyConvertibleTo(CSPT->getElementType())) { + PRINT_PEEPHOLE3("st-src-cast:in ", *Pointer, *Val, *SI); // Insert the new T cast instruction... stealing old T's name + std::string Name(CI->getName()); CI->setName(""); CastInst *NCI = new CastInst(Val, CSPT->getElementType(), - CI->getName()); - CI->setName(""); - BI = ++BB->getInstList().insert(BI, NCI); + Name, BI); // Replace the old store with a new one! ReplaceInstWithInst(BB->getInstList(), BI, SI = new StoreInst(NCI, CastSrc)); - PRINT_PEEPHOLE3("st-src-cast:out", NCI, CastSrc, SI); + PRINT_PEEPHOLE3("st-src-cast:out", *NCI, *CastSrc, *SI); ++NumLoadStorePeepholes; return true; } @@ -385,12 +311,12 @@ static bool PeepholeOptimize(BasicBlock *BB, BasicBlock::iterator &BI) { Value *Pointer = LI->getOperand(0); const Type *PtrElType = cast(Pointer->getType())->getElementType(); - + // Peephole optimize the following instructions: - // %Val = cast * to * ;; If T1 is losslessly convertable to T2 + // %Val = cast * to * ;; If T1 is losslessly convertible to T2 // %t = load * %P // - // Into: + // Into: // %t = load * %P // %Val = cast to // @@ -402,33 +328,68 @@ static bool PeepholeOptimize(BasicBlock *BB, BasicBlock::iterator &BI) { if (CastInst *CI = dyn_cast(Pointer)) if (Value *CastSrc = CI->getOperand(0)) // CSPT = CastSrcPointerType if (const PointerType *CSPT = dyn_cast(CastSrc->getType())) - // convertable types? - if (PtrElType->isLosslesslyConvertableTo(CSPT->getElementType()) && - !LI->hasIndices()) { // No subscripts yet! - PRINT_PEEPHOLE2("load-src-cast:in ", Pointer, LI); + // convertible types? + if (PtrElType->isLosslesslyConvertibleTo(CSPT->getElementType())) { + PRINT_PEEPHOLE2("load-src-cast:in ", *Pointer, *LI); // Create the new load instruction... loading the pre-casted value - LoadInst *NewLI = new LoadInst(CastSrc, LI->getName()); - + LoadInst *NewLI = new LoadInst(CastSrc, LI->getName(), BI); + // Insert the new T cast instruction... stealing old T's name CastInst *NCI = new CastInst(NewLI, LI->getType(), CI->getName()); - BI = ++BB->getInstList().insert(BI, NewLI); // Replace the old store with a new one! ReplaceInstWithInst(BB->getInstList(), BI, NCI); - PRINT_PEEPHOLE3("load-src-cast:out", NCI, CastSrc, NewLI); + PRINT_PEEPHOLE3("load-src-cast:out", *NCI, *CastSrc, *NewLI); ++NumLoadStorePeepholes; return true; } - } else if (I->getOpcode() == Instruction::Add && - isa(I->getOperand(1))) { - - if (PeepholeOptimizeAddCast(BB, BI, I->getOperand(0), - cast(I->getOperand(1)))) { - ++NumGEPInstFormed; + } else if (CallInst *CI = dyn_cast(I)) { + // If we have a call with all varargs arguments, convert the call to use the + // actual argument types present... + // + const PointerType *PTy = cast(CI->getCalledValue()->getType()); + const FunctionType *FTy = cast(PTy->getElementType()); + + // Is the call to a vararg variable with no real parameters? + if (FTy->isVarArg() && FTy->getNumParams() == 0 && + !CI->getCalledFunction()) { + // If so, insert a new cast instruction, casting it to a function type + // that matches the current arguments... + // + std::vector Params; // Parameter types... + for (unsigned i = 1, e = CI->getNumOperands(); i != e; ++i) + Params.push_back(CI->getOperand(i)->getType()); + + FunctionType *NewFT = FunctionType::get(FTy->getReturnType(), + Params, false); + PointerType *NewPFunTy = PointerType::get(NewFT); + + // Create a new cast, inserting it right before the function call... + Value *NewCast; + Constant *ConstantCallSrc = 0; + if (Constant *CS = dyn_cast(CI->getCalledValue())) + ConstantCallSrc = CS; + + if (ConstantCallSrc) + NewCast = ConstantExpr::getCast(ConstantCallSrc, NewPFunTy); + else + NewCast = new CastInst(CI->getCalledValue(), NewPFunTy, + CI->getCalledValue()->getName()+"_c",CI); + + // Create a new call instruction... + CallInst *NewCall = new CallInst(NewCast, + std::vector(CI->op_begin()+1, CI->op_end())); + if (CI->isTailCall()) NewCall->setTailCall(); + NewCall->setCallingConv(CI->getCallingConv()); + ++BI; + ReplaceInstWithInst(CI, NewCall); + + ++NumVarargCallChanges; return true; } + } return false; @@ -437,15 +398,15 @@ static bool PeepholeOptimize(BasicBlock *BB, BasicBlock::iterator &BI) { -static bool DoRaisePass(Function &F) { +bool RPR::DoRaisePass(Function &F) { bool Changed = false; for (Function::iterator BB = F.begin(), BBE = F.end(); BB != BBE; ++BB) for (BasicBlock::iterator BI = BB->begin(); BI != BB->end();) { - DEBUG(cerr << "Processing: " << *BI); - if (dceInstruction(BI) || doConstantPropogation(BI)) { - Changed = true; + DEBUG(std::cerr << "LevelRaising: " << *BI); + if (dceInstruction(BI) || doConstantPropagation(BI)) { + Changed = true; ++NumDCEorCP; - DEBUG(cerr << "***\t\t^^-- DeadCode Elinated!\n"); + DEBUG(std::cerr << "***\t\t^^-- Dead code eliminated!\n"); } else if (PeepholeOptimize(BB, BI)) { Changed = true; } else { @@ -457,19 +418,32 @@ static bool DoRaisePass(Function &F) { } -// RaisePointerReferences::doit - Raise a function representation to a higher -// level. -// -static bool doRPR(Function &F) { - DEBUG(cerr << "\n\n\nStarting to work on Function '" << F.getName() << "'\n"); +// runOnFunction - Raise a function representation to a higher level. +bool RPR::runOnFunction(Function &F) { + DEBUG(std::cerr << "\n\n\nStarting to work on Function '" << F.getName() + << "'\n"); // Insert casts for all incoming pointer pointer values that are treated as // arrays... // bool Changed = false, LocalChange; - + + // If the StartInst option was specified, then Peephole optimize that + // instruction first if it occurs in this function. + // + if (!StartInst.empty()) { + for (Function::iterator BB = F.begin(), BBE = F.end(); BB != BBE; ++BB) + for (BasicBlock::iterator BI = BB->begin(); BI != BB->end(); ++BI) + if (BI->getName() == StartInst) { + bool SavedDebug = DebugFlag; // Save the DEBUG() controlling flag. + DebugFlag = true; // Turn on DEBUG's + Changed |= PeepholeOptimize(BB, BI); + DebugFlag = SavedDebug; // Restore DebugFlag to previous state + } + } + do { - DEBUG(cerr << "Looping: \n" << F); + DEBUG(std::cerr << "Looping: \n" << F); // Iterate over the function, refining it, until it converges on a stable // state @@ -482,20 +456,3 @@ static bool doRPR(Function &F) { return Changed; } -namespace { - struct RaisePointerReferences : public FunctionPass { - const char *getPassName() const { return "Raise Pointer References"; } - - virtual bool runOnFunction(Function &F) { return doRPR(F); } - - virtual void getAnalysisUsage(AnalysisUsage &AU) const { - AU.preservesCFG(); - } - }; -} - -Pass *createRaisePointerReferencesPass() { - return new RaisePointerReferences(); -} - -