From e282c3c35e8cf48c1290d4b437269b2ffac98108 Mon Sep 17 00:00:00 2001 From: Justin Hibbits Date: Wed, 18 Jul 2018 05:19:25 +0000 Subject: Fix build failures from r337347, found by clang * Delete a no-longer-used override, and mark the other getRegisterTypeForCallingConv() as override. * SPE only supports i32, not i64, as the internal type, so simply remove the type check, so that DestReg and Opc are provably always set. GCC 6.4 did not warn about either of the above. git-svn-id: https://llvm.org/svn/llvm-project/llvm/trunk@337350 91177308-0d34-0410-b5e6-96231b3b80d8 --- lib/Target/PowerPC/PPCFastISel.cpp | 12 +++++------- lib/Target/PowerPC/PPCISelLowering.cpp | 6 ------ lib/Target/PowerPC/PPCISelLowering.h | 3 +-- 3 files changed, 6 insertions(+), 15 deletions(-) (limited to 'lib/Target/PowerPC') diff --git a/lib/Target/PowerPC/PPCFastISel.cpp b/lib/Target/PowerPC/PPCFastISel.cpp index 19d94137400..b00655b5022 100644 --- a/lib/Target/PowerPC/PPCFastISel.cpp +++ b/lib/Target/PowerPC/PPCFastISel.cpp @@ -1223,13 +1223,11 @@ bool PPCFastISel::SelectFPToI(const Instruction *I, bool IsSigned) { unsigned Opc; if (PPCSubTarget->hasSPE()) { - if (DstVT == MVT::i32) { - DestReg = createResultReg(&PPC::GPRCRegClass); - if (IsSigned) - Opc = InRC == &PPC::SPE4RCRegClass ? PPC::EFSCTSIZ : PPC::EFDCTSIZ; - else - Opc = InRC == &PPC::SPE4RCRegClass ? PPC::EFSCTUIZ : PPC::EFDCTUIZ; - } + DestReg = createResultReg(&PPC::GPRCRegClass); + if (IsSigned) + Opc = InRC == &PPC::SPE4RCRegClass ? PPC::EFSCTSIZ : PPC::EFDCTSIZ; + else + Opc = InRC == &PPC::SPE4RCRegClass ? PPC::EFSCTUIZ : PPC::EFDCTUIZ; } else { DestReg = createResultReg(&PPC::F8RCRegClass); if (DstVT == MVT::i32) diff --git a/lib/Target/PowerPC/PPCISelLowering.cpp b/lib/Target/PowerPC/PPCISelLowering.cpp index 48355494afe..1e3e14c7114 100644 --- a/lib/Target/PowerPC/PPCISelLowering.cpp +++ b/lib/Target/PowerPC/PPCISelLowering.cpp @@ -1237,12 +1237,6 @@ MVT PPCTargetLowering::getRegisterTypeForCallingConv(LLVMContext &Context, return PPCTargetLowering::getRegisterType(Context, VT); } -MVT PPCTargetLowering::getRegisterTypeForCallingConv(MVT VT) const { - if (Subtarget.hasSPE() && VT == MVT::f64) - return MVT::i32; - return PPCTargetLowering::getRegisterType(VT); -} - bool PPCTargetLowering::useSoftFloat() const { return Subtarget.useSoftFloat(); } diff --git a/lib/Target/PowerPC/PPCISelLowering.h b/lib/Target/PowerPC/PPCISelLowering.h index 11b974e42a0..9b8d6435515 100644 --- a/lib/Target/PowerPC/PPCISelLowering.h +++ b/lib/Target/PowerPC/PPCISelLowering.h @@ -874,9 +874,8 @@ namespace llvm { unsigned getNumRegistersForCallingConv(LLVMContext &Context, EVT VT) const override; - MVT getRegisterTypeForCallingConv(MVT VT) const; MVT getRegisterTypeForCallingConv(LLVMContext &Context, - EVT VT) const; + EVT VT) const override; private: struct ReuseLoadInfo { -- cgit v1.2.3