[PATCH] D35796: [analyzer] Delete with non-virtual destructor check

2017-09-22 Thread Phabricator via Phabricator via cfe-commits
This revision was automatically updated to reflect the committed changes. Closed by commit rL313973: [analyzer] Add new delete with non-virtual destructor check (authored by xazax). Changed prior to commit: https://reviews.llvm.org/D35796?vs=116060=116319#toc Repository: rL LLVM

[PATCH] D35796: [analyzer] Delete with non-virtual destructor check

2017-09-22 Thread Reka Kovacs via Phabricator via cfe-commits
rnkovacs added a comment. In https://reviews.llvm.org/D35796#878200, @dcoughlin wrote: > This looks good to me! Do you have commit access, or do you need someone to > commit it for you? Thanks! I don't, so it would be nice if someone committed it for me. https://reviews.llvm.org/D35796

[PATCH] D35796: [analyzer] Delete with non-virtual destructor check

2017-09-21 Thread Devin Coughlin via Phabricator via cfe-commits
dcoughlin accepted this revision. dcoughlin added a comment. This revision is now accepted and ready to land. This looks good to me! Do you have commit access, or do you need someone to commit it for you? https://reviews.llvm.org/D35796 ___

[PATCH] D35796: [analyzer] Delete with non-virtual destructor check

2017-09-20 Thread Reka Kovacs via Phabricator via cfe-commits
rnkovacs updated this revision to Diff 116060. rnkovacs added a comment. - Accidentally left-in comment removed. - Checker file clang-formatted. https://reviews.llvm.org/D35796 Files: include/clang/StaticAnalyzer/Checkers/Checkers.td lib/StaticAnalyzer/Checkers/CMakeLists.txt

[PATCH] D35796: [analyzer] Delete with non-virtual destructor check

2017-09-14 Thread Reka Kovacs via Phabricator via cfe-commits
rnkovacs updated this revision to Diff 115198. rnkovacs marked an inline comment as done. rnkovacs retitled this revision from "[analyzer] Misused polymorphic object checker" to "[analyzer] Delete with non-virtual destructor check". rnkovacs edited the summary of this revision. rnkovacs added a