[llvm] r231237 - [mips] Specify the correct value type when combining a CMovFP node.

Vasileios Kalintiris Vasileios.Kalintiris at imgtec.com
Wed Mar 4 04:10:19 PST 2015


Author: vkalintiris
Date: Wed Mar  4 06:10:18 2015
New Revision: 231237

URL: http://llvm.org/viewvc/llvm-project?rev=231237&view=rev
Log:
[mips] Specify the correct value type when combining a CMovFP node.

This commit fixes a bug introduced in r230956 where we were creating
CMovFP_{T,F} nodes with multiple return value types (one for each operand).
With this change the return value type of the new node is the same as the
value type of the True/False operands of the original node.

Modified:
    llvm/trunk/lib/Target/Mips/MipsISelLowering.cpp

Modified: llvm/trunk/lib/Target/Mips/MipsISelLowering.cpp
URL: http://llvm.org/viewvc/llvm-project/llvm/trunk/lib/Target/Mips/MipsISelLowering.cpp?rev=231237&r1=231236&r2=231237&view=diff
==============================================================================
--- llvm/trunk/lib/Target/Mips/MipsISelLowering.cpp (original)
+++ llvm/trunk/lib/Target/Mips/MipsISelLowering.cpp Wed Mar  4 06:10:18 2015
@@ -640,10 +640,8 @@ static SDValue performCMovFPCombine(SDNo
                                                          MipsISD::CMovFP_T;
 
   SDValue FCC = N->getOperand(1), Glue = N->getOperand(3);
-  SDVTList VTs = DAG.getVTList(FCC.getValueType(), ValueIfTrue.getValueType(),
-                               ValueIfFalse.getValueType(),
-                               Glue.getValueType());
-  return DAG.getNode(Opc, SDLoc(N), VTs, ValueIfFalse, FCC, ValueIfTrue, Glue);
+  return DAG.getNode(Opc, SDLoc(N), ValueIfFalse.getValueType(),
+                     ValueIfFalse, FCC, ValueIfTrue, Glue);
 }
 
 static SDValue performANDCombine(SDNode *N, SelectionDAG &DAG,





More information about the llvm-commits mailing list