[llvm] [X86][GlobalIsel] support G_FABS for f80 (PR #136718)

Matt Arsenault via llvm-commits llvm-commits at lists.llvm.org
Wed Apr 23 03:26:52 PDT 2025


================
@@ -466,6 +466,11 @@ X86LegalizerInfo::X86LegalizerInfo(const X86Subtarget &STI,
                (UseX87 && typeInSet(0, {s80})(Query));
       });
 
+  //TODO: f32 and f64 FABS require xmm support
+  getActionDefinitionsBuilder(G_FABS)
+      .legalFor(UseX87, {s8, s80})
+      .lower();
----------------
arsenm wrote:

that error comes from the globalisel code, not sdag. You can also just stop testing with those features forced off

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


More information about the llvm-commits mailing list