[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