[llvm-commits] [llvm] r59648 - /llvm/trunk/lib/Target/CellSPU/SPUISelLowering.cpp

[Scott Michel]

Author: pingbak
Date: Wed Nov 19 11:45:08 2008
New Revision: 59648

URL: http://llvm.org/viewvc/llvm-project?rev=59648&view=rev
Log:
CellSPU: Do not custom lower i1 stores, rely on type legalization to do the
right thing and promote the store to i8.

Modified:
    llvm/trunk/lib/Target/CellSPU/SPUISelLowering.cpp

Modified: llvm/trunk/lib/Target/CellSPU/SPUISelLowering.cpp
URL: http://llvm.org/viewvc/llvm-project/llvm/trunk/lib/Target/CellSPU/SPUISelLowering.cpp?rev=59648&r1=59647&r2=59648&view=diff

==============================================================================
--- llvm/trunk/lib/Target/CellSPU/SPUISelLowering.cpp (original)
+++ llvm/trunk/lib/Target/CellSPU/SPUISelLowering.cpp Wed Nov 19 11:45:08 2008
@@ -131,20 +131,14 @@
   addRegisterClass(MVT::i128, SPU::GPRCRegisterClass);
 
   // SPU has no sign or zero extended loads for i1, i8, i16:
-#if 0
   setLoadExtAction(ISD::EXTLOAD,  MVT::i1, Promote);
   setLoadExtAction(ISD::SEXTLOAD, MVT::i1, Promote);
   setLoadExtAction(ISD::ZEXTLOAD, MVT::i1, Promote);
-  setTruncStoreAction(MVT::i8, MVT::i1, Custom);
-  setTruncStoreAction(MVT::i16, MVT::i1, Custom);
-  setTruncStoreAction(MVT::i32, MVT::i1, Custom);
-  setTruncStoreAction(MVT::i64, MVT::i1, Custom);
-  setTruncStoreAction(MVT::i128, MVT::i1, Custom);
-#else
-  setLoadExtAction(ISD::EXTLOAD,  MVT::i1, Promote);
-  setLoadExtAction(ISD::SEXTLOAD,  MVT::i1, Promote);
-  setTruncStoreAction(MVT::i8, MVT::i1, Promote);
-#endif
+  setTruncStoreAction(MVT::i8,    MVT::i1, Promote);
+  setTruncStoreAction(MVT::i16 ,  MVT::i1, Custom);
+  setTruncStoreAction(MVT::i32 ,  MVT::i1, Custom);
+  setTruncStoreAction(MVT::i64 ,  MVT::i1, Custom);
+  setTruncStoreAction(MVT::i128,  MVT::i1, Custom);
 
   setLoadExtAction(ISD::EXTLOAD,  MVT::i8, Custom);
   setLoadExtAction(ISD::SEXTLOAD, MVT::i8, Custom);
@@ -165,7 +159,7 @@
   setOperationAction(ISD::ConstantFP, MVT::f64, Custom);
 
   // SPU's loads and stores have to be custom lowered:
-  for (unsigned sctype = (unsigned) MVT::i1; sctype < (unsigned) MVT::f128;
+  for (unsigned sctype = (unsigned) MVT::i8; sctype < (unsigned) MVT::f128;
        ++sctype) {
     MVT VT = (MVT::SimpleValueType)sctype;
 
@@ -708,12 +702,8 @@
 
     // The vector type we really want to load from the 16-byte chunk, except
     // in the case of MVT::i1, which has to be v16i8.
-    MVT vecVT = MVT::v16i8, stVecVT = MVT::v16i8;
-
-    if (StVT != MVT::i1) {
-      stVecVT = MVT::getVectorVT(StVT, (128 / StVT.getSizeInBits()));
-      vecVT = MVT::getVectorVT(VT, (128 / VT.getSizeInBits()));
-    }
+    MVT vecVT = MVT::getVectorVT(VT, (128 / VT.getSizeInBits())),
+        stVecVT = MVT::getVectorVT(StVT, (128 / StVT.getSizeInBits()));
 
     SDValue alignLoadVec =
       AlignedLoad(Op, DAG, ST, SN, alignment,
@@ -759,21 +749,8 @@
 
     SDValue insertEltOp =
             DAG.getNode(SPUISD::INSERT_MASK, stVecVT, insertEltPtr);
-    SDValue vectorizeOp;
-
-#if 0
-    if (VT == MVT::i1 || StVT != VT) {
-      MVT toVT = (VT != MVT::i1) ? VT : MVT::i8;
-      if (toVT.bitsGT(VT)) {
-        vectorizeOp = DAG.getNode(ISD::ANY_EXTEND, toVT, theValue);
-      } else if (StVT.bitsLT(VT)) {
-        vectorizeOp = DAG.getNode(ISD::TRUNCATE, toVT, theValue);
-      }
-
-      vectorizeOp = DAG.getNode(ISD::SCALAR_TO_VECTOR, vecVT, vectorizeOp);
-    } else
-#endif
-      vectorizeOp = DAG.getNode(ISD::SCALAR_TO_VECTOR, vecVT, theValue);
+    SDValue vectorizeOp =
+            DAG.getNode(ISD::SCALAR_TO_VECTOR, vecVT, theValue);
 
     result = DAG.getNode(SPUISD::SHUFB, vecVT, vectorizeOp, alignLoadVec,
                          DAG.getNode(ISD::BIT_CONVERT, vecVT, insertEltOp));
@@ -782,7 +759,7 @@
                           LN->getSrcValue(), LN->getSrcValueOffset(),
                           LN->isVolatile(), LN->getAlignment());
 
-#ifndef NDEBUG
+#if 0 && defined(NDEBUG)
     if (DebugFlag && isCurrentDebugType(DEBUG_TYPE)) {
       const SDValue &currentRoot = DAG.getRoot();