[PATCH] D55329: [llvm-readobj] Little clean up inside `parseDynamicTable`

[Xing via Phabricator via llvm-commits]

Higuoxing added a comment.

In D55329#1322718 <https://reviews.llvm.org/D55329#1322718>, @jhenderson wrote:

> LGTM for this change. The refactor is certainly good. I feel like I might have suggested this in a previous review a while back, but it obviously never happened (see D49016 <https://reviews.llvm.org/D49016>).
>
> As for whether or not we should error in this case, the problem is that things like the Hash table are looked up using this function, and then an attempt may be made to dump the hash table from the loaded address. Potentially, this could lead to trying to access invalid memory, which is obviously bad. So perhaps before making any decision, you should look up what the implications would be of removing the error?


Thanks for reviewing! I will look into it : )


Repository:
  rL LLVM

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

https://reviews.llvm.org/D55329