If a vector is empty, you're not allowed to access any
authorDuncan Sands <baldrick@free.fr>
Tue, 20 Jan 2009 09:05:19 +0000 (09:05 +0000)
committerDuncan Sands <baldrick@free.fr>
Tue, 20 Jan 2009 09:05:19 +0000 (09:05 +0000)
elements, even if it is only to take the address.  Test:
break-anti-dependencies.ll with ENABLE_EXPENSIVE_CHECKS.

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

include/llvm/CodeGen/ScheduleDAGInstrs.h

index 2e5d8336352f9b4062668320ce8e0fc11d45d1fc..7c0e80afe43888eecf1f1529088facb0648cf018 100644 (file)
@@ -49,10 +49,11 @@ namespace llvm {
     ///
     SUnit *NewSUnit(MachineInstr *MI) {
 #ifndef NDEBUG
-      const SUnit *Addr = &SUnits[0];
+      const SUnit *Addr = SUnits.empty() ? 0 : &SUnits[0];
 #endif
       SUnits.push_back(SUnit(MI, (unsigned)SUnits.size()));
-      assert(Addr == &SUnits[0] && "SUnits std::vector reallocated on the fly!");
+      assert((Addr == 0 || Addr == &SUnits[0]) &&
+             "SUnits std::vector reallocated on the fly!");
       SUnits.back().OrigNode = &SUnits.back();
       return &SUnits.back();
     }