[PATCH] D54191: [llvm-rc] Support joined or separate spelling for /fo flag

Petr Hosek via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Tue Nov 6 17:47:36 PST 2018


phosek created this revision.
phosek added reviewers: mstorsjo, zturner, amccarth, thakis.
Herald added a subscriber: llvm-commits.

CMake invokes rc using the joined spelling which appears to be supported
by Microsoft's rc implementation, so we should support it as well.


Repository:
  rL LLVM

https://reviews.llvm.org/D54191

Files:
  llvm/tools/llvm-rc/Opts.td


Index: llvm/tools/llvm-rc/Opts.td
===================================================================
--- llvm/tools/llvm-rc/Opts.td
+++ llvm/tools/llvm-rc/Opts.td
@@ -4,7 +4,7 @@
 // These options seem to be important for the tool
 // and should be implemented.
 
-def FILEOUT : Separate<[ "/", "-" ], "FO">,
+def FILEOUT : JoinedOrSeparate<[ "/", "-" ], "FO">,
               HelpText<"Change the output file location.">;
 
 def DEFINE : Separate<[ "/", "-" ], "D">,


-------------- next part --------------
A non-text attachment was scrubbed...
Name: D54191.172882.patch
Type: text/x-patch
Size: 469 bytes
Desc: not available
URL: <http://lists.llvm.org/pipermail/llvm-commits/attachments/20181107/7899bedd/attachment.bin>


More information about the llvm-commits mailing list