[PATCH] D56743: [llvm-rc] Support '--' for delimiting options from input paths

Zachary Turner via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Tue Jan 15 13:54:14 PST 2019


zturner added inline comments.


================
Comment at: tools/llvm-rc/llvm-rc.cpp:91
+      std::find_if(Argv + 1, Argv + Argc,
+                   [](const char *str) { return strcmp(str, "--") == 0; });
+  ArrayRef<const char *> ArgsArr = makeArrayRef(Argv + 1, DashDash);
----------------
Should this be `" -- "`?  I don't think `llvm-rc.exe foo--bar` should trigger this, that's one argument that happens to have 2 dashes.   


Repository:
  rL LLVM

CHANGES SINCE LAST ACTION
  https://reviews.llvm.org/D56743/new/

https://reviews.llvm.org/D56743





More information about the llvm-commits mailing list