[Lldb-commits] [lldb] [WIP] [lldb][Progress] Report progress when completing types from DWARF (PR #91452)

Jonas Devlieghere via lldb-commits lldb-commits at lists.llvm.org
Thu May 9 09:45:36 PDT 2024


JDevlieghere wrote:

> If this turns out to be necessary, one way to rate-limit them would be to have a nesting depth counter. Assuming that these imports happen recursively we could create only Progress objects for the top n layers.

Is the code that emits the progress event recursive too? The reason I ask is because on the command line, nested progress events will get shadowed. The same is true for coalesced progress events. I'm not sure how VSCode/DAP clients deal with this, so maybe they're shown there?

Anyway, if the code is recursive, we might need to do something like we did for Swift, with one top-level event and callback that updates the details. 

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


More information about the lldb-commits mailing list