[PATCH] D135850: [AArch64] Enhance 'isBitfieldPositioningOp' to find pattern (shl(and(val,mask), N)

Mingming Liu via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Fri Oct 14 15:51:41 PDT 2022


mingmingl added a comment.

In D135850#3857910 <https://reviews.llvm.org/D135850#3857910>, @dmgreen wrote:

> This seems similar to a demand bits fold. Can it check that `AndMask` is a superset of `AllOnes >> Shift`?

Checking `AndMask` is a superset of `AllOnes  >> Shift` makes sense. Done by checking `isMask_64((AndImm << Shift) >> Shift)`.

- This patch previously introduced a bug (fixed in the latest reply) by checking `isShiftedMask_64((AndImm << Shift) >> Shift)`.

Also make refactor patch a parent of this one (bypassing D135844 <https://reviews.llvm.org/D135844>).


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

https://reviews.llvm.org/D135850



More information about the llvm-commits mailing list