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

[Konrad Wilhelm Kleine via Phabricator via llvm-commits]

kwk updated this revision to Diff 221689.
kwk added a comment.

- [LLDB][ELF] Fixup for comments in D67390 <https://reviews.llvm.org/D67390>
- Change test expectation to find 2 instead of 1 symbol
- symbol uniqueness by using elf::ELFSymbol


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D67390

Files:
  lldb/lit/Modules/ELF/Inputs/load-from-dynsym-alone.c
  lldb/lit/Modules/ELF/Inputs/load-symtab-and-dynsym.c
  lldb/lit/Modules/ELF/load-from-dynsym-alone.test
  lldb/lit/Modules/ELF/load-symtab-and-dynsym.test
  lldb/lit/helper/toolchain.py
  lldb/packages/Python/lldbsuite/test/functionalities/load_unload/TestLoadUnload.py
  lldb/source/Plugins/ObjectFile/ELF/ELFHeader.h
  lldb/source/Plugins/ObjectFile/ELF/ObjectFileELF.cpp
  lldb/source/Plugins/ObjectFile/ELF/ObjectFileELF.h
  lldb/unittests/Target/ModuleCacheTest.cpp

-------------- next part --------------
A non-text attachment was scrubbed...
Name: D67390.221689.patch
Type: text/x-patch
Size: 12281 bytes
Desc: not available
URL: <http://lists.llvm.org/pipermail/llvm-commits/attachments/20190925/4330bd69/attachment.bin>