[PATCH] D116556: [llvm-objcopy] Implement the PE-COFF specific --subsystem option

[Martin Storsjö via Phabricator via llvm-commits]

mstorsjo updated this revision to Diff 397828.
mstorsjo marked 3 inline comments as done.
mstorsjo added a comment.

Mention the new option in the docs. Lowercase an error message. Add a test for erroring out on object files. Spell out the type instead of using `auto`.


Repository:
  rG LLVM Github Monorepo

CHANGES SINCE LAST ACTION
  https://reviews.llvm.org/D116556/new/

https://reviews.llvm.org/D116556

Files:
  llvm/docs/CommandGuide/llvm-objcopy.rst
  llvm/test/tools/llvm-objcopy/COFF/pe-fields.test
  llvm/test/tools/llvm-objcopy/COFF/subsystem.test
  llvm/tools/llvm-objcopy/COFF/COFFConfig.h
  llvm/tools/llvm-objcopy/COFF/COFFObjcopy.cpp
  llvm/tools/llvm-objcopy/ConfigManager.cpp
  llvm/tools/llvm-objcopy/ObjcopyOpts.td

-------------- next part --------------
A non-text attachment was scrubbed...
Name: D116556.397828.patch
Type: text/x-patch
Size: 9890 bytes
Desc: not available
URL: <http://lists.llvm.org/pipermail/llvm-commits/attachments/20220106/8423de5f/attachment.bin>