[PATCH] D44236: [llvm-objcopy] Switch over to using TableGen for parsing arguments

Jake Ehrlich via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Wed Mar 7 16:45:29 PST 2018


jakehehrlich created this revision.
jakehehrlich added a reviewer: jhenderson.
Herald added a subscriber: mgorny.

So I'm starting back on the push to create llvm-strip which requires switching how arguments are being parsed. This change does the following

1. Deletes the old argument parsing code
2. Dactors out arguments into a Config class (will later be used repeatedly by llvm-strip)
3. Removes all uses of "-R=" and "-j=" because they're non-standard and hard to support in TableGen. I was wrong to use them in the first place
4. Adds in Opts.td (which will later be split into Common.td and Objcopy.td when Strip.td exists)
5. Adds in objcopy driver using Opts.td (later there will be a StripDriver and symlink)


Repository:
  rL LLVM

https://reviews.llvm.org/D44236

Files:
  llvm/test/tools/llvm-objcopy/add-section-remove.test
  llvm/test/tools/llvm-objcopy/add-section.test
  llvm/test/tools/llvm-objcopy/basic-only-keep.test
  llvm/test/tools/llvm-objcopy/binary-first-seg-offset-zero.test
  llvm/test/tools/llvm-objcopy/keep-only-keep.test
  llvm/test/tools/llvm-objcopy/only-keep-remove-strtab.test
  llvm/test/tools/llvm-objcopy/parent-loop-check.test
  llvm/test/tools/llvm-objcopy/remove-section.test
  llvm/tools/llvm-objcopy/CMakeLists.txt
  llvm/tools/llvm-objcopy/LLVMBuild.txt
  llvm/tools/llvm-objcopy/Opts.td
  llvm/tools/llvm-objcopy/llvm-objcopy.cpp

-------------- next part --------------
A non-text attachment was scrubbed...
Name: D44236.137511.patch
Type: text/x-patch
Size: 19480 bytes
Desc: not available
URL: <http://lists.llvm.org/pipermail/llvm-commits/attachments/20180308/908cbcce/attachment.bin>


More information about the llvm-commits mailing list