[llvm] DAG: Call SimplifyDemandedBits on copysign value operand (PR #97180)

Matt Arsenault via llvm-commits llvm-commits at lists.llvm.org
Tue Jul 2 08:28:38 PDT 2024


================
@@ -17571,6 +17571,11 @@ SDValue DAGCombiner::visitFCOPYSIGN(SDNode *N) {
                            APInt::getSignMask(SignVT.getScalarSizeInBits())))
     return SDValue(N, 0);
 
+  // We only take the non-sign bits from the value operand
+  if (SimplifyDemandedBits(N0,
----------------
arsenm wrote:

Probably, but SimplifyDemandedBits doesn't handle fabs, fneg or copysign now 

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


More information about the llvm-commits mailing list