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

[SelectionDAG] Allow custom vector widening through ReplaceNodeResults to handle nodes with chain outputs. 

Previously we were assuming all results were vectors and calling SetWidenedVector, but if its a chain result we should just replace uses instead.

This fixes an error found by expensive checks after r318368.

llvm-svn: 318509
This commit is contained in:
Craig Topper 2017-11-17 07:03:57 +00:00
parent 1ac6e8ae61
commit e9a6456ab3
1 changed files with 7 additions and 2 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

9
llvm/lib/CodeGen/SelectionDAG/LegalizeTypes.cpp
View File

@ -1027,8 +1027,13 @@ bool DAGTypeLegalizer::CustomWidenLowerNode(SDNode *N, EVT VT) {
// Update the widening map.
assert(Results.size() == N->getNumValues() &&
"Custom lowering returned the wrong number of results!");
for (unsigned i = 0, e = Results.size(); i != e; ++i)
SetWidenedVector(SDValue(N, i), Results[i]);
for (unsigned i = 0, e = Results.size(); i != e; ++i) {
// If this is a chain output just replace it.
if (Results[i].getValueType() == MVT::Other)
ReplaceValueWith(SDValue(N, i), Results[i]);
else
SetWidenedVector(SDValue(N, i), Results[i]);
}
return true;
}