maxjhandsome
/
llvm-project
forked from OSchip/llvm-project
1
0
Fork 0
Code Issues Pull Requests Packages Releases Wiki Activity

[X86][SSE] combineX86ShufflesRecursively - move all Root node asserts to the same location. NFCI. 

Minor tidyup for some upcoming shuffle combine improvements.
This commit is contained in:
Simon Pilgrim 2020-07-25 12:08:06 +01:00
parent 18d481cdf9
commit 3b21823e4a
1 changed files with 3 additions and 3 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

6
llvm/lib/Target/X86/X86ISelLowering.cpp
View File

@ -35346,6 +35346,9 @@ static SDValue combineX86ShufflesRecursively(
assert(RootMask.size() > 0 &&
(RootMask.size() > 1 || (RootMask[0] == 0 && SrcOpIndex == 0)) &&
"Illegal shuffle root mask");
assert(Root.getSimpleValueType().isVector() &&
"Shuffles operate on vector types!");
unsigned RootSizeInBits = Root.getSimpleValueType().getSizeInBits();
// Bound the depth of our recursive combine because this is ultimately
// quadratic in nature.
@ -35361,9 +35364,6 @@ static SDValue combineX86ShufflesRecursively(
if (!VT.isVector())
return SDValue(); // Bail if we hit a non-vector.
assert(Root.getSimpleValueType().isVector() &&
"Shuffles operate on vector types!");
unsigned RootSizeInBits = Root.getSimpleValueType().getSizeInBits();
assert(VT.getSizeInBits() == RootSizeInBits &&
"Can only combine shuffles of the same vector register size.");