[PATCH] D72761: [ARM][MVE][Intrinsics] Add VMINAQ, VMINNMAQ, VMAXAQ, VMAXNMAQ intrinsics.

[Mark Murray via Phabricator via cfe-commits]

MarkMurrayARM marked an inline comment as done.
MarkMurrayARM added inline comments.


================
Comment at: llvm/lib/Target/ARM/ARMInstrMVE.td:3658
+    // Unpredicated v(max|min)nma
+    def : Pat<(VTI.Vec (unpred_op (VTI.Vec MQPR:$Qd), (fabs (VTI.Vec MQPR:$Qm)))),
+              (VTI.Vec (Inst (VTI.Vec MQPR:$Qd), (VTI.Vec MQPR:$Qm)))>;
----------------
dmgreen wrote:
> If I'm reading the ARMARM correctly, the fp case seems to preform the abs on both operands.
My bad. Fix coming under separate cover.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D72761