[PATCH] D142556: [DebugInfo] Merge partially matching chains of inlined locations

Adrian Prantl via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Fri Jan 27 12:52:03 PST 2023


aprantl added a comment.

In D142556#4080184 <https://reviews.llvm.org/D142556#4080184>, @dstenb wrote:

> As far as I can tell it is valid DWARF to inline at line 0.
>
> I guess that one of the main questions here is whether debuggers accept this. Of what I have tested of lldb trunk and GDB (9 and 11) they seem to handle it, with the caveat regarding the LLDB patch D142552 <https://reviews.llvm.org/D142552> and that GDB seems to ignore line 0 locations in some cases. I do not know how the Sony debugger, dbx, or any other debugger handles it.

To speak for LLDB here: I support emitting an inlined-at: line 0. If LLDB can't deal with it (though I don't see why) we can fix LLDB.


CHANGES SINCE LAST ACTION
  https://reviews.llvm.org/D142556/new/

https://reviews.llvm.org/D142556



More information about the llvm-commits mailing list