[PATCH] D23302: LiveIntervalAnalysis: add an assertion in repairOldRegInRange

Nicolai Hähnle via llvm-commits llvm-commits at lists.llvm.org
Tue Aug 9 02:38:19 PDT 2016


nhaehnle created this revision.
nhaehnle added reviewers: MatzeB, kparzysz.
nhaehnle added a subscriber: llvm-commits.
Herald added a subscriber: MatzeB.

This is triggered by a test case I have, which otherwise leads to
non-deterministic crashes making it rather harder to even
bugpoint-reduce...

https://reviews.llvm.org/D23302

Files:
  lib/CodeGen/LiveIntervalAnalysis.cpp

Index: lib/CodeGen/LiveIntervalAnalysis.cpp
===================================================================
--- lib/CodeGen/LiveIntervalAnalysis.cpp
+++ lib/CodeGen/LiveIntervalAnalysis.cpp
@@ -1398,8 +1398,10 @@
   SlotIndex lastUseIdx;
   if (LII != LR.end() && LII->start < endIdx)
     lastUseIdx = LII->end;
-  else
+  else {
+    assert(LII != LR.begin());
     --LII;
+  }
 
   for (MachineBasicBlock::iterator I = End; I != Begin;) {
     --I;


-------------- next part --------------
A non-text attachment was scrubbed...
Name: D23302.67304.patch
Type: text/x-patch
Size: 456 bytes
Desc: not available
URL: <http://lists.llvm.org/pipermail/llvm-commits/attachments/20160809/65afa0b3/attachment.bin>


More information about the llvm-commits mailing list