[PATCH] D116556: [llvm-objcopy] Implement the PE-COFF specific --subsystem option
Martin Storsjö via Phabricator via llvm-commits
llvm-commits at lists.llvm.org
Thu Jan 6 02:52:40 PST 2022
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>
More information about the llvm-commits
mailing list