[PATCH] D15528: Teach clang-tidy how to -Werror checks.

Aaron Ballman via cfe-commits cfe-commits at lists.llvm.org
Tue Dec 15 08:50:55 PST 2015


aaron.ballman added inline comments.

================
Comment at: tools/llvm-project/extra/test/clang-tidy/werrors-plural.cpp:7
@@ +6,3 @@
+// CHECK-WARN: warning: namespace 'j' not terminated with a closing comment [llvm-namespace-comment]
+// CHECK-WERR: error: namespace 'j' not terminated with a closing comment [llvm-namespace-comment, -Werrors=]
+
----------------
jroelofs wrote:
> aaron.ballman wrote:
> > One of these tests should complete the -Werrors= chunk just to be sure it's getting printed properly.
> Not sure what you mean here. Should I be printing the -Werrors argument as the user wrote it? (it isn't currently... this is just a hardcoded string).
Ah, I didn't pick up on that. What is the = for, then?


http://reviews.llvm.org/D15528





More information about the cfe-commits mailing list