[llvm] [X86][NVPTX][LegalizeDAG] If i16 legal, legalize FABS/FNEG/FCOPYSIGN (f16) with Expand (PR #106153)
via llvm-commits
llvm-commits at lists.llvm.org
Tue Aug 27 18:13:34 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);
----------------
v01dXYZ wrote:
Because it would create even more special cases that don't reflect the function name.
The function is called elsewhere with `Expand` unless it's `f16`.
That's just for consistency, I could do it here if it's better.
https://github.com/llvm/llvm-project/pull/106153
More information about the llvm-commits
mailing list