[PATCH] D151869: [RISCV] Support more builtin for zvfhmin.

Yueh-Ting (eop) Chen via Phabricator via cfe-commits cfe-commits at lists.llvm.org
Thu Jun 1 10:45:03 PDT 2023


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



More information about the cfe-commits mailing list