[PATCH] D9067: LiveIntervalAnalysis: Support moving of subregister defs in handleMove

Axel Davy via llvm-commits llvm-commits at lists.llvm.org
Mon Jan 11 12:00:09 PST 2016


axeldavy added a comment.

In http://reviews.llvm.org/D9067#323823, @MatzeB wrote:

> This is surprising, I would have expected this patch to only have an effect for http://reviews.llvm.org/D14969. Do you have out-of-tree code which reschedules independent subregister definitions? Apart from that this patch is not accepted yet, I wanted to not apply this until the rest of the subreg scheduling patches is accepted.


Improvements to http://reviews.llvm.org/D11885 needs this patch (see the part "#if 0 // To enable when handleMove fix lands").

Note that this patch doesn't solve all handleMove bug.
I have a part "// Restore old ordering (which prevents a LIS->handleMove bug)." which cannot be removed either with this patch.


Repository:
  rL LLVM

http://reviews.llvm.org/D9067





More information about the llvm-commits mailing list