[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:15:28 PDT 2021
foad requested changes to this revision.
foad added a comment.
This revision now requires changes to proceed.
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`.
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