[llvm-commits] CVS: llvm/lib/CodeGen/SelectionDAG/ScheduleDAGRRList.cpp

[Evan Cheng]

Changes in directory llvm/lib/CodeGen/SelectionDAG:

ScheduleDAGRRList.cpp updated: 1.20 -> 1.21
---
Log message:

Remove dead code; added a missing null ptr check.

---
Diffs of the changes:  (+1 -32)

 ScheduleDAGRRList.cpp |   33 +--------------------------------
 1 files changed, 1 insertion(+), 32 deletions(-)


Index: llvm/lib/CodeGen/SelectionDAG/ScheduleDAGRRList.cpp
diff -u llvm/lib/CodeGen/SelectionDAG/ScheduleDAGRRList.cpp:1.20 llvm/lib/CodeGen/SelectionDAG/ScheduleDAGRRList.cpp:1.21
--- llvm/lib/CodeGen/SelectionDAG/ScheduleDAGRRList.cpp:1.20	Sat Nov  4 03:44:31 2006
+++ llvm/lib/CodeGen/SelectionDAG/ScheduleDAGRRList.cpp	Mon Nov  6 15:33:46 2006
@@ -685,7 +685,6 @@
 /// first (lower in the schedule).
 template<class SF>
 void BURegReductionPriorityQueue<SF>::AddPseudoTwoAddrDeps() {
-#if 1
   for (unsigned i = 0, e = SUnits->size(); i != e; ++i) {
     SUnit *SU = (SUnit *)&((*SUnits)[i]);
     if (!SU->isTwoAddress)
@@ -703,6 +702,7 @@
                                     TargetInstrInfo::TIED_TO) != -1) {
         SDNode *DU = SU->Node->getOperand(j).Val;
         SUnit *DUSU = (*SUnitMap)[DU];
+        if (!DUSU) continue;
         for (SUnit::succ_iterator I = DUSU->Succs.begin(),E = DUSU->Succs.end();
              I != E; ++I) {
           if (I->second) continue;
@@ -723,37 +723,6 @@
       }
     }
   }
-#else
-  for (unsigned i = 0, e = SUnits->size(); i != e; ++i) {
-    SUnit *SU = (SUnit *)&((*SUnits)[i]);
-    SDNode *Node = SU->Node;
-    if (!Node->isTargetOpcode())
-      continue;
-
-    if (SU->isTwoAddress) {
-      SUnit *DUSU = getDefUsePredecessor(SU, TII);
-      if (!DUSU) continue;
-
-      for (SUnit::succ_iterator I = DUSU->Succs.begin(), E = DUSU->Succs.end();
-           I != E; ++I) {
-        if (I->second) continue;
-        SUnit *SuccSU = I->first;
-        if (SuccSU != SU &&
-            (!canClobber(SuccSU, DUSU, TII) ||
-             (!SU->isCommutable && SuccSU->isCommutable))){
-          if (SuccSU->Depth == SU->Depth && !isReachable(SuccSU, SU)) {
-            DEBUG(std::cerr << "Adding an edge from SU # " << SU->NodeNum
-                  << " to SU #" << SuccSU->NodeNum << "\n");
-            if (SU->addPred(SuccSU, true))
-              SU->NumChainPredsLeft++;
-            if (SuccSU->addSucc(SU, true))
-              SuccSU->NumChainSuccsLeft++;
-          }
-        }
-      }
-    }
-  }
-#endif
 }
 
 /// CalcNodePriority - Priority is the Sethi Ullman number.