aganea added inline comments.
================
Comment at: llvm/tools/llvm-rc/llvm-rc.cpp:259
-} // anonymous namespace
+static bool consume_back_lower(StringRef &S, const char *str) {
+ if (!S.endswith_lower(str))
----------------
`s/str/Str/`
================
Comment at: llvm/tools/llvm-rc/llvm-rc.cpp:300
+
+std::string unescape(StringRef S) {
+ std::string Out;
----------------
I would also need this function in D43002 (see unescapeSlashes), do you think
we can move it to `sys::path` or any other "support" lib?
================
Comment at: llvm/tools/llvm-rc/llvm-rc.cpp:567
+ if (InputArgs.hasArg(OPT_lang_id)) {
+ if (InputArgs.getLastArgValue(OPT_lang_id).getAsInteger(16, Opts.LangId))
+ fatalError("Invalid language id: " +
----------------
There was a latent issue here - unrelated to the moving of code around - I
don't know if you want to fix it now or after? That is, `s/16/0/`.
Repository:
rG LLVM Github Monorepo
CHANGES SINCE LAST ACTION
https://reviews.llvm.org/D100756/new/
https://reviews.llvm.org/D100756
_______________________________________________
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
