[PATCH] D67139: [llvm-objcopy] Refactor ELF-specific config out to ELFCopyConfig. NFC.

Seiya Nuta via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Tue Sep 3 19:24:43 PDT 2019


seiya created this revision.
Herald added subscribers: llvm-commits, abrachet, MaskRay, jakehehrlich, arichardson, emaste.
Herald added a reviewer: espindola.
Herald added a reviewer: alexshap.
Herald added a reviewer: rupprecht.
Herald added a reviewer: jhenderson.
Herald added a project: LLVM.

This patch splits the command-line parsing into two phases.

First, it parses cross-platform options and leaves ELF-specific options unparsed in UnparsedCopyConfig.

Second, in the ELF implementation, it parses ELF-specific options and creates ELFCopyConfig.


https://reviews.llvm.org/D67139

Files:
  llvm/tools/llvm-objcopy/COFF/COFFObjcopy.cpp
  llvm/tools/llvm-objcopy/CopyConfig.cpp
  llvm/tools/llvm-objcopy/CopyConfig.h
  llvm/tools/llvm-objcopy/ELF/ELFObjcopy.cpp
  llvm/tools/llvm-objcopy/ELF/ELFObjcopy.h
  llvm/tools/llvm-objcopy/MachO/MachOObjcopy.cpp

-------------- next part --------------
A non-text attachment was scrubbed...
Name: D67139.218580.patch
Type: text/x-patch
Size: 42429 bytes
Desc: not available
URL: <http://lists.llvm.org/pipermail/llvm-commits/attachments/20190904/19572184/attachment-0001.bin>


More information about the llvm-commits mailing list