This is an archive of the discontinued LLVM Phabricator instance.

[ELF] - Tweak verneed.s test to support llvm-readobj functionality
ClosedPublic

Authored by grimar on Jun 7 2016, 6:53 AM.

Details

Reviewers
ruiu
rafael
Summary

Since llvm-readobj supports all versioning relative sections for dumping now,
I think this test should use that instead of binary data check.

Diff Detail

Event Timeline

grimar updated this revision to Diff 59881.Jun 7 2016, 6:53 AM
grimar retitled this revision from to [ELF] - Tweak verneed.s test to support llvm-readobj functionality.
grimar updated this object.
grimar added reviewers: ruiu, rafael.
grimar added subscribers: llvm-commits, grimar.
rafael accepted this revision.Jun 7 2016, 11:41 AM
rafael edited edge metadata.

LGTM with a nit.

test/ELF/verneed.s
5

You can delete the llvm-objdump run.

This revision is now accepted and ready to land.Jun 7 2016, 11:41 AM
grimar closed this revision.Jun 8 2016, 12:11 AM

Commited as r272120. Commit contains wrong review reference D21024 instead of D21073, sorry.