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

When expanding an operand, it's not the result value 

type that matters but the operand type.  This fixes
2008-01-08-IllegalCMP.ll which crashed with the new
legalize infrastructure because SETCC with result
type i8 and operand type i64 was being custom expanded
by the X86 backend.  With this fix, the gcc build gets
as far as the first libcall.

llvm-svn: 46525
This commit is contained in:
Duncan Sands 2008-01-29 19:29:08 +00:00
parent 48085b86ae
commit 032a5d2690
1 changed files with 2 additions and 2 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

4
llvm/lib/CodeGen/SelectionDAG/LegalizeTypesExpand.cpp
View File

@ -747,8 +747,8 @@ bool DAGTypeLegalizer::ExpandOperand(SDNode *N, unsigned OpNo) {
DEBUG(cerr << "Expand node operand: "; N->dump(&DAG); cerr << "\n"); DEBUG(cerr << "Expand node operand: "; N->dump(&DAG); cerr << "\n");
SDOperand Res(0, 0); SDOperand Res(0, 0);
if (TLI.getOperationAction(N->getOpcode(), N->getValueType(0)) == if (TLI.getOperationAction(N->getOpcode(), N->getOperand(OpNo).getValueType())
TargetLowering::Custom) == TargetLowering::Custom)
Res = TLI.LowerOperation(SDOperand(N, 0), DAG); Res = TLI.LowerOperation(SDOperand(N, 0), DAG);
if (Res.Val == 0) { if (Res.Val == 0) {