[PATCH] D36201: Merge manifest namespaces.
Rui Ueyama via Phabricator via llvm-commits
llvm-commits at lists.llvm.org
Fri Aug 18 17:10:32 PDT 2017
ruiu accepted this revision.
ruiu added a comment.
This revision is now accepted and ready to land.
LGTM
================
Comment at: llvm/tools/llvm-mt/llvm-mt.cpp:105-106
+ for (auto *Arg : InputArgs.filtered(OPT_INPUT))
+ reportError(Twine("invalid option ") + Arg->getSpelling());
+
----------------
Please add a test for an unknown option.
https://reviews.llvm.org/D36201
More information about the llvm-commits
mailing list