[PATCH] D16535: [clang-tidy] Check to find unintended semicolons that changes the semantics.

Gábor Horváth via cfe-commits cfe-commits at lists.llvm.org
Fri Feb 26 06:29:19 PST 2016


xazax.hun added a comment.

In http://reviews.llvm.org/D16535#362726, @alexfh wrote:

> I think, the check can be submitted as is and guards against spurious errors on invalid AST can be added as a follow up.


This check is already submitted. However I did not found any API in tidy to check whether the compilation failed. It would be great to be able to query that information at the time of registering matchers.

I have one more question though: does it make sense to run clang tidy at all, when the compilation failed? Isn't it a better default behaviour to not to run any of the checks in that case?


Repository:
  rL LLVM

http://reviews.llvm.org/D16535





More information about the cfe-commits mailing list