This is an archive of the discontinued LLVM Phabricator instance.

[llvm-objcopy] Support the rest of the ELF formats
ClosedPublic

Authored by jakehehrlich on Nov 13 2017, 1:27 PM.
Tokens
"Like" token, awarded by xiangzhai.

Details

Summary

We haven't been supporting anything but ELF64LE since the start. Luckily this was always accounted for and the change is pretty trivial. B35281 requests this change for ELF32LE. This change adds support for ELF32LE, ELF64BE, and ELF32BE with all supported features that already existed for ELF64LE.

Diff Detail

Repository
rL LLVM

Event Timeline

jakehehrlich created this revision.Nov 13 2017, 1:27 PM
davide added a subscriber: davide.Nov 13 2017, 6:12 PM
davide added inline comments.
tools/llvm-objcopy/llvm-objcopy.cpp
253

auto *

Replaced explicit types with auto as recommended. Thanks Davide!

jhenderson accepted this revision.Nov 14 2017, 2:52 AM

LGTM.

I don't see any sensible way of doing this, but it would be really nice if every single one of our test cases could be run for all 4 forms, as I believe the same behaviour should be present in all of them. The test coverage for this change though is good enough though. Thanks.

This revision is now accepted and ready to land.Nov 14 2017, 2:52 AM
This revision was automatically updated to reflect the committed changes.