diff --git a/lib/Target/PowerPC/PPC32ISelSimple.cpp b/lib/Target/PowerPC/PPC32ISelSimple.cpp index 1dd946a421f..0e8b7b04c56 100644 --- a/lib/Target/PowerPC/PPC32ISelSimple.cpp +++ b/lib/Target/PowerPC/PPC32ISelSimple.cpp @@ -502,6 +502,10 @@ unsigned PPC32ISel::getReg(Value *V, MachineBasicBlock *MBB, unsigned Reg = makeAnotherReg(V->getType()); copyConstantToRegister(MBB, IPt, C, Reg); return Reg; + } else if (CastInst *CI = dyn_cast(V)) { + // Do not emit noop casts at all, unless it's a double -> float cast. + if (getClassB(CI->getType()) == getClassB(CI->getOperand(0)->getType())) + return getReg(CI->getOperand(0), MBB, IPt); } else if (AllocaInst *AI = dyn_castFixedAlloca(V)) { unsigned Reg = makeAnotherReg(V->getType()); unsigned FI = getFixedSizedAllocaFI(AI); @@ -3129,6 +3133,10 @@ void PPC32ISel::visitCastInst(CastInst &CI) { unsigned SrcClass = getClassB(Op->getType()); unsigned DestClass = getClassB(CI.getType()); + // Noop casts are not emitted: getReg will return the source operand as the + // register to use for any uses of the noop cast. + if (DestClass == SrcClass) return; + // If this is a cast from a 32-bit integer to a Long type, and the only uses // of the cast are GEP instructions, then the cast does not need to be // generated explicitly, it will be folded into the GEP. diff --git a/lib/Target/PowerPC/README.txt b/lib/Target/PowerPC/README.txt index 0e45959ea47..9c7e9e609b8 100644 --- a/lib/Target/PowerPC/README.txt +++ b/lib/Target/PowerPC/README.txt @@ -1,4 +1,6 @@ TODO: +* poor switch statement codegen +* load/store to alloca'd array or struct. * implement not-R0 register GPR class * implement scheduling info * implement do-loop pass