[Lldb-commits] [lldb] Reapply [lldb][DWARF] Delay struct/class/union definition DIE searching when parsing declaration DIEs. (PR #92328)

Michael Buch via lldb-commits lldb-commits at lists.llvm.org
Tue Jun 4 09:20:35 PDT 2024


Michael137 wrote:

> > Should we allow such entries in the index? And does this warrant rephrasing in the DWARF spec?
> 
> It's fine to have those as _nameless_ entries (which we don't emit today), but the entries that are reaching "process entry" have a name by definition. That if statement is a workaround for incorrect dwarf generation. It seems like this test is providing a case where incorrect dwarf is generated?

Yea i misremembered, dsymutil is the one indexing these inline statics. @ZequanWu could you put up a PR for the proposed patch?

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


More information about the lldb-commits mailing list