[all-commits] [llvm/llvm-project] 3973d8: [lldb] Return all line entries matchign a line if ...

Kim-Anh via All-commits all-commits at lists.llvm.org
Mon Aug 30 05:48:13 PDT 2021


  Branch: refs/heads/main
  Home:   https://github.com/llvm/llvm-project
  Commit: 3973d8b29e249a6ea4aceaa4225dfde5663937c6
      https://github.com/llvm/llvm-project/commit/3973d8b29e249a6ea4aceaa4225dfde5663937c6
  Author: Kim-Anh Tran <kimanh at chromium.org>
  Date:   2021-08-30 (Mon, 30 Aug 2021)

  Changed paths:
    M lldb/source/Symbol/CompileUnit.cpp
    M lldb/unittests/Symbol/TestLineEntry.cpp

  Log Message:
  -----------
  [lldb] Return all line entries matchign a line if no column is specified

Previously, if no column was specified, ResolveSymbolContext would take
the first match returned by FindLineEntryIndexByFileIndex, and reuse it
to find subsequent exact matches. With the introduction of columns, columns
are now considered when matching the line entries.

This leads to a problem if one wants to get all existing line entries
that match that line, since now the column is also used for the exact match.
This way, all line entries are filtered out that have a different
column number, but the same line number.

This patch changes that by ignoring the column information of the first match
if the original request of ResolveSymbolContext was also ignoring it.

Reviewed By: mib

Differential Revision: https://reviews.llvm.org/D108816




More information about the All-commits mailing list