[all-commits] [llvm/llvm-project] 151320: [LLDB] Fix symbol breakpoint lookups for non-C-lik...

[Walter Erquinigo via All-commits]

  Branch: refs/heads/users/walter-erquinigo/fix-breakpoints
  Home:   https://github.com/llvm/llvm-project
  Commit: 15132008232d446036a43a8ed3eed5883f80a2d5
      https://github.com/llvm/llvm-project/commit/15132008232d446036a43a8ed3eed5883f80a2d5
  Author: walter erquinigo <walter at modular.com>
  Date:   2024-09-30 (Mon, 30 Sep 2024)

  Changed paths:
    M lldb/include/lldb/Target/Language.h
    M lldb/source/Core/Module.cpp
    M lldb/source/Plugins/Language/CPlusPlus/CPlusPlusLanguage.cpp
    M lldb/source/Plugins/Language/CPlusPlus/CPlusPlusLanguage.h
    M lldb/source/Plugins/SymbolFile/DWARF/DWARFIndex.cpp

  Log Message:
  -----------
  [LLDB] Fix symbol breakpoint lookups for non-C-like languages

I'm developing the Mojo language, and some symbol breakpoints don't work because of two reasons that got fixed1:

- There's a prune step that operates on the assuption that the symbol is C-like. That was discarding lots of the Mojo breakpoints
- When finding functions for symbol breakpoints with eFunctionNameTypeFull, LLDB was only matching die.getMangledName() and not die.GetName(). The latter is the one useful for Mojo, because the former has some unreadable format in Mojo that is not exposed to the user. This shouldn't cause any regression for C-like languages, as the prune step would sanitize them anyway, but it's useful for languages like Mojo.



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