[PATCH] D60139: [clang-tidy] Add misc-placement-new-target-type-mismatch check

2019-04-08 Thread Dennis Luxen via Phabricator via cfe-commits
DennisL updated this revision to Diff 194135. DennisL added a comment. The following has been updated: - moved check to bugprone instead of misc - more tests - rewritten check logic CHANGES SINCE LAST ACTION https://reviews.llvm.org/D60139/new/ https://reviews.llvm.org/D60139 Files:

[PATCH] D60139: [clang-tidy] Add misc-placement-new-target-type-mismatch check

2019-04-05 Thread Dennis Luxen via Phabricator via cfe-commits
DennisL added a comment. In D60139#1456123 , @alexfh wrote: > Looks like this check would fit better into the `bugprone` module. Excellent suggestion. Thanks. Will follow up with an updated Diff. CHANGES SINCE LAST ACTION

[PATCH] D60139: [clang-tidy] Add misc-placement-new-target-type-mismatch check

2019-04-05 Thread Alexander Kornienko via Phabricator via cfe-commits
alexfh added a comment. Looks like this check would fit better into the `bugprone` module. CHANGES SINCE LAST ACTION https://reviews.llvm.org/D60139/new/ https://reviews.llvm.org/D60139 ___ cfe-commits mailing list cfe-commits@lists.llvm.org

[PATCH] D60139: [clang-tidy] Add misc-placement-new-target-type-mismatch check

2019-04-05 Thread Dennis Luxen via Phabricator via cfe-commits
DennisL updated this revision to Diff 193839. DennisL marked an inline comment as done. DennisL added a comment. Sync ReleaseNotes.rst with docs CHANGES SINCE LAST ACTION https://reviews.llvm.org/D60139/new/ https://reviews.llvm.org/D60139 Files: clang-tidy/misc/CMakeLists.txt

[PATCH] D60139: [clang-tidy] Add misc-placement-new-target-type-mismatch check

2019-04-04 Thread Eugene Zelenko via Phabricator via cfe-commits
Eugene.Zelenko added inline comments. Comment at: docs/ReleaseNotes.rst:136 + + Finds placement-new calls where the size of the pointee type of the placement + parameter is smaller than the size of the constructed type and the pointer is Please synchronize

[PATCH] D60139: [clang-tidy] Add misc-placement-new-target-type-mismatch check

2019-04-04 Thread Dennis Luxen via Phabricator via cfe-commits
DennisL updated this revision to Diff 193715. DennisL marked an inline comment as done. DennisL retitled this revision from "[clang-tidy] Add misc-placement-new-target-size check" to "[clang-tidy] Add misc-placement-new-target-type-mismatch check". DennisL edited the summary of this revision.