[llvm] Improve DWARFDebugLine::LineTable::lookupAddressRangeImpl(...) to act as exptected. (PR #226625)

Peter Rong via llvm-commits llvm-commits at lists.llvm.org
Sat Sep 26 10:17:48 PDT 2026


================
@@ -1446,6 +1453,25 @@ bool DWARFDebugLine::LineTable::lookupAddressRangeImpl(
 
     assert(FirstRowIndex != UnknownRowIndex);
     assert(LastRowIndex != UnknownRowIndex);
+    // The matching row range should end at the next row's address, unless the
+    // last row is the last row in the sequence. If we don't do this and we have
----------------
DataCorrupted wrote:

"Last" can mean both "previous" and "final", this took me a sec to realize what it means
```suggestion
    // LastRowIndex is at the end of the sequence. If we don't do this and we have
```

https://github.com/llvm/llvm-project/pull/226625


More information about the llvm-commits mailing list