[PATCH] D23303: LiveIntervalAnalysis: fix a crash in repairOldRegInRange

Krzysztof Parzyszek via llvm-commits llvm-commits at lists.llvm.org
Tue Aug 9 09:25:09 PDT 2016


kparzysz added a comment.

This looks ok to me.


================
Comment at: lib/CodeGen/LiveIntervalAnalysis.cpp:1404
@@ +1403,3 @@
+    // occurs _after_ the range that is to be repaired.
+    assert(LaneMask != 0);
+    return;
----------------
LaneMask is probably always non-zero.

================
Comment at: lib/CodeGen/LiveIntervalAnalysis.cpp:1406
@@ -1402,1 +1405,3 @@
+    return;
+  } else
     --LII;
----------------
Remove the "else".


https://reviews.llvm.org/D23303





More information about the llvm-commits mailing list