craig.topper added inline comments.
================
Comment at: clang/lib/Sema/SemaChecking.cpp:4816
+ case RISCVVector::BI__builtin_rvv_vfadd_vf_rm_tu:
+ case RISCVVector::BI__builtin_rvv_vfadd_vv_rm_tama:
+ case RISCVVector::BI__builtin_rvv_vfadd_vf_rm_tama:
----------------
not related to this patch, but the tama here doesn't match the intrinsic naming.
================
Comment at: llvm/lib/Target/RISCV/RISCVInstrInfoVPseudos.td:5740
+let mayRaiseFPException = true,
+ hasPostISelHook = 1 in {
+defm PseudoVFADD : VPseudoVALU_VV_VF_RM;
----------------
Keep the `let` on one line
Repository:
rG LLVM Github Monorepo
CHANGES SINCE LAST ACTION
https://reviews.llvm.org/D152996/new/
https://reviews.llvm.org/D152996
_______________________________________________
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
