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

[George Rimar via Phabricator via llvm-commits]

grimar planned changes to this revision.
grimar added a comment.

In D92923#2454328 <https://reviews.llvm.org/D92923#2454328>, @jhenderson wrote:

> Sorry, this somehow slipped off my radar, and I've run out of time to do any further significant reviewing today. I've skimmed it briefly, and the general approach looks fine. I'd like ideally to find a way to avoid needing to pass both a pointer and size around for the shndx table, even if it's just to wrap it in a new class, or change to `Optional<ArrayRef<...>>` or something.

Thanks for looking! I am planning to add missing test cases and will update the patch. We can discuss the possible change of signatures after that.
Initially the code had `ArrayRef<Elf_Word> ShndxTable`. In theory we can use it, but then we will have an empty (size == 0) `ArrayRef` with a non-zero data.
I am just not sure that doing it looks nice.


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

https://reviews.llvm.org/D92923