[all-commits] [llvm/llvm-project] c0e1ad: [lldb] Skip declaration DIEs in the debug_names in...

Pavel Labath via All-commits all-commits at lists.llvm.org
Tue Jun 11 07:17:48 PDT 2024


  Branch: refs/heads/main
  Home:   https://github.com/llvm/llvm-project
  Commit: c0e1ad779f8b7b0073b89ecdd44c3b9c4a72e494
      https://github.com/llvm/llvm-project/commit/c0e1ad779f8b7b0073b89ecdd44c3b9c4a72e494
  Author: Pavel Labath <pavel at labath.sk>
  Date:   2024-06-11 (Tue, 11 Jun 2024)

  Changed paths:
    M lldb/source/Plugins/SymbolFile/DWARF/DebugNamesDWARFIndex.cpp
    A lldb/test/Shell/SymbolFile/DWARF/x86/debug-names-signature.s
    A lldb/test/Shell/SymbolFile/DWARF/x86/debug-names-static-constexpr-member.s

  Log Message:
  -----------
  [lldb] Skip declaration DIEs in the debug_names index (#94744)

This makes sure we try to process declaration DIEs that are erroneously
present in the index. Until bd5c6367bd7, clang was emitting index
entries for declaration DIEs with DW_AT_signature attributes. This makes
sure to avoid returning those DIEs as the definitions of a type, but
also makes sure to pass through DIEs referring to static constexpr
member variables, which is a (probably nonconforming) extension used by
dsymutil.

It adds test cases for both of the scenarios. It is essentially a
recommit of #91808.



To unsubscribe from these emails, change your notification settings at https://github.com/llvm/llvm-project/settings/notifications


More information about the All-commits mailing list