Remove redundant 'else's. No functionality change.
authorDan Gohman <gohman@apple.com>
Wed, 7 Jan 2009 22:30:55 +0000 (22:30 +0000)
committerDan Gohman <gohman@apple.com>
Wed, 7 Jan 2009 22:30:55 +0000 (22:30 +0000)
git-svn-id: https://llvm.org/svn/llvm-project/llvm/trunk@61891 91177308-0d34-0410-b5e6-96231b3b80d8

lib/CodeGen/SelectionDAG/ScheduleDAGRRList.cpp

index 25c8bfb33944fa67fe24e8eed1ae83b37f7c7e5a..9fee4b7aae3c50beba84d0259f83f02c5f604524 100644 (file)
@@ -971,27 +971,26 @@ namespace {
         // allocation choices. But if it is a livein then perhaps we want it
         // closer to its uses so it can be coalesced.
         return 0xffff;
-      else if (Opc == ISD::TokenFactor || Opc == ISD::CopyToReg)
+      if (Opc == ISD::TokenFactor || Opc == ISD::CopyToReg)
         // CopyToReg should be close to its uses to facilitate coalescing and
         // avoid spilling.
         return 0;
-      else if (Opc == TargetInstrInfo::EXTRACT_SUBREG ||
-               Opc == TargetInstrInfo::INSERT_SUBREG)
+      if (Opc == TargetInstrInfo::EXTRACT_SUBREG ||
+          Opc == TargetInstrInfo::INSERT_SUBREG)
         // EXTRACT_SUBREG / INSERT_SUBREG should be close to its use to
         // facilitate coalescing.
         return 0;
-      else if (SU->NumSuccs == 0)
+      if (SU->NumSuccs == 0)
         // If SU does not have a use, i.e. it doesn't produce a value that would
         // be consumed (e.g. store), then it terminates a chain of computation.
         // Give it a large SethiUllman number so it will be scheduled right
         // before its predecessors that it doesn't lengthen their live ranges.
         return 0xffff;
-      else if (SU->NumPreds == 0)
+      if (SU->NumPreds == 0)
         // If SU does not have a def, schedule it close to its uses because it
         // does not lengthen any live ranges.
         return 0;
-      else
-        return SethiUllmanNumbers[SU->NodeNum];
+      return SethiUllmanNumbers[SU->NodeNum];
     }
     
     unsigned size() const { return Queue.size(); }