This is an archive of the discontinued LLVM Phabricator instance.

[RISCV] Use FP ABI for some RVV intrinsic tests. NFC
ClosedPublic

Authored by craig.topper on Jan 22 2022, 1:59 PM.

Details

Summary

Removes moves from GPR to FPR and improves f64 tests on RV32.

Diff Detail

Event Timeline

craig.topper created this revision.Jan 22 2022, 1:59 PM
craig.topper requested review of this revision.Jan 22 2022, 1:59 PM
Herald added a project: Restricted Project. · View Herald TranscriptJan 22 2022, 1:59 PM
Herald added a subscriber: MaskRay. · View Herald Transcript
eopXD accepted this revision.Jan 22 2022, 8:17 PM

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.

This revision is now accepted and ready to land.Jan 22 2022, 8:17 PM
This revision was automatically updated to reflect the committed changes.

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.