This is an archive of the discontinued LLVM Phabricator instance.

[llvm-readelf/obj][test] - Stop using precompiled binary in mips-plt.test
ClosedPublic

Authored by grimar on Oct 9 2020, 12:18 AM.

Details

Summary

This removes the precompiled binary and rewrites test to use YAML.

After this change we'll have no more precompiled inputs in llvm-readobj/ELF/Inputs.

Diff Detail

Event Timeline

grimar created this revision.Oct 9 2020, 12:18 AM
grimar requested review of this revision.Oct 9 2020, 12:18 AM

Not really knowing MIPS, it's probably best if @atanasyan reviews this.

llvm/test/tools/llvm-readobj/ELF/mips-plt.test
4–5

What's the point of the 2>&1? There's no check that there are [no] warnings in the readobj case at least.

grimar marked an inline comment as done.Oct 9 2020, 12:37 AM
grimar added inline comments.
llvm/test/tools/llvm-readobj/ELF/mips-plt.test
4–5

You're right. Placed it by mistake, I'll remove.

grimar updated this revision to Diff 297137.Oct 9 2020, 12:56 AM
grimar marked an inline comment as done.
  • Addressed review comment.
This revision is now accepted and ready to land.Oct 9 2020, 1:37 AM