[PATCH] D16008: [clang-tidy] Add calling virtual functions in constructors/destructors check.

Aaron Ballman via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Tue Mar 13 17:47:26 PDT 2018


aaron.ballman added a comment.

In https://reviews.llvm.org/D16008#1035948, @malcolm.parsons wrote:

> In https://reviews.llvm.org/D16008#1035789, @aaron.ballman wrote:
>
> > Do you know why the CSA checker is still in alpha?
>
>
> It isn't - https://reviews.llvm.org/D26768 moved it to optin.
>
> I don't know why https://reviews.llvm.org/D34275 didn't turn it on by default.


Then I'm not certain the utility of this check given that you can call the CSA check from clang-tidy directly; however, I do like the idea of exposing the CSA check under the name cert-oop50-cpp.


Repository:
  rL LLVM

https://reviews.llvm.org/D16008





More information about the llvm-commits mailing list