CMake invokes rc using the joined spelling which appears to be supported
by Microsoft's rc implementation, so we should support it as well.
Details
Diff Detail
- Repository
- rL LLVM
Event Timeline
Could you add a test for this? (I guess it's easiest to change one of the existing tests to use a different syntax in one place.)
Also, this isn't the only option that can be JoinedOrSeparate, with a quick test it seems like this goes for at least CODEPAGE and DEFINE as well, probably for all of them. But I won't demand fixing all of them at once, as they're tangential to what you need, and many of them aren't really implemented in llvm-rc yet.
LGTM
Fwiw, if you use llvm-rc with flags starting with a dash instead of a slash, llvm-rc -FO/path/to/out.res input.rc does run successfully and silently (but doesn't output the file in the intended place, the option just appears to be ignored). That might be an issue someone (myself included) perhaps should take a look at at some point...