[PATCH] D41799: [analyzer] PtrArithChecker: Update to use check::NewAllocator

2018-01-09 Thread Artem Dergachev via Phabricator via cfe-commits
NoQ planned changes to this revision. NoQ added a comment. https://reviews.llvm.org/D41250#971888 reverts the change in how casts work, and the checker doesn't really need the casted value, because it only tracks the pointer. This still leaves us with the problem of `PostStmt` being called twi

[PATCH] D41799: [analyzer] PtrArithChecker: Update to use check::NewAllocator

2018-01-09 Thread Gábor Horváth via Phabricator via cfe-commits
xazax.hun added a comment. > (@xazax.hun - this is an alpha checker last touched by you, do you still have > plans for it?) I think this check could be very useful. Last I checked it was too noisy (e.g.: around 850 hits on LLVM), and I did plan to look into what are the main sources of false p

[PATCH] D41799: [analyzer] PtrArithChecker: Update to use check::NewAllocator

2018-01-05 Thread Artem Dergachev via Phabricator via cfe-commits
NoQ created this revision. NoQ added reviewers: dcoughlin, xazax.hun, a.sidorin, george.karpenkov, szepet. Herald added subscribers: cfe-commits, rnkovacs. Make use of the new callback introduced in https://reviews.llvm.org/D41406 for tracking values allocated by `operator new()` in `-analyzer-co