This is an archive of the discontinued LLVM Phabricator instance.

[RISCV] Use FP ABI on some of the FP tests to reduce the number of CHECK lines. NFC
ClosedPublic

Authored by craig.topper on Jan 8 2022, 12:50 PM.

Details

Summary

These tests are interested in the FP instructions being used, not
the conversions needed to pass the arguments/returns in GPRs.

Diff Detail

Event Timeline

craig.topper created this revision.Jan 8 2022, 12:50 PM
craig.topper requested review of this revision.Jan 8 2022, 12:50 PM
Herald added a project: Restricted Project. · View Herald TranscriptJan 8 2022, 12:50 PM
Herald added a subscriber: MaskRay. · View Herald Transcript
jrtc27 added inline comments.Jan 8 2022, 12:54 PM
llvm/test/CodeGen/RISCV/double-arith-strict.ll
3–4

I assume you meant ilp32d?

craig.topper marked an inline comment as done.Jan 8 2022, 1:40 PM
craig.topper added inline comments.
llvm/test/CodeGen/RISCV/double-arith-strict.ll
3–4

Yes, thank you.

asb accepted this revision.Jan 10 2022, 4:17 AM

Thanks Craig, this is a good cleanup. The use of the soft ABI is just an artifact of the floating point ABI not being implemented until later on in the backend's development.

This revision is now accepted and ready to land.Jan 10 2022, 4:17 AM