Lots of nonfunctional code cleanups

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

diff --git a/lib/Analysis/LiveVar/LiveVarSet.cpp b/lib/Analysis/LiveVar/LiveVarSet.cpp
index bcc9de95686de766e70af0c5bc5293963d4a2f45..8623aa86549242d35c4f36d5afaec26c483a8d42 100644 (file)
--- a/lib/Analysis/LiveVar/LiveVarSet.cpp
+++ b/lib/Analysis/LiveVar/LiveVarSet.cpp
@@ -1,6 +1,6 @@
 #include "llvm/Analysis/LiveVar/LiveVarSet.h"
 #include "llvm/CodeGen/MachineInstr.h"
-
+#include "llvm/Type.h"
 
 // This function applies a machine instr to a live var set (accepts OutSet) and
 // makes necessary changes to it (produces InSet). Note that two for loops are
@@ -9,53 +9,29 @@
 // machine instruction operand.
 
 
-void LiveVarSet::applyTranferFuncForMInst(const MachineInstr *const MInst)
-{
-
-  for( MachineInstr::val_const_op_iterator OpI(MInst); !OpI.done() ; OpI++) {
-
-    if( OpI.isDef() )      // kill only if this operand is a def
+void LiveVarSet::applyTranferFuncForMInst(const MachineInstr *MInst) {
+  for (MachineInstr::val_const_op_iterator OpI(MInst); !OpI.done(); ++OpI) {
+    if (OpI.isDef())      // kill only if this operand is a def
          remove(*OpI);        // this definition kills any uses
   }
 
   // do for implicit operands as well
-  for( unsigned i=0; i < MInst->getNumImplicitRefs(); ++i) {
-    if( MInst->implicitRefIsDefined(i) )
-      remove( MInst->getImplicitRef(i) );
+  for ( unsigned i=0; i < MInst->getNumImplicitRefs(); ++i) {
+    if (MInst->implicitRefIsDefined(i))
+      remove(MInst->getImplicitRef(i));
   }
 
 
-  for( MachineInstr::val_const_op_iterator OpI(MInst); !OpI.done() ; OpI++) {
-
-    if ( ((*OpI)->getType())->isLabelType()) continue; // don't process labels
+  for (MachineInstr::val_const_op_iterator OpI(MInst); !OpI.done(); ++OpI) {
+    if ((*OpI)->getType()->isLabelType()) continue; // don't process labels
     
-    if( ! OpI.isDef() )      // add only if this operand is a use
-       add( *OpI );            // An operand is a use - so add to use set
+    if (!OpI.isDef())      // add only if this operand is a use
+       add(*OpI);            // An operand is a use - so add to use set
   }
 
   // do for implicit operands as well
-  for( unsigned i=0; i < MInst->getNumImplicitRefs(); ++i) {
-    if(  ! MInst->implicitRefIsDefined(i) )
-      add( MInst->getImplicitRef(i) );
+  for (unsigned i=0; i < MInst->getNumImplicitRefs(); ++i) {
+    if (!MInst->implicitRefIsDefined(i))
+      add(MInst->getImplicitRef(i));
   }
-
-}
-
-  
-
-#if 0
-void LiveVarSet::applyTranferFuncForInst(const Instruction *const Inst) 
-{
-
-  if( Inst->isDefinition() ) {  // add to Defs iff this instr is a definition
-       remove(Inst);            // this definition kills any uses
-  }
-  Instruction::const_op_iterator OpI = Inst->op_begin();  // get operand iterat
-
-  for( ; OpI != Inst->op_end() ; OpI++) {              // iterate over operands
-    if ( ((*OpI)->getType())->isLabelType()) continue; // don't process labels 
-    add( *OpI );                     // An operand is a use - so add to use set
-  }
-
 }
-#endif

diff --git a/lib/Target/SparcV9/LiveVar/LiveVarSet.cpp b/lib/Target/SparcV9/LiveVar/LiveVarSet.cpp
index bcc9de95686de766e70af0c5bc5293963d4a2f45..8623aa86549242d35c4f36d5afaec26c483a8d42 100644 (file)
--- a/lib/Target/SparcV9/LiveVar/LiveVarSet.cpp
+++ b/lib/Target/SparcV9/LiveVar/LiveVarSet.cpp
@@ -1,6 +1,6 @@
 #include "llvm/Analysis/LiveVar/LiveVarSet.h"
 #include "llvm/CodeGen/MachineInstr.h"
-
+#include "llvm/Type.h"
 
 // This function applies a machine instr to a live var set (accepts OutSet) and
 // makes necessary changes to it (produces InSet). Note that two for loops are
@@ -9,53 +9,29 @@
 // machine instruction operand.
 
 
-void LiveVarSet::applyTranferFuncForMInst(const MachineInstr *const MInst)
-{
-
-  for( MachineInstr::val_const_op_iterator OpI(MInst); !OpI.done() ; OpI++) {
-
-    if( OpI.isDef() )      // kill only if this operand is a def
+void LiveVarSet::applyTranferFuncForMInst(const MachineInstr *MInst) {
+  for (MachineInstr::val_const_op_iterator OpI(MInst); !OpI.done(); ++OpI) {
+    if (OpI.isDef())      // kill only if this operand is a def
          remove(*OpI);        // this definition kills any uses
   }
 
   // do for implicit operands as well
-  for( unsigned i=0; i < MInst->getNumImplicitRefs(); ++i) {
-    if( MInst->implicitRefIsDefined(i) )
-      remove( MInst->getImplicitRef(i) );
+  for ( unsigned i=0; i < MInst->getNumImplicitRefs(); ++i) {
+    if (MInst->implicitRefIsDefined(i))
+      remove(MInst->getImplicitRef(i));
   }
 
 
-  for( MachineInstr::val_const_op_iterator OpI(MInst); !OpI.done() ; OpI++) {
-
-    if ( ((*OpI)->getType())->isLabelType()) continue; // don't process labels
+  for (MachineInstr::val_const_op_iterator OpI(MInst); !OpI.done(); ++OpI) {
+    if ((*OpI)->getType()->isLabelType()) continue; // don't process labels
     
-    if( ! OpI.isDef() )      // add only if this operand is a use
-       add( *OpI );            // An operand is a use - so add to use set
+    if (!OpI.isDef())      // add only if this operand is a use
+       add(*OpI);            // An operand is a use - so add to use set
   }
 
   // do for implicit operands as well
-  for( unsigned i=0; i < MInst->getNumImplicitRefs(); ++i) {
-    if(  ! MInst->implicitRefIsDefined(i) )
-      add( MInst->getImplicitRef(i) );
+  for (unsigned i=0; i < MInst->getNumImplicitRefs(); ++i) {
+    if (!MInst->implicitRefIsDefined(i))
+      add(MInst->getImplicitRef(i));
   }
-
-}
-
-  
-
-#if 0
-void LiveVarSet::applyTranferFuncForInst(const Instruction *const Inst) 
-{
-
-  if( Inst->isDefinition() ) {  // add to Defs iff this instr is a definition
-       remove(Inst);            // this definition kills any uses
-  }
-  Instruction::const_op_iterator OpI = Inst->op_begin();  // get operand iterat
-
-  for( ; OpI != Inst->op_end() ; OpI++) {              // iterate over operands
-    if ( ((*OpI)->getType())->isLabelType()) continue; // don't process labels 
-    add( *OpI );                     // An operand is a use - so add to use set
-  }
-
 }
-#endif