This is an archive of the discontinued LLVM Phabricator instance.

[llvm-objcopy] - Get rid of dynrel.elf precompiled binary from inputs.
ClosedPublic

Authored by grimar on Jun 21 2019, 3:44 AM.

Details

Summary

We do not have to spread using the precompiled binaries in the test,
when we can use YAML. This patch removes the dynrel.elf binary and adds
a few comments to the test cases.

Diff Detail

Repository
rL LLVM

Event Timeline

grimar created this revision.Jun 21 2019, 3:44 AM
jhenderson accepted this revision.Jun 21 2019, 6:41 AM

LGTM, with one comment fix.

test/tools/llvm-objcopy/ELF/prefix-alloc-sections-dynrelocsec.test
1 ↗(On Diff #205960)

for -> to a

This revision is now accepted and ready to land.Jun 21 2019, 6:41 AM
This revision was automatically updated to reflect the committed changes.
Herald added a project: Restricted Project. · View Herald TranscriptJun 21 2019, 7:12 AM