[PATCH] D50343: [llvm-objcopy] Add support for -I binary -B <arch>.

Jordan Rupprecht via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Wed Aug 8 13:06:18 PDT 2018


rupprecht updated this revision to Diff 159772.
rupprecht added a comment.

- Pass MachineInfo into BinaryReader to push ELFType template down to only BinaryELFBuilder<>
- Move getElfType out of Reader (and also ELFReader), it can be implemented purely in the driver class. This make it clear that "Reader" is not tied to ELF types, even if that's all that's supported currently.


Repository:
  rL LLVM

https://reviews.llvm.org/D50343

Files:
  test/tools/llvm-objcopy/binary-input-aarch64.test
  test/tools/llvm-objcopy/binary-input-arm.test
  test/tools/llvm-objcopy/binary-input-error.test
  test/tools/llvm-objcopy/binary-input-i386.test
  test/tools/llvm-objcopy/binary-input-reconstitute.test
  test/tools/llvm-objcopy/binary-input-x86-64.test
  test/tools/llvm-objcopy/binary-input.test
  tools/llvm-objcopy/Object.cpp
  tools/llvm-objcopy/Object.h
  tools/llvm-objcopy/llvm-objcopy.cpp

-------------- next part --------------
A non-text attachment was scrubbed...
Name: D50343.159772.patch
Type: text/x-patch
Size: 25458 bytes
Desc: not available
URL: <http://lists.llvm.org/pipermail/llvm-commits/attachments/20180808/000a64f7/attachment.bin>


More information about the llvm-commits mailing list