[Lldb-commits] [PATCH] D67390: [LLDB][ELF] Load both, .symtab and .dynsym sections

[Konrad Wilhelm Kleine via Phabricator via lldb-commits]

kwk marked an inline comment as done.
kwk added a comment.

@JDevlieghere I change the support tool. It was @labath who requested (D66791#inline-601050 <https://reviews.llvm.org/D66791#inline-601050>) to outsource this patch. In that change you can find the whole reason which is somewhat more longish:

> Let's put this bit into a separate patch. As I said over IRC, I view this bit as functionally independent of the debugdata stuff (it's definitely independent in it's current form, as it will kick in for non-debugdata files too, which I think is fine).

But apart from that I think the comment indeed needs to change. Anyway, please let me know if the connection to the other change D66791 <https://reviews.llvm.org/D66791> does make sense to you.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D67390