[llvm] Added --debugger-view option to llvm-debuginfo-analyzer (PR #159853)
Carlos Alberto Enciso via llvm-commits
llvm-commits at lists.llvm.org
Thu Sep 25 03:02:03 PDT 2025
CarlosAlbertoEnciso wrote:
@adam-yang, The concept of the `debugger view` is very interesting. Thanks for doing this.
Without looking in detail at the specific details, these are some points that should be changed to fit with the tool/library framework:
- Instead of introducing new descriptors for the logical line and symbol
```
LINE: [0x0000000050] example.hlsl:16 [main]
VAR: dtid: uint3 : regx VGPR0, piece 4 (line 15)
```
we should keep the existing ones: {Line} and {Variable}
```
{Line}: [0x0000000050] example.hlsl:16 [main]
{Variable}: dtid: uint3 : regx VGPR0, piece 4 (line 15)
```
- The `debugger view` should be available via the `LogicalView library` library and not via the `llvm-debuginfo-analyzer` tool.
Basically the proposed changes should be moved to `llvm/lib/DebugInfo/LogicalView/Core`. In that way any consumer of that library will have access to the view.
- The new options should be treated in the same way (location and checks) as the existing options.
Be in `llvm/tools/llvm-debuginfo-analyzer/Options.cpp` and updated in `LVOptions::resolveDependencies`
- May be add the new options to an existing category: `--report` or `--output`.
Control the element included via the `--print=xxx`
```
--debugger-view --> --report=debugger` or `--output=debugger
--debugger-view-vars --> --print=symbols
--debugger-view-code --> --print=instructions
```
- The general documentation `llvm/docs/CommandGuide/llvm-debuginfo-analyzer.rst` needs to be updated.
https://github.com/llvm/llvm-project/pull/159853
More information about the llvm-commits
mailing list