[PATCH] D27409: [analyzer] RetainCountChecker: Improve support for libdispatch APIs.

2016-12-06 Thread Devin Coughlin via Phabricator via cfe-commits
dcoughlin added a comment. The analyzer currently doesn't do any checking for dispatch retain/release APIs in C. It similarly doesn't do any checking in Objective-C when OS_OBJECT_USE_OBJC is 0 (and thus the dispatch types are defined to their C-struct versions). This happens when the user

[PATCH] D27409: [analyzer] RetainCountChecker: Improve support for libdispatch APIs.

2016-12-06 Thread Devin Coughlin via Phabricator via cfe-commits
dcoughlin added a comment. In https://reviews.llvm.org/D27409#614601, @NoQ wrote: > Not sure we need to stay merged with `retain-release-arc.m`, as we're not > really reusing many declarations across these files. I find it more helpful to organize tests around functionality ('retain count

[PATCH] D27409: [analyzer] RetainCountChecker: Improve support for libdispatch APIs.

2016-12-06 Thread Artem Dergachev via Phabricator via cfe-commits
NoQ retitled this revision from "[analyzer] RetainCountChecker: The callback in dispatch_data_create() doesn't free the return symbol." to "[analyzer] RetainCountChecker: Improve support for libdispatch APIs.". NoQ updated the summary for this revision. NoQ updated this revision to Diff 80432.