[PATCH] D147236: [AArch64][Combine]: combine <2xi64> Mul-Add.

Dave Green via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Mon Apr 3 01:20:57 PDT 2023


dmgreen added a comment.

This sounds like it will be good for performance. I think this needs to be more careful about what it extracts from though. It probably needs a check that we have SVE, and that the extract is from the bottom (index 0) of a scalable vector. (The "scalable vector" might imply the "have-SVE" part though). But otherwise it could trigger in a lot more cases than it should.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D147236



More information about the llvm-commits mailing list