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

Krzysztof Parzyszek via llvm-commits llvm-commits at lists.llvm.org
Fri Sep 2 06:59:50 PDT 2016


kparzysz abandoned this revision.
kparzysz added a comment.

The problem is somewhere else: live subranges should not be extended to other subregs' defs.


================
Comment at: lib/CodeGen/LiveIntervalAnalysis.cpp:1049-1050
@@ +1048,4 @@
+        //   1200B    ... = COPY vreg131
+        // The live subrange for vreg131:sub2 will contain
+        //   [...,1008r)[1040r,1200r)
+        //
----------------
Actually, you are right.  My comment is incorrect.


Repository:
  rL LLVM

https://reviews.llvm.org/D23879





More information about the llvm-commits mailing list