lib/Transforms/LevelRaise.cpp

   1 //===- LevelRaise.cpp - Code to change LLVM to higher level -----------------=//
   2 //
   3 // This file implements the 'raising' part of the LevelChange API.  This is
   4 // useful because, in general, it makes the LLVM code terser and easier to
   5 // analyze.
   6 //
   7 //===----------------------------------------------------------------------===//
   8
   9 #include "llvm/Transforms/RaisePointerReferences.h"
  10 #include "llvm/Transforms/Utils/Local.h"
  11 #include "TransformInternals.h"
  12 #include "llvm/iOther.h"
  13 #include "llvm/iMemory.h"
  14 #include "llvm/Pass.h"
  15 #include "llvm/ConstantHandling.h"
  16 #include "llvm/Analysis/Expressions.h"
  17 #include "llvm/Analysis/Verifier.h"
  18 #include "llvm/Transforms/Utils/BasicBlockUtils.h"
  19 #include "Support/STLExtras.h"
  20 #include "Support/Statistic.h"
  21 #include "Support/CommandLine.h"
  22 #include <algorithm>
  23 using std::cerr;
  24
  25 // StartInst - This enables the -raise-start-inst=foo option to cause the level
  26 // raising pass to start at instruction "foo", which is immensely useful for
  27 // debugging!
  28 //
  29 static cl::opt<std::string>
  30 StartInst("raise-start-inst", cl::Hidden, cl::value_desc("inst name"),
  31        cl::desc("Start raise pass at the instruction with the specified name"));
  32
  33 static Statistic<>
  34 NumLoadStorePeepholes("raise", "Number of load/store peepholes");
  35
  36 static Statistic<>
  37 NumGEPInstFormed("raise", "Number of other getelementptr's formed");
  38
  39 static Statistic<>
  40 NumExprTreesConv("raise", "Number of expression trees converted");
  41
  42 static Statistic<>
  43 NumCastOfCast("raise", "Number of cast-of-self removed");
  44
  45 static Statistic<>
  46 NumDCEorCP("raise", "Number of insts DCEd or constprop'd");
  47
  48 static Statistic<>
  49 NumVarargCallChanges("raise", "Number of vararg call peepholes");
  50
  51
  52 #define PRINT_PEEPHOLE(ID, NUM, I)            \
  53   DEBUG(std::cerr << "Inst P/H " << ID << "[" << NUM << "] " << I)
  54
  55 #define PRINT_PEEPHOLE1(ID, I1) do { PRINT_PEEPHOLE(ID, 0, I1); } while (0)
  56 #define PRINT_PEEPHOLE2(ID, I1, I2) \
  57   do { PRINT_PEEPHOLE(ID, 0, I1); PRINT_PEEPHOLE(ID, 1, I2); } while (0)
  58 #define PRINT_PEEPHOLE3(ID, I1, I2, I3) \
  59   do { PRINT_PEEPHOLE(ID, 0, I1); PRINT_PEEPHOLE(ID, 1, I2); \
  60        PRINT_PEEPHOLE(ID, 2, I3); } while (0)
  61 #define PRINT_PEEPHOLE4(ID, I1, I2, I3, I4) \
  62   do { PRINT_PEEPHOLE(ID, 0, I1); PRINT_PEEPHOLE(ID, 1, I2); \
  63        PRINT_PEEPHOLE(ID, 2, I3); PRINT_PEEPHOLE(ID, 3, I4); } while (0)
  64
  65 namespace {
  66   struct RPR : public FunctionPass {
  67     virtual bool runOnFunction(Function &F);
  68
  69     virtual void getAnalysisUsage(AnalysisUsage &AU) const {
  70       AU.setPreservesCFG();
  71       AU.addRequired<TargetData>();
  72     }
  73
  74   private:
  75     bool DoRaisePass(Function &F);
  76     bool PeepholeOptimize(BasicBlock *BB, BasicBlock::iterator &BI);
  77   };
  78
  79   RegisterOpt<RPR> X("raise", "Raise Pointer References");
  80 }
  81
  82 Pass *createRaisePointerReferencesPass() {
  83   return new RPR();
  84 }
  85
  86
  87
  88 // isReinterpretingCast - Return true if the cast instruction specified will
  89 // cause the operand to be "reinterpreted".  A value is reinterpreted if the
  90 // cast instruction would cause the underlying bits to change.
  91 //
  92 static inline bool isReinterpretingCast(const CastInst *CI) {
  93   return!CI->getOperand(0)->getType()->isLosslesslyConvertableTo(CI->getType());
  94 }
  95
  96
  97 // Peephole optimize the following instructions:
  98 // %t1 = cast ? to x *
  99 // %t2 = add x * %SP, %t1              ;; Constant must be 2nd operand
 100 //
 101 // Into: %t3 = getelementptr {<...>} * %SP, <element indices>
 102 //       %t2 = cast <eltype> * %t3 to {<...>}*
 103 //
 104 static bool HandleCastToPointer(BasicBlock::iterator BI,
 105                                 const PointerType *DestPTy,
 106                                 const TargetData &TD) {
 107   CastInst &CI = cast<CastInst>(*BI);
 108   if (CI.use_empty()) return false;
 109
 110   // Scan all of the uses, looking for any uses that are not add or sub
 111   // instructions.  If we have non-adds, do not make this transformation.
 112   //
 113   bool HasSubUse = false;  // Keep track of any subtracts...
 114   for (Value::use_iterator I = CI.use_begin(), E = CI.use_end();
 115        I != E; ++I)
 116     if (BinaryOperator *BO = dyn_cast<BinaryOperator>(*I)) {
 117       if ((BO->getOpcode() != Instruction::Add &&
 118            BO->getOpcode() != Instruction::Sub) ||
 119           // Avoid add sbyte* %X, %X cases...
 120           BO->getOperand(0) == BO->getOperand(1))
 121         return false;
 122       else
 123         HasSubUse |= BO->getOpcode() == Instruction::Sub;
 124     } else {
 125       return false;
 126     }
 127
 128   std::vector<Value*> Indices;
 129   Value *Src = CI.getOperand(0);
 130   const Type *Result = ConvertableToGEP(DestPTy, Src, Indices, TD, &BI);
 131   if (Result == 0) return false;  // Not convertable...
 132
 133   // Cannot handle subtracts if there is more than one index required...
 134   if (HasSubUse && Indices.size() != 1) return false;
 135
 136   PRINT_PEEPHOLE2("cast-add-to-gep:in", Src, CI);
 137
 138   // If we have a getelementptr capability... transform all of the
 139   // add instruction uses into getelementptr's.
 140   while (!CI.use_empty()) {
 141     BinaryOperator *I = cast<BinaryOperator>(*CI.use_begin());
 142     assert((I->getOpcode() == Instruction::Add ||
 143             I->getOpcode() == Instruction::Sub) &&
 144            "Use is not a valid add instruction!");
 145
 146     // Get the value added to the cast result pointer...
 147     Value *OtherPtr = I->getOperand((I->getOperand(0) == &CI) ? 1 : 0);
 148
 149     Instruction *GEP = new GetElementPtrInst(OtherPtr, Indices, I->getName());
 150     PRINT_PEEPHOLE1("cast-add-to-gep:i", I);
 151
 152     // If the instruction is actually a subtract, we are guaranteed to only have
 153     // one index (from code above), so we just need to negate the pointer index
 154     // long value.
 155     if (I->getOpcode() == Instruction::Sub) {
 156       Instruction *Neg = BinaryOperator::createNeg(GEP->getOperand(1),
 157                                        GEP->getOperand(1)->getName()+".neg", I);
 158       GEP->setOperand(1, Neg);
 159     }
 160
 161     if (GEP->getType() == I->getType()) {
 162       // Replace the old add instruction with the shiny new GEP inst
 163       ReplaceInstWithInst(I, GEP);
 164     } else {
 165       // If the type produced by the gep instruction differs from the original
 166       // add instruction type, insert a cast now.
 167       //
 168
 169       // Insert the GEP instruction before the old add instruction...
 170       I->getParent()->getInstList().insert(I, GEP);
 171
 172       PRINT_PEEPHOLE1("cast-add-to-gep:o", GEP);
 173       GEP = new CastInst(GEP, I->getType());
 174
 175       // Replace the old add instruction with the shiny new GEP inst
 176       ReplaceInstWithInst(I, GEP);
 177     }
 178
 179     PRINT_PEEPHOLE1("cast-add-to-gep:o", GEP);
 180   }
 181   return true;
 182 }
 183
 184 // Peephole optimize the following instructions:
 185 // %t1 = cast ulong <const int> to {<...>} *
 186 // %t2 = add {<...>} * %SP, %t1              ;; Constant must be 2nd operand
 187 //
 188 //    or
 189 // %t1 = cast {<...>}* %SP to int*
 190 // %t5 = cast ulong <const int> to int*
 191 // %t2 = add int* %t1, %t5                   ;; int is same size as field
 192 //
 193 // Into: %t3 = getelementptr {<...>} * %SP, <element indices>
 194 //       %t2 = cast <eltype> * %t3 to {<...>}*
 195 //
 196 static bool PeepholeOptimizeAddCast(BasicBlock *BB, BasicBlock::iterator &BI,
 197                                     Value *AddOp1, CastInst *AddOp2,
 198                                     const TargetData &TD) {
 199   const CompositeType *CompTy;
 200   Value *OffsetVal = AddOp2->getOperand(0);
 201   Value *SrcPtr = 0;  // Of type pointer to struct...
 202
 203   if ((CompTy = getPointedToComposite(AddOp1->getType()))) {
 204     SrcPtr = AddOp1;                      // Handle the first case...
 205   } else if (CastInst *AddOp1c = dyn_cast<CastInst>(AddOp1)) {
 206     SrcPtr = AddOp1c->getOperand(0);      // Handle the second case...
 207     CompTy = getPointedToComposite(SrcPtr->getType());
 208   }
 209
 210   // Only proceed if we have detected all of our conditions successfully...
 211   if (!CompTy || !SrcPtr || !OffsetVal->getType()->isInteger())
 212     return false;
 213
 214   std::vector<Value*> Indices;
 215   if (!ConvertableToGEP(SrcPtr->getType(), OffsetVal, Indices, TD, &BI))
 216     return false;  // Not convertable... perhaps next time
 217
 218   if (getPointedToComposite(AddOp1->getType())) {  // case 1
 219     PRINT_PEEPHOLE2("add-to-gep1:in", AddOp2, *BI);
 220   } else {
 221     PRINT_PEEPHOLE3("add-to-gep2:in", AddOp1, AddOp2, *BI);
 222   }
 223
 224   GetElementPtrInst *GEP = new GetElementPtrInst(SrcPtr, Indices,
 225                                                  AddOp2->getName(), BI);
 226
 227   Instruction *NCI = new CastInst(GEP, AddOp1->getType());
 228   ReplaceInstWithInst(BB->getInstList(), BI, NCI);
 229   PRINT_PEEPHOLE2("add-to-gep:out", GEP, NCI);
 230   return true;
 231 }
 232
 233 bool RPR::PeepholeOptimize(BasicBlock *BB, BasicBlock::iterator &BI) {
 234   Instruction *I = BI;
 235   const TargetData &TD = getAnalysis<TargetData>();
 236
 237   if (CastInst *CI = dyn_cast<CastInst>(I)) {
 238     Value       *Src    = CI->getOperand(0);
 239     Instruction *SrcI   = dyn_cast<Instruction>(Src); // Nonnull if instr source
 240     const Type  *DestTy = CI->getType();
 241
 242     // Peephole optimize the following instruction:
 243     // %V2 = cast <ty> %V to <ty>
 244     //
 245     // Into: <nothing>
 246     //
 247     if (DestTy == Src->getType()) {   // Check for a cast to same type as src!!
 248       PRINT_PEEPHOLE1("cast-of-self-ty", CI);
 249       CI->replaceAllUsesWith(Src);
 250       if (!Src->hasName() && CI->hasName()) {
 251         std::string Name = CI->getName();
 252         CI->setName("");
 253         Src->setName(Name, &BB->getParent()->getSymbolTable());
 254       }
 255
 256       // DCE the instruction now, to avoid having the iterative version of DCE
 257       // have to worry about it.
 258       //
 259       BI = BB->getInstList().erase(BI);
 260
 261       ++NumCastOfCast;
 262       return true;
 263     }
 264
 265     // Check to see if it's a cast of an instruction that does not depend on the
 266     // specific type of the operands to do it's job.
 267     if (!isReinterpretingCast(CI)) {
 268       ValueTypeCache ConvertedTypes;
 269
 270       // Check to see if we can convert the source of the cast to match the
 271       // destination type of the cast...
 272       //
 273       ConvertedTypes[CI] = CI->getType();  // Make sure the cast doesn't change
 274       if (ExpressionConvertableToType(Src, DestTy, ConvertedTypes, TD)) {
 275         PRINT_PEEPHOLE3("CAST-SRC-EXPR-CONV:in ", Src, CI, BB->getParent());
 276
 277         DEBUG(cerr << "\nCONVERTING SRC EXPR TYPE:\n");
 278         { // ValueMap must be destroyed before function verified!
 279           ValueMapCache ValueMap;
 280           Value *E = ConvertExpressionToType(Src, DestTy, ValueMap, TD);
 281
 282           if (Constant *CPV = dyn_cast<Constant>(E))
 283             CI->replaceAllUsesWith(CPV);
 284
 285           PRINT_PEEPHOLE1("CAST-SRC-EXPR-CONV:out", E);
 286           DEBUG(cerr << "DONE CONVERTING SRC EXPR TYPE: \n" << BB->getParent());
 287         }
 288
 289         DEBUG(assert(verifyFunction(*BB->getParent()) == false &&
 290                      "Function broken!"));
 291         BI = BB->begin();  // Rescan basic block.  BI might be invalidated.
 292         ++NumExprTreesConv;
 293         return true;
 294       }
 295
 296       // Check to see if we can convert the users of the cast value to match the
 297       // source type of the cast...
 298       //
 299       ConvertedTypes.clear();
 300       // Make sure the source doesn't change type
 301       ConvertedTypes[Src] = Src->getType();
 302       if (ValueConvertableToType(CI, Src->getType(), ConvertedTypes, TD)) {
 303         PRINT_PEEPHOLE3("CAST-DEST-EXPR-CONV:in ", Src, CI, BB->getParent());
 304
 305         DEBUG(cerr << "\nCONVERTING EXPR TYPE:\n");
 306         { // ValueMap must be destroyed before function verified!
 307           ValueMapCache ValueMap;
 308           ConvertValueToNewType(CI, Src, ValueMap, TD);  // This will delete CI!
 309         }
 310
 311         PRINT_PEEPHOLE1("CAST-DEST-EXPR-CONV:out", Src);
 312         DEBUG(cerr << "DONE CONVERTING EXPR TYPE: \n\n" << BB->getParent());
 313
 314         DEBUG(assert(verifyFunction(*BB->getParent()) == false &&
 315                      "Function broken!"));
 316         BI = BB->begin();  // Rescan basic block.  BI might be invalidated.
 317         ++NumExprTreesConv;
 318         return true;
 319       }
 320     }
 321
 322     // Otherwise find out it this cast is a cast to a pointer type, which is
 323     // then added to some other pointer, then loaded or stored through.  If
 324     // so, convert the add into a getelementptr instruction...
 325     //
 326     if (const PointerType *DestPTy = dyn_cast<PointerType>(DestTy)) {
 327       if (HandleCastToPointer(BI, DestPTy, TD)) {
 328         BI = BB->begin();  // Rescan basic block.  BI might be invalidated.
 329         ++NumGEPInstFormed;
 330         return true;
 331       }
 332     }
 333
 334     // Check to see if we are casting from a structure pointer to a pointer to
 335     // the first element of the structure... to avoid munching other peepholes,
 336     // we only let this happen if there are no add uses of the cast.
 337     //
 338     // Peephole optimize the following instructions:
 339     // %t1 = cast {<...>} * %StructPtr to <ty> *
 340     //
 341     // Into: %t2 = getelementptr {<...>} * %StructPtr, <0, 0, 0, ...>
 342     //       %t1 = cast <eltype> * %t1 to <ty> *
 343     //
 344     if (const CompositeType *CTy = getPointedToComposite(Src->getType()))
 345       if (const PointerType *DestPTy = dyn_cast<PointerType>(DestTy)) {
 346
 347         // Loop over uses of the cast, checking for add instructions.  If an add
 348         // exists, this is probably a part of a more complex GEP, so we don't
 349         // want to mess around with the cast.
 350         //
 351         bool HasAddUse = false;
 352         for (Value::use_iterator I = CI->use_begin(), E = CI->use_end();
 353              I != E; ++I)
 354           if (isa<Instruction>(*I) &&
 355               cast<Instruction>(*I)->getOpcode() == Instruction::Add) {
 356             HasAddUse = true; break;
 357           }
 358
 359         // If it doesn't have an add use, check to see if the dest type is
 360         // losslessly convertable to one of the types in the start of the struct
 361         // type.
 362         //
 363         if (!HasAddUse) {
 364           const Type *DestPointedTy = DestPTy->getElementType();
 365           unsigned Depth = 1;
 366           const CompositeType *CurCTy = CTy;
 367           const Type *ElTy = 0;
 368
 369           // Build the index vector, full of all zeros
 370           std::vector<Value*> Indices;
 371           Indices.push_back(ConstantSInt::get(Type::LongTy, 0));
 372           while (CurCTy && !isa<PointerType>(CurCTy)) {
 373             if (const StructType *CurSTy = dyn_cast<StructType>(CurCTy)) {
 374               // Check for a zero element struct type... if we have one, bail.
 375               if (CurSTy->getElementTypes().size() == 0) break;
 376
 377               // Grab the first element of the struct type, which must lie at
 378               // offset zero in the struct.
 379               //
 380               ElTy = CurSTy->getElementTypes()[0];
 381             } else {
 382               ElTy = cast<ArrayType>(CurCTy)->getElementType();
 383             }
 384
 385             // Insert a zero to index through this type...
 386             Indices.push_back(Constant::getNullValue(CurCTy->getIndexType()));
 387
 388             // Did we find what we're looking for?
 389             if (ElTy->isLosslesslyConvertableTo(DestPointedTy)) break;
 390
 391             // Nope, go a level deeper.
 392             ++Depth;
 393             CurCTy = dyn_cast<CompositeType>(ElTy);
 394             ElTy = 0;
 395           }
 396
 397           // Did we find what we were looking for? If so, do the transformation
 398           if (ElTy) {
 399             PRINT_PEEPHOLE1("cast-for-first:in", CI);
 400
 401             std::string Name = CI->getName(); CI->setName("");
 402
 403             // Insert the new T cast instruction... stealing old T's name
 404             GetElementPtrInst *GEP = new GetElementPtrInst(Src, Indices,
 405                                                            Name, BI);
 406
 407             // Make the old cast instruction reference the new GEP instead of
 408             // the old src value.
 409             //
 410             CI->setOperand(0, GEP);
 411
 412             PRINT_PEEPHOLE2("cast-for-first:out", GEP, CI);
 413             ++NumGEPInstFormed;
 414             return true;
 415           }
 416         }
 417       }
 418
 419   } else if (StoreInst *SI = dyn_cast<StoreInst>(I)) {
 420     Value *Val     = SI->getOperand(0);
 421     Value *Pointer = SI->getPointerOperand();
 422
 423     // Peephole optimize the following instructions:
 424     // %t = cast <T1>* %P to <T2> * ;; If T1 is losslessly convertable to T2
 425     // store <T2> %V, <T2>* %t
 426     //
 427     // Into:
 428     // %t = cast <T2> %V to <T1>
 429     // store <T1> %t2, <T1>* %P
 430     //
 431     // Note: This is not taken care of by expr conversion because there might
 432     // not be a cast available for the store to convert the incoming value of.
 433     // This code is basically here to make sure that pointers don't have casts
 434     // if possible.
 435     //
 436     if (CastInst *CI = dyn_cast<CastInst>(Pointer))
 437       if (Value *CastSrc = CI->getOperand(0)) // CSPT = CastSrcPointerType
 438         if (const PointerType *CSPT = dyn_cast<PointerType>(CastSrc->getType()))
 439           // convertable types?
 440           if (Val->getType()->isLosslesslyConvertableTo(CSPT->getElementType())) {
 441             PRINT_PEEPHOLE3("st-src-cast:in ", Pointer, Val, SI);
 442
 443             // Insert the new T cast instruction... stealing old T's name
 444             std::string Name(CI->getName()); CI->setName("");
 445             CastInst *NCI = new CastInst(Val, CSPT->getElementType(),
 446                                          Name, BI);
 447
 448             // Replace the old store with a new one!
 449             ReplaceInstWithInst(BB->getInstList(), BI,
 450                                 SI = new StoreInst(NCI, CastSrc));
 451             PRINT_PEEPHOLE3("st-src-cast:out", NCI, CastSrc, SI);
 452             ++NumLoadStorePeepholes;
 453             return true;
 454           }
 455
 456   } else if (LoadInst *LI = dyn_cast<LoadInst>(I)) {
 457     Value *Pointer = LI->getOperand(0);
 458     const Type *PtrElType =
 459       cast<PointerType>(Pointer->getType())->getElementType();
 460
 461     // Peephole optimize the following instructions:
 462     // %Val = cast <T1>* to <T2>*    ;; If T1 is losslessly convertable to T2
 463     // %t = load <T2>* %P
 464     //
 465     // Into:
 466     // %t = load <T1>* %P
 467     // %Val = cast <T1> to <T2>
 468     //
 469     // Note: This is not taken care of by expr conversion because there might
 470     // not be a cast available for the store to convert the incoming value of.
 471     // This code is basically here to make sure that pointers don't have casts
 472     // if possible.
 473     //
 474     if (CastInst *CI = dyn_cast<CastInst>(Pointer))
 475       if (Value *CastSrc = CI->getOperand(0)) // CSPT = CastSrcPointerType
 476         if (const PointerType *CSPT = dyn_cast<PointerType>(CastSrc->getType()))
 477           // convertable types?
 478           if (PtrElType->isLosslesslyConvertableTo(CSPT->getElementType())) {
 479             PRINT_PEEPHOLE2("load-src-cast:in ", Pointer, LI);
 480
 481             // Create the new load instruction... loading the pre-casted value
 482             LoadInst *NewLI = new LoadInst(CastSrc, LI->getName(), BI);
 483
 484             // Insert the new T cast instruction... stealing old T's name
 485             CastInst *NCI = new CastInst(NewLI, LI->getType(), CI->getName());
 486
 487             // Replace the old store with a new one!
 488             ReplaceInstWithInst(BB->getInstList(), BI, NCI);
 489             PRINT_PEEPHOLE3("load-src-cast:out", NCI, CastSrc, NewLI);
 490             ++NumLoadStorePeepholes;
 491             return true;
 492           }
 493
 494   } else if (I->getOpcode() == Instruction::Add &&
 495              isa<CastInst>(I->getOperand(1))) {
 496
 497     if (PeepholeOptimizeAddCast(BB, BI, I->getOperand(0),
 498                                 cast<CastInst>(I->getOperand(1)), TD)) {
 499       ++NumGEPInstFormed;
 500       return true;
 501     }
 502   } else if (CallInst *CI = dyn_cast<CallInst>(I)) {
 503     // If we have a call with all varargs arguments, convert the call to use the
 504     // actual argument types present...
 505     //
 506     const PointerType *PTy = cast<PointerType>(CI->getCalledValue()->getType());
 507     const FunctionType *FTy = cast<FunctionType>(PTy->getElementType());
 508
 509     // Is the call to a vararg variable with no real parameters?
 510     if (FTy->isVarArg() && FTy->getNumParams() == 0 &&
 511         !CI->getCalledFunction()) {
 512       // If so, insert a new cast instruction, casting it to a function type
 513       // that matches the current arguments...
 514       //
 515       std::vector<const Type *> Params;  // Parameter types...
 516       for (unsigned i = 1, e = CI->getNumOperands(); i != e; ++i)
 517         Params.push_back(CI->getOperand(i)->getType());
 518
 519       FunctionType *NewFT = FunctionType::get(FTy->getReturnType(),
 520                                               Params, false);
 521       PointerType *NewPFunTy = PointerType::get(NewFT);
 522
 523       // Create a new cast, inserting it right before the function call...
 524       Value *NewCast;
 525       Constant *ConstantCallSrc = 0;
 526       if (Constant *CS = dyn_cast<Constant>(CI->getCalledValue()))
 527         ConstantCallSrc = CS;
 528       else if (GlobalValue *GV = dyn_cast<GlobalValue>(CI->getCalledValue()))
 529         ConstantCallSrc = ConstantPointerRef::get(GV);
 530
 531       if (ConstantCallSrc)
 532         NewCast = ConstantExpr::getCast(ConstantCallSrc, NewPFunTy);
 533       else
 534         NewCast = new CastInst(CI->getCalledValue(), NewPFunTy,
 535                                CI->getCalledValue()->getName()+"_c",CI);
 536
 537       // Create a new call instruction...
 538       CallInst *NewCall = new CallInst(NewCast,
 539                            std::vector<Value*>(CI->op_begin()+1, CI->op_end()));
 540       ++BI;
 541       ReplaceInstWithInst(CI, NewCall);
 542
 543       ++NumVarargCallChanges;
 544       return true;
 545     }
 546
 547   }
 548
 549   return false;
 550 }
 551
 552
 553
 554
 555 bool RPR::DoRaisePass(Function &F) {
 556   bool Changed = false;
 557   for (Function::iterator BB = F.begin(), BBE = F.end(); BB != BBE; ++BB)
 558     for (BasicBlock::iterator BI = BB->begin(); BI != BB->end();) {
 559       DEBUG(cerr << "Processing: " << *BI);
 560       if (dceInstruction(BI) || doConstantPropogation(BI)) {
 561         Changed = true;
 562         ++NumDCEorCP;
 563         DEBUG(cerr << "***\t\t^^-- Dead code eliminated!\n");
 564       } else if (PeepholeOptimize(BB, BI)) {
 565         Changed = true;
 566       } else {
 567         ++BI;
 568       }
 569     }
 570
 571   return Changed;
 572 }
 573
 574
 575 // runOnFunction - Raise a function representation to a higher level.
 576 bool RPR::runOnFunction(Function &F) {
 577   DEBUG(cerr << "\n\n\nStarting to work on Function '" << F.getName() << "'\n");
 578
 579   // Insert casts for all incoming pointer pointer values that are treated as
 580   // arrays...
 581   //
 582   bool Changed = false, LocalChange;
 583
 584   // If the StartInst option was specified, then Peephole optimize that
 585   // instruction first if it occurs in this function.
 586   //
 587   if (!StartInst.empty()) {
 588     for (Function::iterator BB = F.begin(), BBE = F.end(); BB != BBE; ++BB)
 589       for (BasicBlock::iterator BI = BB->begin(); BI != BB->end(); ++BI)
 590         if (BI->getName() == StartInst) {
 591           bool SavedDebug = DebugFlag;  // Save the DEBUG() controlling flag.
 592           DebugFlag = true;             // Turn on DEBUG's
 593           Changed |= PeepholeOptimize(BB, BI);
 594           DebugFlag = SavedDebug;       // Restore DebugFlag to previous state
 595         }
 596   }
 597
 598   do {
 599     DEBUG(cerr << "Looping: \n" << F);
 600
 601     // Iterate over the function, refining it, until it converges on a stable
 602     // state
 603     LocalChange = false;
 604     while (DoRaisePass(F)) LocalChange = true;
 605     Changed |= LocalChange;
 606
 607   } while (LocalChange);
 608
 609   return Changed;
 610 }