[PATCH] D48102: Improve handling of COPY instructions with identical value numbers

Tim Renouf via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Fri Jun 22 05:06:14 PDT 2018


tpr requested changes to this revision.
tpr added a comment.
This revision now requires changes to proceed.

Hi Krzysztof

With my latest fix for the "in the way value" problem, it all passes my local testing.

That latest fix is quite a big diff in pruneSubRegValues because it puts the loops the other way round (outer loop is subranges, inner loop is values). So I figured it would be easier to start a new review https://reviews.llvm.org/D48478 to supersede and subsume this one. Could you head over and review that one please?

Thanks for all your help on this issue.


Repository:
  rL LLVM

https://reviews.llvm.org/D48102





More information about the llvm-commits mailing list