[PATCH] D111839: [AArch64][GlobalISel] combine and + [la]sr => [su]bfx

Jay Foad via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Fri Oct 15 02:17:18 PDT 2021


foad added a comment.

In D111839#3066249 <https://reviews.llvm.org/D111839#3066249>, @foad wrote:

> it would be better just to remove the `and` and leave it as an `ashr`.

... and that should be done by a general demanded bits analysis, which could spot that the `and` is redundant.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D111839



More information about the llvm-commits mailing list