svenpanne added a comment.

I think `ninja clang-test-tools` is the right target here, and it is happy with 
the change, which is not very surprising: AFAICT, there is just a single, very 
simple check for `run-clang-tidy.py`. Anyway, my manual testing with and 
without `-quiet` shows no problems, either. Furthermore, we have a patched 
version with the above change in our CI for quite some time, and it works fine. 
And if I read the git history correctly, the output should be identical to the 
one before a fix for interleaved output was landed (that fix added the newlines 
somehow).


Repository:
  rCTE Clang Tools Extra

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

https://reviews.llvm.org/D61850



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

Reply via email to