Skip to content

Conversation

JDevlieghere
Copy link
Member

Add LLDB data formatters to the document about debugging LLVM.

Add LLDB data formatters to the document about debugging LLVM.
Copy link
Contributor

@fmayer fmayer left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Please fix the build error

provided for some of the core LLVM libraries. To use them, execute the
following (or add it to your ``~/.lldbinit``)::

command script import /path/to/lldbDataFormatters.py
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Are they part of the llvm-project? In which case I would put the correct relative path

following (or add it to your ``~/.lldbinit``)::

command script import /path/to/lldbDataFormatters.py
command script import /path/to/llvm/src/utils/lldbDataFormatters.py
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I would just make it utils/... but either way is ok

@JDevlieghere
Copy link
Member Author

I updated both the LLDB and GDB path to be relative to the monorepo root. This hopefully avoid ambiguity and is what I've seen used elsewhere.

@pcc
Copy link
Contributor

pcc commented Sep 8, 2025

It seems better to use /path/to/llvm instead of llvm. I guess the relative path would only work if you happen to start the debugger from the monorepo root.

@fmayer
Copy link
Contributor

fmayer commented Sep 8, 2025

It seems better to use /path/to/llvm instead of llvm. I guess the relative path would only work if you happen to start the debugger from the monorepo root.

/path/to/llvm verbatim works nowhere, so that's strictly better.. and at least I would immediately interpret it as relative to the root

@JDevlieghere JDevlieghere merged commit 073cf98 into llvm:main Sep 8, 2025
10 checks passed
@JDevlieghere JDevlieghere deleted the lldb-data-formatters branch September 8, 2025 23:30
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants