[llvm-commits] CVS: llvm/lib/CodeGen/SelectionDAG/SelectionDAGISel.cpp

Evan Cheng evan.cheng at apple.com
Fri May 26 16:13:32 PDT 2006



Changes in directory llvm/lib/CodeGen/SelectionDAG:

SelectionDAGISel.cpp updated: 1.252 -> 1.253
---
Log message:

Make CALL node consistent with RET node. Signness of value has type MVT::i32
instead of MVT::i1. Either is fine except MVT::i32 is probably a legal type
for most (if not all) platforms while MVT::i1 is not.


---
Diffs of the changes:  (+5 -5)

 SelectionDAGISel.cpp |   10 +++++-----
 1 files changed, 5 insertions(+), 5 deletions(-)


Index: llvm/lib/CodeGen/SelectionDAG/SelectionDAGISel.cpp
diff -u llvm/lib/CodeGen/SelectionDAG/SelectionDAGISel.cpp:1.252 llvm/lib/CodeGen/SelectionDAG/SelectionDAGISel.cpp:1.253
--- llvm/lib/CodeGen/SelectionDAG/SelectionDAGISel.cpp:1.252	Fri May 26 18:08:15 2006
+++ llvm/lib/CodeGen/SelectionDAG/SelectionDAGISel.cpp	Fri May 26 18:13:20 2006
@@ -2515,7 +2515,7 @@
     default: assert(0 && "Unknown type action!");
     case Legal: 
       Ops.push_back(Op);
-      Ops.push_back(DAG.getConstant(isSigned, MVT::i1));
+      Ops.push_back(DAG.getConstant(isSigned, MVT::i32));
       break;
     case Promote:
       if (MVT::isInteger(VT)) {
@@ -2526,7 +2526,7 @@
         Op = DAG.getNode(ISD::FP_EXTEND, getTypeToTransformTo(VT), Op);
       }
       Ops.push_back(Op);
-      Ops.push_back(DAG.getConstant(isSigned, MVT::i1));
+      Ops.push_back(DAG.getConstant(isSigned, MVT::i32));
       break;
     case Expand:
       if (VT != MVT::Vector) {
@@ -2544,9 +2544,9 @@
             std::swap(Lo, Hi);
           
           Ops.push_back(Lo);
-          Ops.push_back(DAG.getConstant(isSigned, MVT::i1));
+          Ops.push_back(DAG.getConstant(isSigned, MVT::i32));
           Ops.push_back(Hi);
-          Ops.push_back(DAG.getConstant(isSigned, MVT::i1));
+          Ops.push_back(DAG.getConstant(isSigned, MVT::i32));
         } else {
           // Value scalarized into many values.  Unimp for now.
           assert(0 && "Cannot expand i64 -> i16 yet!");
@@ -2565,7 +2565,7 @@
           // Insert a VBIT_CONVERT of the MVT::Vector type to the packed type.
           Op = DAG.getNode(ISD::VBIT_CONVERT, TVT, Op);
           Ops.push_back(Op);
-          Ops.push_back(DAG.getConstant(isSigned, MVT::i1));
+          Ops.push_back(DAG.getConstant(isSigned, MVT::i32));
         } else {
           assert(0 && "Don't support illegal by-val vector call args yet!");
           abort();






More information about the llvm-commits mailing list