[PATCH] D71062: [ARM][MVE] Add vector reduction intrinsics with two vector operands

[Mark Murray via Phabricator via cfe-commits]

MarkMurrayARM accepted this revision.
MarkMurrayARM added a comment.

Tiny observation of types - fix only if you feel like it.



================
Comment at: clang/include/clang/Basic/arm_mve.td:845
+
+let params = [s16, s32] in {
+defm vmlaldav : MVEBinaryVectorHoriz64<V.False, V.True, "x">;
----------------
Types again?


================
Comment at: clang/include/clang/Basic/arm_mve.td:854
+
+let params = [s32] in {
+defm vrmlaldavh : MVEBinaryVectorHoriz64R<V.False, V.True, "x">;
----------------
Types again?


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D71062