[PATCH] D38675: [analyzer] MoveChecker Pt.10: Move the checker out of alpha state.

2018-12-17 Thread Kristóf Umann via Phabricator via cfe-commits
Szelethus added a comment. A little late to the party, but I don't see anything against this. Repository: rC Clang CHANGES SINCE LAST ACTION https://reviews.llvm.org/D38675/new/ https://reviews.llvm.org/D38675 ___ cfe-commits mailing list cfe-

[PATCH] D38675: [analyzer] MoveChecker Pt.10: Move the checker out of alpha state.

2018-12-16 Thread Phabricator via Phabricator via cfe-commits
This revision was not accepted when it landed; it landed in state "Needs Review". This revision was automatically updated to reflect the committed changes. Closed by commit rC349328: [analyzer] MoveChecker: Enable by default as cplusplus.Move. (authored by dergachev, committed by ). Repository:

[PATCH] D38675: [analyzer] MoveChecker Pt.10: Move the checker out of alpha state.

2018-12-16 Thread Artem Dergachev via Phabricator via cfe-commits
NoQ added a comment. Currently the checker is very quiet. I have seen exactly 6 warnings on internal codebases on which i tested it (on which we otherwise emit around 12000 warnings), and all of them were good. The only false positive i've seen so far was fixed in D55566

[PATCH] D38675: [analyzer] MoveChecker Pt.10: Move the checker out of alpha state.

2018-12-14 Thread Artem Dergachev via Phabricator via cfe-commits
NoQ updated this revision to Diff 178324. NoQ retitled this revision from "[analyzer] MisusedMovedObjectChecker: Moving the checker out of alpha state" to "[analyzer] MoveChecker Pt.10: Move the checker out of alpha state.". NoQ added a comment. Rebase! CHANGES SINCE LAST ACTION https://revi