This is an archive of the discontinued LLVM Phabricator instance.

[llvm-objdump] Add -x --all-headers options
ClosedPublic

Authored by MaskRay on Jun 26 2018, 11:58 PM.

Diff Detail

Repository
rL LLVM

Event Timeline

MaskRay created this revision.Jun 26 2018, 11:58 PM
paulsemel accepted this revision.Jun 27 2018, 2:37 AM

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 :)

This revision is now accepted and ready to land.Jun 27 2018, 2:37 AM

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) :)

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 ! :)

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 ! :)

And FYI https://lists.freebsd.org/pipermail/freebsd-toolchain/2018-June/003809.html FreeBSD devs are moving away from GNU binutils and migrating to LLVM counterparts. They are beneficiaries of your work :)

echristo accepted this revision.Jun 27 2018, 1:48 PM

Oh, sure :)

This revision was automatically updated to reflect the committed changes.