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

Reply via email to