[llvm] [X86][NVPTX][LegalizeDAG] If i16 legal, legalize FABS/FNEG/FCOPYSIGN (f16) with Expand (PR #106153)

Phoebe Wang via llvm-commits llvm-commits at lists.llvm.org
Tue Aug 27 18:08:25 PDT 2024


================
@@ -601,7 +601,7 @@ X86TargetLowering::X86TargetLowering(const X86TargetMachine &TM,
   auto setF16Action = [&] (MVT VT, LegalizeAction Action) {
     setOperationAction(ISD::FABS, VT, Action);
     setOperationAction(ISD::FNEG, VT, Action);
-    setOperationAction(ISD::FCOPYSIGN, VT, Expand);
+    setOperationAction(ISD::FCOPYSIGN, VT, Action);
----------------
phoebewang wrote:

What's the reason to change this? And why don't set above Expand here?

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


More information about the llvm-commits mailing list