[PATCH] D48555: Account for undef values from predecessors in extendSegmentsToUses
David Stuttard via Phabricator via llvm-commits
llvm-commits at lists.llvm.org
Tue Jun 26 03:00:13 PDT 2018
dstuttard accepted this revision.
dstuttard added a comment.
This revision is now accepted and ready to land.
Yes, this change looks better based on what you say about not requiring live-in's for undef sub-ranges.
Regarding cutting down the .mir test case. Presumably you're happy with the version you've copied across to this change (removing the unnecessary register classes)? I've already spent quite a lot of time reducing the test case as much as this - the problem is that it requires spilling to trigger the issue. Using -stress-regalloc worked to some extent, but couldn't be reduced further in this case - hence some of the extra code that doesn't directly impinge on the problem.
Repository:
rL LLVM
https://reviews.llvm.org/D48555
More information about the llvm-commits
mailing list