[PATCH] D23879: Properly handle live subranges when moving instructions

[Matthias Braun via llvm-commits]

MatzeB added a comment.

This should have a test in unittests/MI/LiveIntervalTest.cpp. Hopefully you can remodel the situation from the example that way.

(And just to warn you: I am in vacation for the next two weeks and will only get back to this after that time).


================
Comment at: lib/CodeGen/LiveIntervalAnalysis.cpp:1049-1050
@@ +1048,4 @@
+        //   1200B    ... = COPY vreg131
+        // The live subrange for vreg131:sub2 will contain
+        //   [...,1008r)[1040r,1200r)
+        //
----------------
Why does the liverange for sub2 end at 1008B in this example? This is about the subregister liverange not about the main liverange is it? Then nothing seems to be reading the value at 1008 so I don't see why the liverange ends there.


Repository:
  rL LLVM

https://reviews.llvm.org/D23879