[PATCH] D92923: [llvm-readelf/obj] - Add support of multiple SHT_SYMTAB_SHNDX sections.

George Rimar via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Fri Dec 18 05:28:49 PST 2020


grimar updated this revision to Diff 312767.
grimar retitled this revision from "[llvm-readelf/obj][WIP] - Add support for multiple SHT_SYMTAB_SHNDX sections." to "[llvm-readelf/obj] - Add support of multiple SHT_SYMTAB_SHNDX sections.".
grimar edited the summary of this revision.
grimar added a comment.

- Added test cases.
- Implemented helper `DataRegion` struct as discussed.

Note: if `struct DataRegion` is fine, then I'll add unit tests for it.
I believe It is impossible to trigger one of errors it might report
from the llvm-readelf/obj code.


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

https://reviews.llvm.org/D92923

Files:
  llvm/include/llvm/Object/ELF.h
  llvm/test/Object/invalid.test
  llvm/test/tools/llvm-readobj/ELF/dyn-symbols.test
  llvm/test/tools/llvm-readobj/ELF/dynamic-tags.test
  llvm/test/tools/llvm-readobj/ELF/mips-got.test
  llvm/test/tools/llvm-readobj/ELF/mips-plt.test
  llvm/test/tools/llvm-readobj/ELF/section-symbols.test
  llvm/test/tools/llvm-readobj/ELF/symbol-shndx.test
  llvm/test/tools/llvm-readobj/ELF/symtab-shndx.test
  llvm/test/tools/obj2yaml/ELF/sht-symtab-shndx.yaml
  llvm/test/tools/yaml2obj/ELF/sht-symtab-shndx.yaml
  llvm/tools/llvm-readobj/ELFDumper.cpp

-------------- next part --------------
A non-text attachment was scrubbed...
Name: D92923.312767.patch
Type: text/x-patch
Size: 56451 bytes
Desc: not available
URL: <http://lists.llvm.org/pipermail/llvm-commits/attachments/20201218/26038a6b/attachment-0001.bin>


More information about the llvm-commits mailing list