dmgreen added a comment. Nice one. Good to see codegen changes coming out of these intrinsics.
It took a while for me to figure out what the integer instruction was doing. That's a strange one. The fp case I have a question about below. ================ 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)))>; ---------------- If I'm reading the ARMARM correctly, the fp case seems to preform the abs on both operands. Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D72761/new/ https://reviews.llvm.org/D72761 _______________________________________________ cfe-commits mailing list cfe-commits@lists.llvm.org https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
