[llvm-commits] [llvm] r140471 - /llvm/trunk/lib/CodeGen/SelectionDAG/LegalizeIntegerTypes.cpp

[Nadav Rotem]

Author: nadav
Date: Sun Sep 25 13:59:42 2011
New Revision: 140471

URL: http://llvm.org/viewvc/llvm-project?rev=140471&view=rev
Log:
[vector-select] Address one of the issues in pr10902. EXTRACT_VECTOR_ELEMENT
SDNodes may return values which are wider than the incoming element types. In
this patch we fix the integer promotion of these nodes.

Fixes spill-q.ll when running -promote-elements.


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

Modified: llvm/trunk/lib/CodeGen/SelectionDAG/LegalizeIntegerTypes.cpp
URL: http://llvm.org/viewvc/llvm-project/llvm/trunk/lib/CodeGen/SelectionDAG/LegalizeIntegerTypes.cpp?rev=140471&r1=140470&r2=140471&view=diff
==============================================================================
--- llvm/trunk/lib/CodeGen/SelectionDAG/LegalizeIntegerTypes.cpp (original)
+++ llvm/trunk/lib/CodeGen/SelectionDAG/LegalizeIntegerTypes.cpp Sun Sep 25 13:59:42 2011
@@ -2987,8 +2987,13 @@
   SDValue Ext = DAG.getNode(ISD::EXTRACT_VECTOR_ELT, dl,
     V0->getValueType(0).getScalarType(), V0, V1);
 
-  return DAG.getNode(ISD::TRUNCATE, dl, N->getValueType(0), Ext);
-
+  // EXTRACT_VECTOR_ELT can return types which are wider than the incoming
+  // element types (see PromoteIntRes_EXTRACT_VECTOR_ELT). If this is the case
+  // then we need to expand the outgoing value and not truncate it.
+  bool trunc = (N->getValueType(0).getSizeInBits() <
+                Ext.getValueType().getSizeInBits());
+  return DAG.getNode(trunc ? ISD::TRUNCATE : ISD::ANY_EXTEND,
+                     dl, N->getValueType(0), Ext);
 }
 
 SDValue DAGTypeLegalizer::PromoteIntOp_CONCAT_VECTORS(SDNode *N) {