[llvm] [RISCV][GISel] Add ZFA FP legalization and full tests for 9 insn (PR #118723)

[Luke Quinn via llvm-commits]

================
@@ -582,12 +582,19 @@ RISCVLegalizerInfo::RISCVLegalizerInfo(const RISCVSubtarget &ST)
       .libcallFor(ST.is64Bit(), {{s32, s128}, {s64, s128}});
 
   // FIXME: We can do custom inline expansion like SelectionDAG.
-  // FIXME: Legal with Zfa.
   getActionDefinitionsBuilder({G_FCEIL, G_FFLOOR, G_FRINT, G_FNEARBYINT,
                                G_INTRINSIC_TRUNC, G_INTRINSIC_ROUND,
                                G_INTRINSIC_ROUNDEVEN})
+      .legalFor(ST.hasStdExtZfa() /*ST.hasStdExtF()*/, {s32})
----------------
lquinn2015 wrote:

When reading the spec https://github.com/riscv/riscv-isa-manual/blob/main/src/zfa.adoc     it seems that ZFA implies F extension thus checking for F is redundant. Should I remove the comment i agree that it is probably more confusing

https://github.com/llvm/llvm-project/pull/118723