[Lldb-commits] [lldb] [lldb-dap] Provide `declarationLocation` for variables (PR #102928)

Adrian Vogelsgesang via lldb-commits lldb-commits at lists.llvm.org
Wed Aug 21 19:14:56 PDT 2024


vogelsgesang wrote:

> @vogelsgesang , can you show a screenshot of the variables pane after this change? I want to make sure that simple types, like ints, are not displayed as having children.

**With this commit (and also with the changes from #104589)**

<img width="1644" alt="image" src="https://github.com/user-attachments/assets/862b301c-6321-47dc-9243-bb27447b093b">

Current `main`

<img width="1637" alt="image" src="https://github.com/user-attachments/assets/a6102469-a41a-4b0d-873a-ce8a1fee5b74">

As you can see, there are no differences re the absence / presence of the "expandable chevron" next to the variables.

That being said, I do find it a bit surprising that function pointers apparently return `true` for `MightHaveChildren`

https://github.com/llvm/llvm-project/pull/102928


More information about the lldb-commits mailing list