This is an archive of the discontinued LLVM Phabricator instance.

[llvm-readobj] - Teach readobj to print PT_OPENBSD_BOOTDATA header
ClosedPublic

Authored by grimar on Dec 6 2016, 8:03 AM.

Diff Detail

Event Timeline

grimar updated this revision to Diff 80419.Dec 6 2016, 8:03 AM
grimar retitled this revision from to [llvm-readobj] - Teach readobj to print PT_OPENBSD_BOOTDATA header.
grimar updated this object.
grimar added reviewers: rafael, davide.
grimar added subscribers: llvm-commits, grimar, evgeny777.
davide added inline comments.Dec 6 2016, 8:42 AM
test/tools/llvm-readobj/program-headers.test
16

why did you have to change the linker script?

tools/llvm-readobj/ELFDumper.cpp
1161–1163

unsorted.

grimar added inline comments.Dec 6 2016, 8:44 AM
test/tools/llvm-readobj/program-headers.test
16

To add new program header 0x65a41be6 (PT_OPENBSD_BOOTDATA).

tools/llvm-readobj/ELFDumper.cpp
1161–1163

Sorted by value, linke in ELF.h and above.

davide added inline comments.Dec 6 2016, 8:46 AM
test/tools/llvm-readobj/program-headers.test
16

They look too magic (same for foo and bar). Please explain the mapping PT_OPENBSD_* -> number in a comment

grimar updated this revision to Diff 80435.Dec 6 2016, 9:01 AM
  • Addressed review comments.
davide accepted this revision.Dec 6 2016, 9:32 AM
davide edited edge metadata.

LGTM modulo nit.

test/tools/llvm-readobj/program-headers.test
17–19

Nit:
is *the* value of

This revision is now accepted and ready to land.Dec 6 2016, 9:32 AM
grimar added inline comments.Dec 6 2016, 9:36 AM
test/tools/llvm-readobj/program-headers.test
17–19

Thanks !

This revision was automatically updated to reflect the committed changes.