]> granicus.if.org Git - llvm/commitdiff
AMDGPU: Fix unused variable warnings in release builds
authorMatt Arsenault <Matthew.Arsenault@amd.com>
Fri, 17 May 2019 12:59:27 +0000 (12:59 +0000)
committerMatt Arsenault <Matthew.Arsenault@amd.com>
Fri, 17 May 2019 12:59:27 +0000 (12:59 +0000)
git-svn-id: https://llvm.org/svn/llvm-project/llvm/trunk@361030 91177308-0d34-0410-b5e6-96231b3b80d8

lib/Target/AMDGPU/AMDGPULegalizerInfo.cpp

index 4b45b7015ba809290f87b2347cecda589c47971f..6ee40cb2de0df56d094ff96aa0fc85d4a6381ea0 100644 (file)
@@ -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);