[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 05:43:56 PDT 2018


aaron.ballman added a comment.

In https://reviews.llvm.org/D16008#1035683, @alexfh wrote:

> If the CSA checker is still in alpha, I'd proceed with this check instead of investing time in polishing the CSA implementation.


Do you know why the CSA checker is still in alpha? As best I can tell, that check and this one are attempting to do the same work. I'm kind of worried about having competing functionalities because users have to guess as to which one is the "correct" one to use.


Repository:
  rL LLVM

https://reviews.llvm.org/D16008





More information about the llvm-commits mailing list