[llvm-commits] CVS: llvm/lib/CodeGen/InstrSched/InstrScheduling.cpp

Brian Gaeke gaeke at cs.uiuc.edu
Wed Jul 28 12:24:58 PDT 2004



Changes in directory llvm/lib/CodeGen/InstrSched:

InstrScheduling.cpp updated: 1.73 -> 1.74

---
Log message:

TargetInstrInfo::hasOperandInterlock() is always true, because it is
never overridden by any target.


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

Index: llvm/lib/CodeGen/InstrSched/InstrScheduling.cpp
diff -u llvm/lib/CodeGen/InstrSched/InstrScheduling.cpp:1.73 llvm/lib/CodeGen/InstrSched/InstrScheduling.cpp:1.74
--- llvm/lib/CodeGen/InstrSched/InstrScheduling.cpp:1.73	Thu Jul 15 19:04:54 2004
+++ llvm/lib/CodeGen/InstrSched/InstrScheduling.cpp	Wed Jul 28 14:24:48 2004
@@ -1045,11 +1045,6 @@
 	&& (*EI)->getDepType() == SchedGraphEdge::CtrlDep)
       return false;
   
-  // for now, don't put an instruction that does not have operand
-  // interlocks in the delay slot of a branch
-  if (! S.getInstrInfo().hasOperandInterlock(node->getOpcode()))
-    return false;
-  
   // Finally, if the instruction precedes the branch, we make sure the
   // instruction can be reordered relative to the branch.  We simply check
   // if the instr. has only 1 outgoing edge, viz., a CD edge to the branch.
@@ -1326,11 +1321,7 @@
       const SchedGraphNode* dnode = delayNodeVec[i];
       if ( ! S.isScheduled(dnode)
            && S.schedInfo.instrCanUseSlot(dnode->getOpcode(), nextSlot)
-           && instrIsFeasible(S, dnode->getOpcode()))
-      {
-        assert(S.getInstrInfo().hasOperandInterlock(dnode->getOpcode())
-               && "Instructions without interlocks not yet supported "
-               "when filling branch delay slots");
+           && instrIsFeasible(S, dnode->getOpcode())) {
         S.scheduleInstr(dnode, nextSlot, nextTime);
         break;
       }





More information about the llvm-commits mailing list