[all-commits] [llvm/llvm-project] 6e3ecd: [lldb] Fix dwo variant of TestLibCxxFunction

Pavel Labath via All-commits all-commits at lists.llvm.org
Thu Nov 14 07:30:08 PST 2019


  Branch: refs/heads/master
  Home:   https://github.com/llvm/llvm-project
  Commit: 6e3ecd18847cb5c5bbe41d23428e1aa57ed1b339
      https://github.com/llvm/llvm-project/commit/6e3ecd18847cb5c5bbe41d23428e1aa57ed1b339
  Author: Pavel Labath <pavel at labath.sk>
  Date:   2019-11-14 (Thu, 14 Nov 2019)

  Changed paths:
    M lldb/source/Plugins/SymbolFile/DWARF/SymbolFileDWARF.cpp

  Log Message:
  -----------
  [lldb] Fix dwo variant of TestLibCxxFunction

The test was failing due to a bug in SymbolFileDWARF::FindFunctions --
the function was searching the main dwarf unit for DW_TAG_subprograms,
but the main unit is empty in case of split dwarf.  The fix is simple --
search the non-skeleton unit instead.

This bug went unnoticed because this function is expensive, and so one
generally avoids calling it.




More information about the All-commits mailing list