mib added inline comments.
================ Comment at: lldb/source/API/SBCompileUnit.cpp:94 + + if (!exact || + (exact && !LineEntry::Compare(line_entry.ref(), found_line_entry))) ---------------- jingham wrote: > Why do you have to do this Compare? You already passed exact to > FindLineEntry, does FindLineEntry really return line entries that fail your > Compare test when exact is passed in as true? That doesn't seem right. `exact` could be false, in which case the line entry index returned by `FindLineEntry` will be different from the one provided by the user. But if the user really wants the exact line entry index, we need to compare the returned line entry and the one provided by the user. Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D125437/new/ https://reviews.llvm.org/D125437 _______________________________________________ lldb-commits mailing list lldb-commits@lists.llvm.org https://lists.llvm.org/cgi-bin/mailman/listinfo/lldb-commits
