[PATCH] D45458: MallocChecker refactoring of calls checkers

2018-04-12 Thread Gábor Horváth via Phabricator via cfe-commits
xazax.hun added a comment. In https://reviews.llvm.org/D45458#1062342, @NoQ wrote: > @xazax.hun: do you think the approach you took in the `Valist` checker is > applicable here? Did you like how it ended up working? Cause i'd love to see > `CallDescription` and initializer lists used for readab

[PATCH] D45458: MallocChecker refactoring of calls checkers

2018-04-09 Thread Artem Dergachev via Phabricator via cfe-commits
NoQ added a reviewer: xazax.hun. NoQ added a subscriber: xazax.hun. NoQ added a comment. Herald added a subscriber: rnkovacs. @xazax.hun: do you think the approach you took in the `Valist` checker is applicable here? Did you like how it ended up working? Cause i'd love to see `CallDescription` a

[PATCH] D45458: MallocChecker refactoring of calls checkers

2018-04-09 Thread David CARLIER via Phabricator via cfe-commits
devnexen added a comment. Splitting from https://reviews.llvm.org/D45149 as requested. Repository: rC Clang https://reviews.llvm.org/D45458 ___ cfe-commits mailing list cfe-commits@lists.llvm.org http://lists.llvm.org/cgi-bin/mailman/listinfo/cfe

[PATCH] D45458: MallocChecker refactoring of calls checkers

2018-04-09 Thread David CARLIER via Phabricator via cfe-commits
devnexen created this revision. devnexen added reviewers: NoQ, thakis. Herald added a subscriber: cfe-commits. - Splitting some by family to reduce the list of function identifiers. Repository: rC Clang https://reviews.llvm.org/D45458 Files: lib/StaticAnalyzer/Checkers/MallocChecker.cpp I