[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