[PATCH] D117969: [RISCV] Use FP ABI for some RVV intrinsic tests. NFC

Sat Jan 22 22:21:00 PST 2022

craig.topper added a comment.

In D117969#3264104 <https://reviews.llvm.org/D117969#3264104>, @eopXD wrote:

> LGTM. My understanding to the test case changed because with the specified ABI, the function parameter will already be stored in `fa0` so we don't need an extra `fmv`.
> Thank you for the patch.

That's correct. It should make the test cases easier to merge for rv32/rv64 now that rv32 isn't generating different code for doubles due to needing two GPRs.

Repository:
  rG LLVM Github Monorepo

CHANGES SINCE LAST ACTION
  https://reviews.llvm.org/D117969/new/

https://reviews.llvm.org/D117969