This is an archive of the discontinued LLVM Phabricator instance.

[llvm-readobj][ELF] New `llvm-readobj` command line option `-mips-plt-got` to print a MIPS .got section
ClosedPublic

Authored by atanasyan on Jun 4 2014, 12:26 AM.

Details

Summary

This patch adds to llvm-readobj a new command line option -mips-plt-got for printing a MIPS .got section. This option exists in the readelf tool but missed in the llvm-readobj.

Diff Detail

Event Timeline

atanasyan updated this revision to Diff 10079.Jun 4 2014, 12:26 AM
atanasyan retitled this revision from to [llvm-readobj][ELF] New `llvm-readobj` command line option `-mips-plt-got` to print a MIPS .got section.
atanasyan updated this object.
atanasyan edited the test plan for this revision. (Show Details)
atanasyan added a reviewer: Bigcheese.
atanasyan added a subscriber: Unknown Object (MLST).
rafael added inline comments.
tools/llvm-readobj/ELFDumper.cpp
83

Can this be static?

690

Can this be static.

700–701

This is a refactoring, no? Can you split it?

702–703

This part too.

atanasyan updated this revision to Diff 10262.Jun 9 2014, 11:31 PM
  1. Refactored some code by the separate commit rL210509.
  2. The findSectionByAddress() is a static function now.
rafael accepted this revision.Jun 17 2014, 9:30 AM
rafael added a reviewer: rafael.

LGTM in that this is now fully MIPS specific.

Is there any documentation about this format that you can put a link to in the comments.

tools/llvm-readobj/ELFDumper.cpp
1056

Style, start with a lowercase.

This revision is now accepted and ready to land.Jun 17 2014, 9:30 AM
atanasyan closed this revision.Jun 18 2014, 2:01 AM

Thanks for review.

Closed by commit rL211150.