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

[Jon Roelofs via Phabricator via llvm-commits]

jroelofs added a comment.

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

> I don't think there are any cases where it is beneficial to combine `ashr (and x, n), k -> sbfx` (are there?), because for any case where you can do that, it would be better just to remove the `and` and leave it as an `ashr`.

Great point! Thanks for catching that.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D111839