canClobberPhysRegDefs shouldn't called without checking hasPhysRegDefs;
authorDan Gohman <gohman@apple.com>
Sat, 21 Jun 2008 22:05:24 +0000 (22:05 +0000)
committerDan Gohman <gohman@apple.com>
Sat, 21 Jun 2008 22:05:24 +0000 (22:05 +0000)
check this with an assert.

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

lib/CodeGen/SelectionDAG/ScheduleDAGRRList.cpp

index 6175bc275ea6ddcaeed8ddc458d32df0426932a4..a50dd7b7c5d37c33476b292f4d95dfcdd71d2f3b 100644 (file)
@@ -1572,15 +1572,14 @@ static bool hasCopyToRegUse(SUnit *SU) {
 }
 
 /// canClobberPhysRegDefs - True if SU would clobber one of SuccSU's
-/// physical register def.
+/// physical register defs.
 static bool canClobberPhysRegDefs(SUnit *SuccSU, SUnit *SU,
                                   const TargetInstrInfo *TII,
                                   const TargetRegisterInfo *TRI) {
   SDNode *N = SuccSU->Node;
   unsigned NumDefs = TII->get(N->getTargetOpcode()).getNumDefs();
   const unsigned *ImpDefs = TII->get(N->getTargetOpcode()).getImplicitDefs();
-  if (!ImpDefs)
-    return false;
+  assert(ImpDefs && "Caller should check hasPhysRegDefs");
   const unsigned *SUImpDefs =
     TII->get(SU->Node->getTargetOpcode()).getImplicitDefs();
   if (!SUImpDefs)