[PATCH] D58529: [analyzer] MIGChecker: Enable by default.

2019-02-21 Thread Artem Dergachev via Phabricator via cfe-commits
NoQ closed this revision. NoQ added a comment. Committed as rC354644 but i forgot to update the link as usual. Repository: rC Clang CHANGES SINCE LAST ACTION https://reviews.llvm.org/D58529/new/ https://reviews.llvm.org/D58529 ___

[PATCH] D58529: [analyzer] MIGChecker: Enable by default.

2019-02-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. LGTM. Repository: rC Clang CHANGES SINCE LAST ACTION https://reviews.llvm.org/D58529/new/ https://reviews.llvm.org/D58529 ___ cfe-com

[PATCH] D58529: [analyzer] MIGChecker: Enable by default.

2019-02-21 Thread Artem Dergachev via Phabricator via cfe-commits
NoQ added a comment. The TODOs for future polishing are: - Add a second `trackExpressionValue()` visitor to track argument values as they're copied around. - Introduce annotations for deallocator functions so that we no longer had to hardcode them and users could annotate their consuming functi

[PATCH] D58529: [analyzer] MIGChecker: Enable by default.

2019-02-21 Thread Artem Dergachev via Phabricator via cfe-commits
NoQ created this revision. NoQ added a reviewer: dcoughlin. Herald added subscribers: cfe-commits, dkrupp, donat.nagy, Szelethus, mikhail.ramalho, a.sidorin, szepet, baloghadamsoftware, xazax.hun. Herald added a project: clang. NoQ added a comment. The TODOs for future polishing are: - Add a sec