Skip to content

Commit 158d54d

Browse files
committed
[X86] Add a missing return to combineGatherScatter after sucessful combine.
Not sure how to test this cause I think the worst that happens is that we don't revisit the node a second time to look for additional combines. We used UpdateNodeOperands so the updating the DAG work was already done. llvm-svn: 321148
1 parent 11e571e commit 158d54d

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
@@ -36258,6 +36258,7 @@ static SDValue combineGatherScatter(SDNode *N, SelectionDAG &DAG,
3625836258
SmallVector<SDValue, 5> NewOps(N->op_begin(), N->op_end());
3625936259
NewOps[2] = Mask.getOperand(0);
3626036260
DAG.UpdateNodeOperands(N, NewOps);
36261+
return SDValue(N, 0);
3626136262
}
3626236263

3626336264
// With AVX2 we only demand the upper bit of the mask.

0 commit comments

Comments
 (0)