This is an archive of the discontinued LLVM Phabricator instance.

[RISCV] support fcmp in zfinx,zdinx,zhinx
AbandonedPublic

Authored by realqhc on Mar 16 2022, 12:41 AM.

Details

Summary

Co-Author: @liaolucy @s
add support for fcmp in zfinx and its double and half instructions.

Diff Detail

Event Timeline

realqhc created this revision.Mar 16 2022, 12:41 AM
Herald added a project: Restricted Project. · View Herald TranscriptMar 16 2022, 12:41 AM
realqhc requested review of this revision.Mar 16 2022, 12:41 AM
realqhc updated this revision to Diff 415723.Mar 16 2022, 1:07 AM

[RISCV] Remove fcmp-strict instructions in zfinx, zdinx, and zhinx

realqhc updated this revision to Diff 415773.Mar 16 2022, 3:55 AM

[RISCV] remove strict_fcmp and arith

realqhc updated this revision to Diff 415781.Mar 16 2022, 4:39 AM

[RISCV] revert target-abi test and abi error string format

realqhc edited the summary of this revision. (Show Details)Mar 16 2022, 4:42 AM
craig.topper added inline comments.Mar 16 2022, 9:16 PM
llvm/lib/Target/RISCV/RISCVISelLowering.cpp
101

I think you need to check Subtarget.is64Bit() here. You can't assign a type to two register classes. The type is used as the key to a map. The second call overwrites the first.

realqhc updated this revision to Diff 417829.Mar 23 2022, 10:28 PM
realqhc edited the summary of this revision. (Show Details)

address concerns, add test for rv32

realqhc updated this revision to Diff 417831.Mar 23 2022, 11:00 PM
realqhc marked an inline comment as done.

revert changes to buildpairf64pesudo, fix test for rv32 zdinx

realqhc updated this revision to Diff 417833.Mar 23 2022, 11:15 PM

revert changes to splitpairf64pesudo, remove SplitPairF64Pseudo_INX

realqhc abandoned this revision.Jul 4 2023, 1:55 AM
evandro removed a subscriber: evandro.Jul 6 2023, 1:54 PM