[llvm] r361030 - AMDGPU: Fix unused variable warnings in release builds

[Matt Arsenault via llvm-commits]

Author: arsenm
Date: Fri May 17 05:59:27 2019
New Revision: 361030

URL: http://llvm.org/viewvc/llvm-project?rev=361030&view=rev
Log:
AMDGPU: Fix unused variable warnings in release builds

Modified:
    llvm/trunk/lib/Target/AMDGPU/AMDGPULegalizerInfo.cpp

Modified: llvm/trunk/lib/Target/AMDGPU/AMDGPULegalizerInfo.cpp
URL: http://llvm.org/viewvc/llvm-project/llvm/trunk/lib/Target/AMDGPU/AMDGPULegalizerInfo.cpp?rev=361030&r1=361029&r2=361030&view=diff
==============================================================================
--- llvm/trunk/lib/Target/AMDGPU/AMDGPULegalizerInfo.cpp (original)
+++ llvm/trunk/lib/Target/AMDGPU/AMDGPULegalizerInfo.cpp Fri May 17 05:59:27 2019
@@ -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::legalizeIntrin
   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);