[llvm] aa3c877 - Move variable only used in an assert into the assert itself.

Sterling Augustine via llvm-commits llvm-commits at lists.llvm.org
Fri Dec 6 17:10:08 PST 2019


Author: Sterling Augustine
Date: 2019-12-06T17:09:19-08:00
New Revision: aa3c877fb58db238a86e734511bdeed9bc40086c

URL: https://github.com/llvm/llvm-project/commit/aa3c877fb58db238a86e734511bdeed9bc40086c
DIFF: https://github.com/llvm/llvm-project/commit/aa3c877fb58db238a86e734511bdeed9bc40086c.diff

LOG: Move variable only used in an assert into the assert itself.

This prevents unused variable warnings from breaking the build.

Added: 
    

Modified: 
    llvm/lib/Target/AArch64/AArch64InstructionSelector.cpp

Removed: 
    


################################################################################
diff  --git a/llvm/lib/Target/AArch64/AArch64InstructionSelector.cpp b/llvm/lib/Target/AArch64/AArch64InstructionSelector.cpp
index d44dbf765985..29fa8c0f9632 100644
--- a/llvm/lib/Target/AArch64/AArch64InstructionSelector.cpp
+++ b/llvm/lib/Target/AArch64/AArch64InstructionSelector.cpp
@@ -1010,8 +1010,7 @@ bool AArch64InstructionSelector::selectCompareBranch(
 /// This needs to detect a splat-like operation, e.g. a G_BUILD_VECTOR.
 static Optional<int64_t> getVectorShiftImm(Register Reg,
                                            MachineRegisterInfo &MRI) {
-  const LLT Ty = MRI.getType(Reg);
-  assert(Ty.isVector() && "Expected a *vector* shift operand");
+  assert(MRI.getType(Reg).isVector() && "Expected a *vector* shift operand");
   MachineInstr *OpMI = MRI.getVRegDef(Reg);
   assert(OpMI && "Expected to find a vreg def for vector shift operand");
   if (OpMI->getOpcode() != TargetOpcode::G_BUILD_VECTOR)


        


More information about the llvm-commits mailing list