diff options
| author | Craig Topper <craig.topper@intel.com> | 2017-12-20 06:44:50 +0000 |
|---|---|---|
| committer | Craig Topper <craig.topper@intel.com> | 2017-12-20 06:44:50 +0000 |
| commit | 158d54d9545f5fd54aea587732a9cbf7360d6b0e (patch) | |
| tree | c1b9ec0d81069f9df07a7cb2529ff6639bb11ed4 | |
| parent | 11e571e0c68f872052f8827b4ee976a54a200052 (diff) | |
| download | bcm5719-llvm-158d54d9545f5fd54aea587732a9cbf7360d6b0e.tar.gz bcm5719-llvm-158d54d9545f5fd54aea587732a9cbf7360d6b0e.zip | |
[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
| -rw-r--r-- | llvm/lib/Target/X86/X86ISelLowering.cpp | 1 |
1 files changed, 1 insertions, 0 deletions
diff --git a/llvm/lib/Target/X86/X86ISelLowering.cpp b/llvm/lib/Target/X86/X86ISelLowering.cpp index d9a72ebec01..987d77b551b 100644 --- a/llvm/lib/Target/X86/X86ISelLowering.cpp +++ b/llvm/lib/Target/X86/X86ISelLowering.cpp @@ -36258,6 +36258,7 @@ static SDValue combineGatherScatter(SDNode *N, SelectionDAG &DAG, SmallVector<SDValue, 5> NewOps(N->op_begin(), N->op_end()); NewOps[2] = Mask.getOperand(0); DAG.UpdateNodeOperands(N, NewOps); + return SDValue(N, 0); } // With AVX2 we only demand the upper bit of the mask. |
