[PATCH] D125607: [DAG] Fold (shl (srl x, c), c) -> and(x, m) even if srl has other uses

Jay Foad via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Tue May 17 02:46:29 PDT 2022


foad added a comment.

In D125607#3518370 <https://reviews.llvm.org/D125607#3518370>, @RKSimon wrote:

> @foad @dmgreen Any objections to the AMDGPU / ARM changes?

AMDGPU looks OK to me. Using a literal 0xffff0000 operand increases code size, but on the other had some sequences have one fewer instruction which is nice.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D125607



More information about the llvm-commits mailing list