aaron.ballman added a comment. > I do agree that it makes sense to keep it as low as possible, but also i see > a clear logic between all thee current checks:
Thank you for the explanation. I think that makes sense to me, but I'd like to hear from @alexfh before accepting. ================ Comment at: test/clang-tidy/check_clang_tidy.py:79-80 except subprocess.CalledProcessError as e: print('clang-tidy failed:\n' + e.output.decode()) raise print('------------------------ clang-tidy output -----------------------\n' + ---------------- I think the text should be rephrased to "CHECK-FIXES, CHECK-MESSAGES, or CHECK-NOTES not found in the input" Repository: rL LLVM https://reviews.llvm.org/D36892 _______________________________________________ cfe-commits mailing list cfe-commits@lists.llvm.org http://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits