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

Eugene Zelenko via cfe-commits cfe-commits at lists.llvm.org
Fri Jan 8 17:25:59 PST 2016


Eugene.Zelenko added a subscriber: Eugene.Zelenko.
Eugene.Zelenko added a comment.

This check is duplicate of clang-analyzer-alpha.cplusplus.VirtualCall.

>From my point of view, Clang-tidy is better place, since such calls doesn't depend of run-time paths.

I think will be good idea to try to establish better functionality separation between Clang/Clang-tidy/Clang Static Analyzer. Current situation looks like different teams try to take everything coming to them without considering big picture. This my impression based on including padding check in Clang Static Analyzer.


Repository:
  rL LLVM

http://reviews.llvm.org/D16008





More information about the cfe-commits mailing list