[x86] Delete some extraneous logic from the new vector shuffle lowering.
authorChandler Carruth <chandlerc@gmail.com>
Wed, 1 Oct 2014 11:13:57 +0000 (11:13 +0000)
committerChandler Carruth <chandlerc@gmail.com>
Wed, 1 Oct 2014 11:13:57 +0000 (11:13 +0000)
Nothing was relying on this and there are potentially some edge cases
that it would not be correct under. Removing it seems better than trying
to "fix" it as nothing was relying on it.

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

lib/Target/X86/X86ISelLowering.cpp

index 3607f525c2ed1c77a2e199ccd55cb6e6a7c52971..494c965b7d755fa6063c5ecef99f5607778c1281 100644 (file)
@@ -7768,13 +7768,6 @@ static SDValue lowerVectorShuffleAsElementInsertion(
         return SDValue(); // Not inserting into a zero vector.
   }
 
-  // Step over any bitcasts on either input so we can scan the actual
-  // BUILD_VECTOR nodes.
-  while (V1.getOpcode() == ISD::BITCAST)
-    V1 = V1.getOperand(0);
-  while (V2.getOpcode() == ISD::BITCAST)
-    V2 = V2.getOperand(0);
-
   // Check for a single input from a SCALAR_TO_VECTOR node.
   // FIXME: All of this should be canonicalized into INSERT_VECTOR_ELT and
   // all the smarts here sunk into that routine. However, the current