[llvm] b9443cb - [X86] narrowExtractedVectorSelect - don't peek through bitcasts to find source vector

[Simon Pilgrim via llvm-commits]

Author: Simon Pilgrim
Date: 2022-05-31T14:57:18+01:00
New Revision: b9443cb6fa6ba0be030cb0ed5f556d69c7f7de56

URL: https://github.com/llvm/llvm-project/commit/b9443cb6fa6ba0be030cb0ed5f556d69c7f7de56
DIFF: https://github.com/llvm/llvm-project/commit/b9443cb6fa6ba0be030cb0ed5f556d69c7f7de56.diff

LOG: [X86] narrowExtractedVectorSelect - don't peek through bitcasts to find source vector

We don't seem to need this for any test coverage and it was making tracking of the uses() of the source vector more difficult

Noticed while investigating Issue #55648

Added: 
    

Modified: 
    llvm/lib/Target/X86/X86ISelLowering.cpp

Removed: 
    


################################################################################
diff  --git a/llvm/lib/Target/X86/X86ISelLowering.cpp b/llvm/lib/Target/X86/X86ISelLowering.cpp
index 324161e590d9..fda8a22c1991 100644
--- a/llvm/lib/Target/X86/X86ISelLowering.cpp
+++ b/llvm/lib/Target/X86/X86ISelLowering.cpp
@@ -53722,7 +53722,7 @@ static SDValue combineINSERT_SUBVECTOR(SDNode *N, SelectionDAG &DAG,
 /// This function should only be called with legal types (otherwise, the calls
 /// to get simple value types will assert).
 static SDValue narrowExtractedVectorSelect(SDNode *Ext, SelectionDAG &DAG) {
-  SDValue Sel = peekThroughBitcasts(Ext->getOperand(0));
+  SDValue Sel = Ext->getOperand(0);
   SmallVector<SDValue, 4> CatOps;
   if (Sel.getOpcode() != ISD::VSELECT ||
       !collectConcatOps(Sel.getOperand(0).getNode(), CatOps, DAG))