Fix a bug in SelectScalarSSELoad. Since the load is wrapped in a
authorEvan Cheng <evan.cheng@apple.com>
Fri, 10 Nov 2006 21:23:04 +0000 (21:23 +0000)
committerEvan Cheng <evan.cheng@apple.com>
Fri, 10 Nov 2006 21:23:04 +0000 (21:23 +0000)
SCALAR_TO_VECTOR, even if the hasOneUse() check pass we may end up folding
the load into two instructions. Make sure we check the SCALAR_TO_VECTOR
has only one use as well.

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

lib/Target/X86/X86ISelDAGToDAG.cpp

index b8f54145722bc827dcd0f39e7e9766d455620ed5..68b89a4d160f48f81be2466eedc1acb21f696ddf 100644 (file)
@@ -815,6 +815,7 @@ bool X86DAGToDAGISel::SelectScalarSSELoad(SDOperand Op, SDOperand Pred,
     InChain = N.getOperand(0).getValue(1);
     if (ISD::isNON_EXTLoad(InChain.Val) &&
         InChain.getValue(0).hasOneUse() &&
+        N.hasOneUse() &&
         CanBeFoldedBy(N.Val, Pred.Val, Op.Val)) {
       LoadSDNode *LD = cast<LoadSDNode>(InChain);
       if (!SelectAddr(Op, LD->getBasePtr(), Base, Scale, Index, Disp))