[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