<div dir="ltr"><div dir="ltr"><br></div><br><div class="gmail_quote"><div dir="ltr" class="gmail_attr">On Fri, Sep 17, 2021 at 6:10 AM Simon Pilgrim via llvm-commits <<a href="mailto:llvm-commits@lists.llvm.org">llvm-commits@lists.llvm.org</a>> wrote:<br></div><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex"><br>
Author: Simon Pilgrim<br>
Date: 2021-09-17T14:04:54+01:00<br>
New Revision: 72e57862818c92617e22595923cb0a2ed49d603c<br>
<br>
URL: <a href="https://github.com/llvm/llvm-project/commit/72e57862818c92617e22595923cb0a2ed49d603c" rel="noreferrer" target="_blank">https://github.com/llvm/llvm-project/commit/72e57862818c92617e22595923cb0a2ed49d603c</a><br>
DIFF: <a href="https://github.com/llvm/llvm-project/commit/72e57862818c92617e22595923cb0a2ed49d603c.diff" rel="noreferrer" target="_blank">https://github.com/llvm/llvm-project/commit/72e57862818c92617e22595923cb0a2ed49d603c.diff</a><br>
<br>
LOG: [DebugInfo] DWARF - Use const-ref iterator in for-range loop. NFCI.<br></blockquote><div><br>Thanks for the cleanup - though the commit message confused me until I checked the code: These aren't iterators that are being changed, they're the "value" (for want of any other better term) in a range-based for loop. (iterators, on the other hand, - generally should be handled by value - not by reference)<br> </div><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex">
<br>
Avoid unnecessary copies, reported by MSVC static analyzer.<br>
<br>
Added: <br>
<br>
<br>
Modified: <br>
    llvm/lib/DebugInfo/DWARF/DWARFAcceleratorTable.cpp<br>
    llvm/lib/DebugInfo/DWARF/DWARFVerifier.cpp<br>
<br>
Removed: <br>
<br>
<br>
<br>
################################################################################<br>
diff  --git a/llvm/lib/DebugInfo/DWARF/DWARFAcceleratorTable.cpp b/llvm/lib/DebugInfo/DWARF/DWARFAcceleratorTable.cpp<br>
index 28d35b609c24..c77d4d4d989c 100644<br>
--- a/llvm/lib/DebugInfo/DWARF/DWARFAcceleratorTable.cpp<br>
+++ b/llvm/lib/DebugInfo/DWARF/DWARFAcceleratorTable.cpp<br>
@@ -767,7 +767,7 @@ LLVM_DUMP_METHOD void DWARFDebugNames::NameIndex::dump(ScopedPrinter &W) const {<br>
   }<br>
<br>
   W.startLine() << "Hash table not present\n";<br>
-  for (NameTableEntry NTE : *this)<br>
+  for (const NameTableEntry &NTE : *this)<br>
     dumpName(W, NTE, None);<br>
 }<br>
<br>
@@ -799,7 +799,7 @@ DWARFDebugNames::ValueIterator::findEntryOffsetInCurrentIndex() {<br>
   const Header &Hdr = CurrentIndex->Hdr;<br>
   if (Hdr.BucketCount == 0) {<br>
     // No Hash Table, We need to search through all names in the Name Index.<br>
-    for (NameTableEntry NTE : *CurrentIndex) {<br>
+    for (const NameTableEntry &NTE : *CurrentIndex) {<br>
       if (NTE.getString() == Key)<br>
         return NTE.getEntryOffset();<br>
     }<br>
<br>
diff  --git a/llvm/lib/DebugInfo/DWARF/DWARFVerifier.cpp b/llvm/lib/DebugInfo/DWARF/DWARFVerifier.cpp<br>
index addf78ed4b2d..11301175768e 100644<br>
--- a/llvm/lib/DebugInfo/DWARF/DWARFVerifier.cpp<br>
+++ b/llvm/lib/DebugInfo/DWARF/DWARFVerifier.cpp<br>
@@ -430,7 +430,7 @@ unsigned DWARFVerifier::verifyDieRanges(const DWARFDie &Die,<br>
<br>
   if (!IsObjectFile || IsMachOObject || Die.getTag() != DW_TAG_compile_unit) {<br>
     bool DumpDieAfterError = false;<br>
-    for (auto Range : Ranges) {<br>
+    for (const auto &Range : Ranges) {<br>
       if (!Range.valid()) {<br>
         ++NumErrors;<br>
         error() << "Invalid address range " << Range << "\n";<br>
@@ -1515,7 +1515,7 @@ unsigned DWARFVerifier::verifyDebugNames(const DWARFSection &AccelSection,<br>
   if (NumErrors > 0)<br>
     return NumErrors;<br>
   for (const auto &NI : AccelTable)<br>
-    for (DWARFDebugNames::NameTableEntry NTE : NI)<br>
+    for (const DWARFDebugNames::NameTableEntry &NTE : NI)<br>
       NumErrors += verifyNameIndexEntries(NI, NTE);<br>
<br>
   if (NumErrors > 0)<br>
<br>
<br>
<br>
_______________________________________________<br>
llvm-commits mailing list<br>
<a href="mailto:llvm-commits@lists.llvm.org" target="_blank">llvm-commits@lists.llvm.org</a><br>
<a href="https://lists.llvm.org/cgi-bin/mailman/listinfo/llvm-commits" rel="noreferrer" target="_blank">https://lists.llvm.org/cgi-bin/mailman/listinfo/llvm-commits</a><br>
</blockquote></div></div>