[PATCH] D48622: [llvm-objdump] Add -x --all-headers options

[Paul Semel via Phabricator via llvm-commits]

paulsemel added a comment.

In https://reviews.llvm.org/D48622#1145147, @MaskRay wrote:

> In https://reviews.llvm.org/D48622#1144670, @paulsemel wrote:
>
> > It seems to be displaying the correct information.
> >  I just noticed that we are not printing dynamic section in the `-private-headers` option, this might somehow be fixed.
> >
> > LGTM, but please, wait for @echristo  approval :)
>
>
> We also lack `-a` (`--archive-headers`) and `-f` (`--file-headers`) :)


That's right, I will take a look at it when I have more time, thanks a lot ! :)


Repository:
  rL LLVM

https://reviews.llvm.org/D48622