[PATCH] D136158: [AArch64] Adjust operand sequence for Add+Sub to combine more inline shift

Dave Green via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Mon Oct 24 01:23:52 PDT 2022


dmgreen added a comment.

Have you tried testing this? I think it will get stuck in DAG combines.

Otherwise the idea sounds good. Have you considered doing it for GlobalISel, or in a way that GlobalISel would also benefit?


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

https://reviews.llvm.org/D136158



More information about the llvm-commits mailing list