[PATCH] D21189: Create subranges for new intervals resulting from live interval splitting

Krzysztof Parzyszek via llvm-commits llvm-commits at lists.llvm.org
Thu Jun 9 14:50:20 PDT 2016


kparzysz marked an inline comment as done.

================
Comment at: lib/CodeGen/VirtRegMap.cpp:339
@@ -338,2 +338,3 @@
          "Reads of completely dead register should be marked undef already");
+  assert(LI.hasSubRanges());
   unsigned SubRegIdx = MO.getSubReg();
----------------
qcolombet wrote:
> I believe the assertion should be on SubRegIdx != 0, because hasSubRanges is only populated when we have sub register liveness enabled.
This function is only called when the subregister liveness is enabled, and when MO has a non-zero subregister (line 399 below).


Repository:
  rL LLVM

http://reviews.llvm.org/D21189





More information about the llvm-commits mailing list