alexfh added a comment.

Feel free to commit this to fix buildbots, however the underlying 
https://reviews.llvm.org/D33013 raises some questions.

Clang-tidy is also used as a library, in which case it should capture all 
compiler diagnostics and pass them to the caller. This test verifies that 
clang-tidy actually captures the compilation error, and with 
https://reviews.llvm.org/D33013 this seems to have broken for the diagnostics 
related to command line parsing. Modifying the driver to directly output these 
errors to stderr instead of using `DiagnosticConsumer` seems like a wrong thing 
to do.


https://reviews.llvm.org/D33173



_______________________________________________
cfe-commits mailing list
cfe-commits@lists.llvm.org
http://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits

Reply via email to