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

[Petr Hosek via Phabricator via llvm-commits]

This revision was automatically updated to reflect the committed changes.
Closed by commit rL346470: [llvm-rc] Support joined or separate spelling for /fo flag (authored by phosek, committed by ).

Changed prior to commit:
  https://reviews.llvm.org/D54191?vs=173045&id=173260#toc

Repository:
  rL LLVM

https://reviews.llvm.org/D54191

Files:
  llvm/trunk/test/tools/llvm-rc/Inputs/empty.rc
  llvm/trunk/test/tools/llvm-rc/flags.test
  llvm/trunk/tools/llvm-rc/Opts.td


Index: llvm/trunk/tools/llvm-rc/Opts.td
===================================================================
--- llvm/trunk/tools/llvm-rc/Opts.td
+++ llvm/trunk/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">,
Index: llvm/trunk/test/tools/llvm-rc/flags.test
===================================================================
--- llvm/trunk/test/tools/llvm-rc/flags.test
+++ llvm/trunk/test/tools/llvm-rc/flags.test
@@ -0,0 +1,4 @@
+; RUN: llvm-rc /dry-run /FO %t %p/Inputs/empty.rc 2>&1 | FileCheck %s --allow-empty --check-prefix=FO
+; RUN: llvm-rc /dry-run /FO%t %p/Inputs/empty.rc 2>&1 | FileCheck %s --allow-empty --check-prefix=FO
+
+; FO-NOT: Exactly one input file should be provided.


-------------- next part --------------
A non-text attachment was scrubbed...
Name: D54191.173260.patch
Type: text/x-patch
Size: 970 bytes
Desc: not available
URL: <http://lists.llvm.org/pipermail/llvm-commits/attachments/20181109/5ae656c0/attachment.bin>