[PATCH] D79061: [ELF] --gdb-index: support .debug_loclists

Fangrui Song via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Wed Apr 29 15:06:39 PDT 2020


MaskRay added a comment.

In D79061#2011271 <https://reviews.llvm.org/D79061#2011271>, @dblaikie wrote:

> Fair enough - seems like a reasonable fix/consistent with the existing code (are there any other missing sections?) though might be nice to revisit the API at some point. I don't think there should be any reading/need for loc/loclists in LLD's use of DWARF (except the full DWARF-aware linking stuff being prototyped)


Thanks! --gdb-index does not need .debug_loclists . Only the full DWARF aware linking may need it.

I'll update the comment in `lld/test/ELF/gdb-index-loclists.s` to mention that this is just a regression test.

> are there any other missing sections?

Hopefully no :)


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D79061





More information about the llvm-commits mailing list