From 8983896a9f250d6e9629de95c256607241897099 Mon Sep 17 00:00:00 2001 From: Matt Arsenault Date: Fri, 17 May 2019 12:59:27 +0000 Subject: [PATCH] AMDGPU: Fix unused variable warnings in release builds llvm-svn: 361030 --- lib/Target/AMDGPU/AMDGPULegalizerInfo.cpp | 21 +++++++++------------ 1 file changed, 9 insertions(+), 12 deletions(-) diff --git a/lib/Target/AMDGPU/AMDGPULegalizerInfo.cpp b/lib/Target/AMDGPU/AMDGPULegalizerInfo.cpp index 4b45b7015ba..6ee40cb2de0 100644 --- a/lib/Target/AMDGPU/AMDGPULegalizerInfo.cpp +++ b/lib/Target/AMDGPU/AMDGPULegalizerInfo.cpp @@ -882,20 +882,18 @@ bool AMDGPULegalizerInfo::legalizeFceil( MachineIRBuilder &B) const { B.setInstr(MI); + const LLT S1 = LLT::scalar(1); + const LLT S64 = LLT::scalar(64); + unsigned Src = MI.getOperand(1).getReg(); - LLT Ty = MRI.getType(Src); - assert(Ty.isScalar() && Ty.getSizeInBits() == 64); + assert(MRI.getType(Src) == S64); // result = trunc(src) // if (src > 0.0 && src != result) // result += 1.0 - LLT S1 = LLT::scalar(1); - LLT S64 = LLT::scalar(64); - auto Trunc = B.buildInstr(TargetOpcode::G_INTRINSIC_TRUNC, {S64}, {Src}); - const auto Zero = B.buildFConstant(S64, 0.0); const auto One = B.buildFConstant(S64, 1.0); auto Lt0 = B.buildFCmp(CmpInst::FCMP_OGT, S1, Src, Zero); @@ -929,13 +927,12 @@ bool AMDGPULegalizerInfo::legalizeIntrinsicTrunc( MachineIRBuilder &B) const { B.setInstr(MI); - unsigned Src = MI.getOperand(1).getReg(); - LLT Ty = MRI.getType(Src); - assert(Ty.isScalar() && Ty.getSizeInBits() == 64); + const LLT S1 = LLT::scalar(1); + const LLT S32 = LLT::scalar(32); + const LLT S64 = LLT::scalar(64); - LLT S1 = LLT::scalar(1); - LLT S32 = LLT::scalar(32); - LLT S64 = LLT::scalar(64); + unsigned Src = MI.getOperand(1).getReg(); + assert(MRI.getType(Src) == S64); // TODO: Should this use extract since the low half is unused? auto Unmerge = B.buildUnmerge({S32, S32}, Src);