From a1b88ab2c14c1332a864a1fd4c2746bc43d005fe Mon Sep 17 00:00:00 2001 From: Chandler Carruth Date: Wed, 1 Oct 2014 11:13:57 +0000 Subject: [PATCH] [x86] Delete some extraneous logic from the new vector shuffle lowering. 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 | 7 ------- 1 file changed, 7 deletions(-) diff --git a/lib/Target/X86/X86ISelLowering.cpp b/lib/Target/X86/X86ISelLowering.cpp index 3607f525c2e..494c965b7d7 100644 --- a/lib/Target/X86/X86ISelLowering.cpp +++ b/lib/Target/X86/X86ISelLowering.cpp @@ -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