[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