[PATCH] D30558: Fix value numbers in successor blocks if liveout number has changed

Matthias Braun via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Fri Mar 10 11:08:03 PST 2017


MatzeB added a comment.

In https://reviews.llvm.org/D30558#691892, @rampitec wrote:

> The LR before handleMove: [16r,32r:0)[32r,96r:1)[112B,128r:1).
>  After handleMoveDown: [8r,32r:0)[32r,96r:1)[112B,128r:1).
>  After handleMoveUp: [8r,16r:1)[16r,96r:0)[112B,128r:1).


Yep, this is a bug, it should look like:

[8r,16r:0)[16r,96r:1)[112B,128r:1) after handleMoveUp and I see no reason why it couldn't.


Repository:
  rL LLVM

https://reviews.llvm.org/D30558





More information about the llvm-commits mailing list