From c44d41c086d1affdea849a7d49ae5e32cdecfe75 Mon Sep 17 00:00:00 2001 From: Steven Wu Date: Thu, 10 Jul 2025 14:39:35 -0700 Subject: [PATCH] [NFC] Fix unused variable warning in MCCAS with no assert Fix some unused variable warning in MCCAS implementation when assertion is off. --- llvm/lib/MCCAS/MCCASObjectV1.cpp | 13 ++++++------- 1 file changed, 6 insertions(+), 7 deletions(-) diff --git a/llvm/lib/MCCAS/MCCASObjectV1.cpp b/llvm/lib/MCCAS/MCCASObjectV1.cpp index 790b503699b0c..5178882c104d3 100644 --- a/llvm/lib/MCCAS/MCCASObjectV1.cpp +++ b/llvm/lib/MCCAS/MCCASObjectV1.cpp @@ -1782,12 +1782,12 @@ MCSymbolIdFragmentRef::materialize(MCCASReader &Reader, if (!B) \ return B.takeError(); \ MB.Asm.writeFragmentPadding(MB.FragmentOS, F, FragmentSize); \ - ArrayRef Contents = F.getContents(); \ B->Data.append(MB.FragmentData); \ B->Data.append(FragmentContents.begin(), FragmentContents.end()); \ - assert(((MB.FragmentData.empty() && Contents.empty()) || \ - (MB.FragmentData.size() + Contents.size() == FragmentSize)) && \ - "Size should match"); \ + assert( \ + ((MB.FragmentData.empty() && F.getContents().empty()) || \ + (MB.FragmentData.size() + F.getContents().size() == FragmentSize)) && \ + "Size should match"); \ return get(B->build()); \ } \ Expected MCFragmentName##Ref::materialize( \ @@ -4123,11 +4123,10 @@ Error DIEVisitor::visitDIERef(DIEDedupeTopLevelRef StartDIERef) { readAbbrevIdx(DistinctExtractor, DistinctCursor); if (!MaybeAbbrevIdx) return MaybeAbbrevIdx.takeError(); - auto AbbrevIdx = *MaybeAbbrevIdx; // The tag of a fresh block must be meaningful, otherwise we wouldn't have // made a new block. - assert(AbbrevIdx != getEndOfDIESiblingsMarker() && - AbbrevIdx != getDIEInAnotherBlockMarker()); + assert(*MaybeAbbrevIdx != getEndOfDIESiblingsMarker() && + *MaybeAbbrevIdx != getDIEInAnotherBlockMarker()); DistinctCursor.seek(Offset);