[PATCH] D74169: [WIP][LLD][ELF][DebugInfo] Skeleton implementation of removing obsolete debug info.

[Fangrui Song via Phabricator via llvm-commits]

MaskRay added a comment.

I agree with

> if the additional complexity is not too much, and the new code is nicely isolated from existing code. I think I agree with you that linker is perhaps the best place to drop dead DWARF info.

(One thing I plan to work on is .debug_names accelerator table, which is similar to but more power than `.gdb_index`.)

On one side, I've heard thoughts that: if we use -gsplit-dwarf, is .debug* garbage collection still valuable?
On the other side, I think this approach may be better than D54747 <https://reviews.llvm.org/D54747> (reverted by rL360955 <https://reviews.llvm.org/rL360955>) and may address the problem it tried to mitigate. Given the savings, we may bend our mind a bit and give more tolerance even if the additional complexity is more than `not too much`.

I am reluctant to let individual patches in before we can see the benefits, because they could easily turn out to be insufficient and cluttered the history.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D74169