[PATCH] Replace redundant code in DAGCombiner

Matt Arsenault Matthew.Arsenault at amd.com
Thu May 9 10:41:58 PDT 2013


Use EVT::changeExtendedVectorElementTypeToInteger instead of doing the same thing that it does

http://llvm-reviews.chandlerc.com/D773

Files:
  lib/CodeGen/SelectionDAG/DAGCombiner.cpp

Index: lib/CodeGen/SelectionDAG/DAGCombiner.cpp
===================================================================
--- lib/CodeGen/SelectionDAG/DAGCombiner.cpp
+++ lib/CodeGen/SelectionDAG/DAGCombiner.cpp
@@ -4526,16 +4526,11 @@
         return DAG.getSetCC(N->getDebugLoc(), VT, N0.getOperand(0),
                              N0.getOperand(1),
                              cast<CondCodeSDNode>(N0.getOperand(2))->get());
+
       // If the desired elements are smaller or larger than the source
       // elements we can use a matching integer vector type and then
       // truncate/sign extend
-      EVT MatchingElementType =
-        EVT::getIntegerVT(*DAG.getContext(),
-                          N0VT.getScalarType().getSizeInBits());
-      EVT MatchingVectorType =
-        EVT::getVectorVT(*DAG.getContext(), MatchingElementType,
-                         N0VT.getVectorNumElements());
-
+      EVT MatchingVectorType = N0VT.changeVectorElementTypeToInteger();
       if (SVT == MatchingVectorType) {
         SDValue VsetCC = DAG.getSetCC(N->getDebugLoc(), MatchingVectorType,
                                N0.getOperand(0), N0.getOperand(1),
-------------- next part --------------
A non-text attachment was scrubbed...
Name: D773.1.patch
Type: text/x-patch
Size: 1159 bytes
Desc: not available
URL: <http://lists.llvm.org/pipermail/llvm-commits/attachments/20130509/eb5fc52c/attachment.bin>


More information about the llvm-commits mailing list