eopXD added a comment. I think the topic of this patch is not accurate. The compiler already support these intrinsics with `zvfh` specified. Implementing `zvfhmin` is relaxing the implementation and let come the intrinsics available when only `zvfhmin` and not `zvfm` is specified. So I think we can adjust the patch description a bit.
I think you may relax the extension specified, which is currently `zvfh`, to using `zvfhmin`, under the existing test cases under `clang/test/CodeGen/RISCV/rvv-intrinsics-autogenerated` to verify your implementation. Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D151869/new/ https://reviews.llvm.org/D151869 _______________________________________________ cfe-commits mailing list cfe-commits@lists.llvm.org https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits