[PATCH] D21854: Remove AND mask of bottom 6 bits when result is used for SystemZ shift/rotate

Ulrich Weigand via llvm-commits llvm-commits at lists.llvm.org
Tue Jul 5 09:07:47 PDT 2016


uweigand added a comment.

In http://reviews.llvm.org/D21854#474231, @colpell wrote:

> Update to latest trunk and remove hasOneUse check.


Ah, now it seems the CombineTo use is incorrect (this will replace *all* uses of the AND with its input value, which is incorrect if there is indeed another use).  So I think we have to generate a new shift/rotate output node instead of using CombineTo here.


http://reviews.llvm.org/D21854





More information about the llvm-commits mailing list