maxjhandsome
/
llvm-project
forked from OSchip/llvm-project
1
0
Fork 0
Code Issues Pull Requests Packages Releases Wiki Activity

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

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

Differential Revision: https://reviews.llvm.org/D54191

llvm-svn: 346470
This commit is contained in:
Petr Hosek 2018-11-09 03:16:53 +00:00
parent 397765bc51
commit e2f6896eef
3 changed files with 5 additions and 1 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

0
llvm/test/tools/llvm-rc/Inputs/empty.rc Normal file
View File

4
llvm/test/tools/llvm-rc/flags.test Normal file
View File

@ -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.

2
llvm/tools/llvm-rc/Opts.td
View File

@ -4,7 +4,7 @@ include "llvm/Option/OptParser.td"
// 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">,