From f45c052c9e62a0482d80059ad1eddd8f36ed40ca Mon Sep 17 00:00:00 2001 From: Mikhail Goncharov Date: Wed, 18 Nov 2020 15:10:57 +0100 Subject: [PATCH] Fix unused variables in release build Differential Revision: https://reviews.llvm.org/D91705 --- llvm/lib/Target/ARM/ARMLowOverheadLoops.cpp | 8 +++----- 1 file changed, 3 insertions(+), 5 deletions(-) diff --git a/llvm/lib/Target/ARM/ARMLowOverheadLoops.cpp b/llvm/lib/Target/ARM/ARMLowOverheadLoops.cpp index 6ed6260c05e43..9f6f6d03aad50 100644 --- a/llvm/lib/Target/ARM/ARMLowOverheadLoops.cpp +++ b/llvm/lib/Target/ARM/ARMLowOverheadLoops.cpp @@ -1569,7 +1569,7 @@ void ARMLowOverheadLoops::ConvertVPTBlocks(LowOverheadLoop &LoLoop) { auto Next = ++MachineBasicBlock::iterator(VPST); assert(getVPTInstrPredicate(*Next) != ARMVCC::None && "The instruction after a VPST must be predicated"); - + (void)Next; MachineInstr *VprDef = RDA->getUniqueReachingMIDef(VPST, ARM::VPR); if (VprDef && VCMPOpcodeToVPT(VprDef->getOpcode()) && !LoLoop.ToRemove.contains(VprDef)) { @@ -1578,13 +1578,11 @@ void ARMLowOverheadLoops::ConvertVPTBlocks(LowOverheadLoop &LoLoop) { // the same values at the VPST if (RDA->hasSameReachingDef(VCMP, VPST, VCMP->getOperand(1).getReg()) && RDA->hasSameReachingDef(VCMP, VPST, VCMP->getOperand(2).getReg())) { - bool IntermediateInstrsUseVPR = - std::any_of(++MachineBasicBlock::iterator(VCMP), - MachineBasicBlock::iterator(VPST), hasVPRUse); // If the instruction after the VCMP is predicated then a different // code path is expected to have merged the VCMP and VPST already. // This assertion protects against changes to that behaviour - assert(!IntermediateInstrsUseVPR && + assert(!std::any_of(++MachineBasicBlock::iterator(VCMP), + MachineBasicBlock::iterator(VPST), hasVPRUse) && "Instructions between the VCMP and VPST are not expected to " "be predicated"); ReplaceVCMPWithVPT(VCMP, VPST);