Fix comment and unnecessary check for FP build_vectors.
authorMatt Arsenault <Matthew.Arsenault@amd.com>
Tue, 2 Sep 2014 18:33:51 +0000 (18:33 +0000)
committerMatt Arsenault <Matthew.Arsenault@amd.com>
Tue, 2 Sep 2014 18:33:51 +0000 (18:33 +0000)
This was copy-paste from the integer version, but
FP build_vectors don't truncate.

git-svn-id: https://llvm.org/svn/llvm-project/llvm/trunk@216928 91177308-0d34-0410-b5e6-96231b3b80d8

lib/CodeGen/SelectionDAG/DAGCombiner.cpp

index 1186683aebafd64fb48dfd47d8505b19ba29f7d1..e556e74980c7a46690566463e36ce8263b7ec93a 100644 (file)
@@ -711,11 +711,7 @@ static ConstantFPSDNode *isConstOrConstSplatFP(SDValue N) {
     BitVector UndefElements;
     ConstantFPSDNode *CN = BV->getConstantFPSplatNode(&UndefElements);
 
-    // BuildVectors can truncate their operands. Ignore that case here.
-    // FIXME: We blindly ignore splats which include undef which is overly
-    // pessimistic.
-    if (CN && UndefElements.none() &&
-        CN->getValueType(0) == N.getValueType().getScalarType())
+    if (CN && UndefElements.none())
       return CN;
   }