[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