[PATCH] D74169: [WIP][LLD][ELF][DebugInfo] Remove obsolete debug info.

[David Blaikie via Phabricator via llvm-commits]

dblaikie added a comment.

In D74169#1972348 <https://reviews.llvm.org/D74169#1972348>, @avl wrote:

> @jhenderson
>
> > One drive-by request/question: have you done any measurements of the impact on debuggers? In other words, does it improve/make worse the time it takes them to load the debug data?
>
> loading times for debuggers:
>
> no gc-debuginfo case:
>  gdb 8.1 clang11 1,4G: loading time 56 sec used memory 2166872kb
>  lldb 8.0 clang11 1,4G: loading time 7.5 sec used memory 1508860kb
>
> gc-debuginfo case:
>  gdb 8.1 clang11 783M: loading time 51 sec used memory 1741552kb
>  lldb 8.1 clang11 783M: loading time 4 sec used memory 842980kb
>
> Overall result: gc-debuginfo makes loading time shorter and reduces memory usage for debuggers,


Might be worth comparing with -gdb-index too (though not sure if gdb-index will be immediately compatible with the DWARF aware linking, so a difficult comparison - but even comparing "gdb-index without DWARF-aware linking V DWARF-aware linking without gdb-index", etc)


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

https://reviews.llvm.org/D74169