Skip to content

Commit c38c9c1

Browse files
committed
[X86] After turning VSELECT into SHRUNKBLEND, make we push the VSELECT into the worklist so it can be deleted.
Found while trying to figure out why my second version of D56421 worked better than the first version. We weren't deleting the vselect in a timely fashion and that caused SimplfyDemandedBit to see an additional user. The new version doesn't have this problem so this fix isn't needed there, but seemed like the right thing to do. llvm-svn: 350781
1 parent 2f72a75 commit c38c9c1

File tree

1 file changed

+1
-0
lines changed

1 file changed

+1
-0
lines changed

llvm/lib/Target/X86/X86ISelLowering.cpp

Lines changed: 1 addition & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -33927,6 +33927,7 @@ static SDValue combineVSelectToShrunkBlend(SDNode *N, SelectionDAG &DAG,
3392733927
SDValue SB = DAG.getNode(X86ISD::SHRUNKBLEND, SDLoc(U), U->getValueType(0),
3392833928
Cond, U->getOperand(1), U->getOperand(2));
3392933929
DAG.ReplaceAllUsesOfValueWith(SDValue(U, 0), SB);
33930+
DCI.AddToWorklist(U);
3393033931
}
3393133932
DCI.CommitTargetLoweringOpt(TLO);
3393233933
return SDValue(N, 0);

0 commit comments

Comments
 (0)