[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