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

Michael Maitland via llvm-commits llvm-commits at lists.llvm.org
Thu Dec 5 06:52:50 PST 2024


================
@@ -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})
----------------
michaelmaitland wrote:

Is it intentional for ST.hasStdExtF to be commented out? I am a little confused on its meaning as a comment.

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


More information about the llvm-commits mailing list