Page MenuHomePhabricator

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

Authored by grimar on Fri, Jun 21, 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.Fri, Jun 21, 3:44 AM
jhenderson accepted this revision.Fri, Jun 21, 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.Fri, Jun 21, 6:41 AM
This revision was automatically updated to reflect the committed changes.
Herald added a project: Restricted Project. · View Herald TranscriptFri, Jun 21, 7:12 AM