[PATCH] D122010: [InstCombine] try to canonicalize logical shift after bswap

Sanjay Patel via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Fri Mar 18 09:30:21 PDT 2022


spatel added a comment.

In D122010#3392573 <https://reviews.llvm.org/D122010#3392573>, @RKSimon wrote:

> I haven't had chance to look at https://github.com/llvm/llvm-project/issues/53867 yet but this has a lot in common with it (and feel free to grab it if you have the bandwidth) - please can you add test coverage for that as well?

Sure - if we canonicalize in this direction, then that problem won't be solved by this patch, but I think it becomes a simpler fold (we don't have to worry about the trailing trunc/demanded bits):
https://alive2.llvm.org/ce/z/zC5Gez


CHANGES SINCE LAST ACTION
  https://reviews.llvm.org/D122010/new/

https://reviews.llvm.org/D122010



More information about the llvm-commits mailing list