[llvm] r329964 - [DAGCombiner] simplify code; NFC

[Sanjay Patel via llvm-commits]

Author: spatel
Date: Thu Apr 12 15:14:58 2018
New Revision: 329964

URL: http://llvm.org/viewvc/llvm-project?rev=329964&view=rev
Log:
[DAGCombiner] simplify code; NFC

Modified:
    llvm/trunk/lib/CodeGen/SelectionDAG/DAGCombiner.cpp

Modified: llvm/trunk/lib/CodeGen/SelectionDAG/DAGCombiner.cpp
URL: http://llvm.org/viewvc/llvm-project/llvm/trunk/lib/CodeGen/SelectionDAG/DAGCombiner.cpp?rev=329964&r1=329963&r2=329964&view=diff
==============================================================================
--- llvm/trunk/lib/CodeGen/SelectionDAG/DAGCombiner.cpp (original)
+++ llvm/trunk/lib/CodeGen/SelectionDAG/DAGCombiner.cpp Thu Apr 12 15:14:58 2018
@@ -673,6 +673,7 @@ static char isNegatibleForFree(SDValue O
   // Don't recurse exponentially.
   if (Depth > 6) return 0;
 
+  EVT VT = Op.getValueType();
   switch (Op.getOpcode()) {
   default: return false;
   case ISD::ConstantFP: {
@@ -681,7 +682,6 @@ static char isNegatibleForFree(SDValue O
 
     // Don't invert constant FP values after legalization unless the target says
     // the negated constant is legal.
-    EVT VT = Op.getValueType();
     return TLI.isOperationLegal(ISD::ConstantFP, VT) ||
       TLI.isFPImmLegal(neg(cast<ConstantFPSDNode>(Op)->getValueAPF()), VT);
   }
@@ -690,8 +690,7 @@ static char isNegatibleForFree(SDValue O
     if (!Options->UnsafeFPMath) return 0;
 
     // After operation legalization, it might not be legal to create new FSUBs.
-    if (LegalOperations &&
-        !TLI.isOperationLegalOrCustom(ISD::FSUB,  Op.getValueType()))
+    if (LegalOperations && !TLI.isOperationLegalOrCustom(ISD::FSUB, VT))
       return 0;
 
     // fold (fneg (fadd A, B)) -> (fsub (fneg A), B)