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

[LegalizeTypes] Check for already split condition before calilng SplitVecRes_SETCC in SplitRes_SELECT. 

No point in manually splitting the SETCC if it was already done.

llvm-svn: 373535
This commit is contained in:
Craig Topper 2019-10-02 22:34:49 +00:00
parent 641ecbd014
commit 2772b970e3
1 changed files with 4 additions and 4 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

8
llvm/lib/CodeGen/SelectionDAG/LegalizeTypesGeneric.cpp
View File

@ -514,15 +514,15 @@ void DAGTypeLegalizer::SplitRes_SELECT(SDNode *N, SDValue &Lo, SDValue &Hi) {
if (Cond.getValueType().isVector()) {
if (SDValue Res = WidenVSELECTAndMask(N))
std::tie(CL, CH) = DAG.SplitVector(Res->getOperand(0), dl);
// It seems to improve code to generate two narrow SETCCs as opposed to
// splitting a wide result vector.
else if (Cond.getOpcode() == ISD::SETCC)
SplitVecRes_SETCC(Cond.getNode(), CL, CH);
// Check if there are already splitted versions of the vector available and
// use those instead of splitting the mask operand again.
else if (getTypeAction(Cond.getValueType()) ==
TargetLowering::TypeSplitVector)
GetSplitVector(Cond, CL, CH);
// It seems to improve code to generate two narrow SETCCs as opposed to
// splitting a wide result vector.
else if (Cond.getOpcode() == ISD::SETCC)
SplitVecRes_SETCC(Cond.getNode(), CL, CH);
else
std::tie(CL, CH) = DAG.SplitVector(Cond, dl);
}