diff options
author | Simon Pilgrim <llvm-dev@redking.me.uk> | 2018-11-16 15:01:05 +0000 |
---|---|---|
committer | Simon Pilgrim <llvm-dev@redking.me.uk> | 2018-11-16 15:01:05 +0000 |
commit | bcd6631a2a761286ce4b09baeefa143214cfcda2 (patch) | |
tree | 0fcb10d2d274e25b6744a446a26c2b620d3266a9 | |
parent | 12354c910c7174d1c092c313ca91a92193207803 (diff) | |
download | bcm5719-llvm-bcd6631a2a761286ce4b09baeefa143214cfcda2.tar.gz bcm5719-llvm-bcd6631a2a761286ce4b09baeefa143214cfcda2.zip |
[X86][SSE] Move number of input limit out of resolveTargetShuffleInputs.
Only combineX86ShufflesRecursively needs this limit.
llvm-svn: 347054
-rw-r--r-- | llvm/lib/Target/X86/X86ISelLowering.cpp | 8 |
1 files changed, 5 insertions, 3 deletions
diff --git a/llvm/lib/Target/X86/X86ISelLowering.cpp b/llvm/lib/Target/X86/X86ISelLowering.cpp index 703747b885d..ab5cb594445 100644 --- a/llvm/lib/Target/X86/X86ISelLowering.cpp +++ b/llvm/lib/Target/X86/X86ISelLowering.cpp @@ -6627,8 +6627,7 @@ static bool resolveTargetShuffleInputs(SDValue Op, return false; resolveTargetShuffleInputsAndMask(Inputs, Mask); - // TODO - Add support for more than 2 inputs. - return Inputs.size() <= 2; + return true; } /// Returns the scalar element that will make up the ith @@ -31005,7 +31004,10 @@ static SDValue combineX86ShufflesRecursively( if (!resolveTargetShuffleInputs(Op, OpInputs, OpMask, DAG)) return SDValue(); - assert(OpInputs.size() <= 2 && "Too many shuffle inputs"); + // TODO - Add support for more than 2 inputs. + if (2 < OpInputs.size()) + return SDValue(); + SDValue Input0 = (OpInputs.size() > 0 ? OpInputs[0] : SDValue()); SDValue Input1 = (OpInputs.size() > 1 ? OpInputs[1] : SDValue()); |