This is an archive of the discontinued LLVM Phabricator instance.

[llvm-readobj] Dump DT_USED value as string like GNU readelf does
ClosedPublic

Authored by Higuoxing on Mar 7 2019, 3:53 AM.

Diff Detail

Repository
rL LLVM

Event Timeline

Higuoxing created this revision.Mar 7 2019, 3:53 AM
Herald added a project: Restricted Project. · View Herald TranscriptMar 7 2019, 3:53 AM
jhenderson added inline comments.Mar 7 2019, 4:37 AM
test/tools/llvm-readobj/elf-dynamic-tags.test
264–265 ↗(On Diff #189689)

Could you put this above DT_FILTER, please, since it is before it in the enum range.

tools/llvm-readobj/ELFDumper.cpp
1882–1884 ↗(On Diff #189689)

Put this before DT_FILTER, similar to my above comment.

To confirm, GNU readelf prints it as "Not needed object"?

Higuoxing updated this revision to Diff 189692.Mar 7 2019, 4:53 AM

Addressed @jhenderson 's comments

  • Put DT_USED before DT_FILTER. (Sorry, I should have noticed this)
  • Yes, my GNU readelf prints Not needed object: [U]
This revision is now accepted and ready to land.Mar 7 2019, 5:33 AM
This revision was automatically updated to reflect the committed changes.