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

Martin Storsjö via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Thu Nov 8 01:14:51 PST 2018


mstorsjo accepted this revision.
mstorsjo added a comment.
This revision is now accepted and ready to land.

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


Repository:
  rL LLVM

https://reviews.llvm.org/D54191





More information about the llvm-commits mailing list