[llvm] c546825 - [llvm] Fix unused variable warnings

[David Blaikie via llvm-commits]

If a variable is only used once in the assertion ,could you roll the
initialization into the assert (folding the named variable away)?

On Wed, Jun 3, 2020 at 2:49 AM Kadir Cetinkaya via llvm-commits
<llvm-commits at lists.llvm.org> wrote:
>
>
> Author: Kadir Cetinkaya
> Date: 2020-06-03T11:49:01+02:00
> New Revision: c5468253aa555c2df98bd1f49d1e9d44b0150a2e
>
> URL: https://github.com/llvm/llvm-project/commit/c5468253aa555c2df98bd1f49d1e9d44b0150a2e
> DIFF: https://github.com/llvm/llvm-project/commit/c5468253aa555c2df98bd1f49d1e9d44b0150a2e.diff
>
> LOG: [llvm] Fix unused variable warnings
>
> Added:
>
>
> Modified:
>     llvm/lib/CodeGen/AsmPrinter/DwarfDebug.cpp
>
> Removed:
>
>
>
> ################################################################################
> diff  --git a/llvm/lib/CodeGen/AsmPrinter/DwarfDebug.cpp b/llvm/lib/CodeGen/AsmPrinter/DwarfDebug.cpp
> index 67504f827d63..061330677680 100644
> --- a/llvm/lib/CodeGen/AsmPrinter/DwarfDebug.cpp
> +++ b/llvm/lib/CodeGen/AsmPrinter/DwarfDebug.cpp
> @@ -807,6 +807,7 @@ static void collectCallSiteParameters(const MachineInstr *CallMI,
>      auto Suc = std::next(CallMI->getIterator());
>      // Only one-instruction delay slot is supported.
>      auto BundleEnd = llvm::getBundleEnd(CallMI->getIterator());
> +    (void)BundleEnd;
>      assert(std::next(Suc) == BundleEnd &&
>             "More than one instruction in call delay slot");
>      // Try to interpret value loaded by instruction.
> @@ -856,7 +857,9 @@ void DwarfDebug::constructCallSiteEntryDIEs(const DISubprogram &SP,
>        return false;
>      auto Suc = std::next(MI.getIterator());
>      auto CallInstrBundle = getBundleStart(MI.getIterator());
> +    (void)CallInstrBundle;
>      auto DelaySlotBundle = getBundleStart(Suc);
> +    (void)DelaySlotBundle;
>      // Ensure that label after call is following delay slot instruction.
>      // Ex. CALL_INSTRUCTION {
>      //       DELAY_SLOT_INSTRUCTION }
> @@ -1893,6 +1896,7 @@ void DwarfDebug::beginInstruction(const MachineInstr *MI) {
>      if (!MI.isBundledWithSucc())
>        return false;
>      auto Suc = std::next(MI.getIterator());
> +    (void)Suc;
>      // Ensure that delay slot instruction is successor of the call instruction.
>      // Ex. CALL_INSTRUCTION {
>      //        DELAY_SLOT_INSTRUCTION }
>
>
>
> _______________________________________________
> llvm-commits mailing list
> llvm-commits at lists.llvm.org
> https://lists.llvm.org/cgi-bin/mailman/listinfo/llvm-commits