craig.topper added inline comments.
================ Comment at: clang/include/clang/Basic/riscv_vector.td:2131 +} + ---------------- Extra blank line ================ Comment at: clang/test/CodeGen/RISCV/rvv-intrinsics-autogenerated/non-policy/non-overloaded/vaadd.c:16 // -vint8mf8_t test_vaadd_vv_i8mf8(vint8mf8_t op1, vint8mf8_t op2, size_t vl) { - return __riscv_vaadd_vv_i8mf8(op1, op2, vl); +vint8mf8_t test_vaadd_vv_i8mf8(vint8mf8_t op1, vint8mf8_t op2, unsigned int vxrm, size_t vl) { + return __riscv_vaadd_vv_i8mf8(op1, op2, VXRM_RNU, vl); ---------------- The vxrm argument is unused ================ Comment at: llvm/include/llvm/IR/IntrinsicsRISCV.td:1365 defm vaaddu : RISCVSaturatingBinaryAAX; - defm vaadd : RISCVSaturatingBinaryAAX; - defm vaadd_rm : RISCVSaturatingBinaryAAXRoundingMode; + defm vaadd : RISCVSaturatingBinaryAAXRoundingMode; defm vasubu : RISCVSaturatingBinaryAAX; ---------------- Why did the intrinsic change names? Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D151397/new/ https://reviews.llvm.org/D151397 _______________________________________________ cfe-commits mailing list cfe-commits@lists.llvm.org https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits