[PATCH] D138107: [AArch64][MachineCombiner] Update isAssociativeAndCommutative

Dave Green via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Wed Nov 16 03:11:15 PST 2022


dmgreen added a comment.

I think it would be best to split this up into some instruction groups. The scalar were already being handled by https://reviews.llvm.org/D134260 (ANDS can also be added). I hadn't committed that because it ran into the same pcsections test failure. I have a patch to fix it up though, I can put that up for review and get D134260 <https://reviews.llvm.org/D134260> committed.

The others could be split into scalar fp/neon/sve/etc.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D138107



More information about the llvm-commits mailing list