[PATCH] D120648: [RISCV] DAGcombine fold (bswap(srl (bswap c), x)) -> (shl c, x)

Craig Topper via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Thu Mar 10 17:49:29 PST 2022


craig.topper added a comment.

In D120648#3374242 <https://reviews.llvm.org/D120648#3374242>, @Chenbing.Zheng wrote:

> Ping

We need test cases that are targetted specifically at the change being made. If we add a RISCV DAGCombine for (bitreverse (bswap X)) -> brev8 pre-type legalization, then this code change will become untested.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D120648



More information about the llvm-commits mailing list