commit 68b07b7c5de181d58b6833defccd6ebb79e689b9
author Chris Lattner <sabre@nondot.org> Thu Nov 01 07:00:51 2001 +0000
committer Chris Lattner <sabre@nondot.org> Thu Nov 01 07:00:51 2001 +0000
tree f2584c2ae2126d0153e3e779c862bfb0eea9687e
parent a1f6e648bed9d6eb28d6f292a3ac4e3e5bcedd2e

Add DCE as integral part of the level raising to avoid processing instructions that are dead


git-svn-id: https://llvm.org/svn/llvm-project/llvm/trunk@1084 91177308-0d34-0410-b5e6-96231b3b80d8

lib/Transforms/LevelRaise.cpp

diff --git a/lib/Transforms/LevelRaise.cpp b/lib/Transforms/LevelRaise.cpp
index 61610ac..300224e 100644
--- a/lib/Transforms/LevelRaise.cpp
+++ b/lib/Transforms/LevelRaise.cpp
@@ -37,6 +37,7 @@
 #include "llvm/ConstPoolVals.h"
 #include "llvm/Target/TargetData.h"
 #include "llvm/Optimizations/ConstantHandling.h"
+#include "llvm/Optimizations/DCE.h"
 #include <map>
 #include <algorithm>
 
@@ -137,7 +138,8 @@
     if (Offset >= SL->MemberOffsets[i] && Offset <  SL->MemberOffsets[i+1])
       break;
   
-  assert(Offset >= SL->MemberOffsets[i] && Offset <  SL->MemberOffsets[i+1]);
+  assert(Offset >= SL->MemberOffsets[i] &&
+         (i == SL->MemberOffsets.size()-1 || Offset <  SL->MemberOffsets[i+1]));
 
   // Make sure to save the current index...
   Offsets.push_back(ConstPoolUInt::get(Type::UByteTy, i));
@@ -459,8 +461,6 @@
 
 static bool PeepholeOptimize(BasicBlock *BB, BasicBlock::iterator &BI) {
   Instruction *I = *BI;
-  // TODO: replace this with a DCE call
-  if (I->use_size() == 0 && I->getType() != Type::VoidTy) return false;
 
   if (CastInst *CI = dyn_cast<CastInst>(I)) {
     Value       *Src    = CI->getOperand(0);
@@ -642,7 +642,8 @@
     BasicBlock::InstListType &BIL = BB->getInstList();
 
     for (BasicBlock::iterator BI = BB->begin(); BI != BB->end();) {
-      if (PeepholeOptimize(BB, BI))
+      if (opt::DeadCodeElimination::dceInstruction(BIL, BI) ||
+          PeepholeOptimize(BB, BI))
         Changed = true;
       else
         ++BI;
@@ -659,6 +660,10 @@
   if (M->isExternal()) return false;
   bool Changed = false;
 
+#ifdef DEBUG_PEEPHOLE_INSTS
+  cerr << "\n\n\nStarting to work on Method '" << M->getName() << "'\n";
+#endif
+
   while (DoRaisePass(M)) Changed = true;
 
   // PtrCasts - Keep a mapping between the pointer values (the key of the