This revision was automatically updated to reflect the committed changes.
Closed by commit rL354643: [analyzer] MIGChecker: Add support for more APIs.
(authored by dergachev, committed by ).
Herald added a project: LLVM.
Herald added a subscriber: llvm-commits.
Changed prior to commit:
NoQ updated this revision to Diff 187876.
NoQ added a comment.
Address comments.
CHANGES SINCE LAST ACTION
https://reviews.llvm.org/D58397/new/
https://reviews.llvm.org/D58397
Files:
clang/lib/StaticAnalyzer/Checkers/MIGChecker.cpp
clang/test/Analysis/mig.mm
Index:
NoQ marked an inline comment as done.
NoQ added inline comments.
Comment at: clang/lib/StaticAnalyzer/Checkers/MIGChecker.cpp:125
+ // See if there's an annotated method in the superclass.
+ if (const auto *MD = dyn_cast(D))
dcoughlin wrote:
> Perhaps we
dcoughlin accepted this revision.
dcoughlin added a comment.
This revision is now accepted and ready to land.
Aah, MIG_NO_REPLY.
LGTM.
Comment at: clang/lib/StaticAnalyzer/Checkers/MIGChecker.cpp:42
+ std::vector> Deallocators = {
+#define CALL(required_args,
NoQ updated this revision to Diff 187505.
NoQ added a comment.
Rebase. Fix behavior when the return code is not constrained enough. Test the
C++11 attribute syntax (just in case). Update comments.
CHANGES SINCE LAST ACTION
https://reviews.llvm.org/D58397/new/
https://reviews.llvm.org/D58397
NoQ created this revision.
NoQ added a reviewer: dcoughlin.
Herald added subscribers: cfe-commits, jdoerfert, dkrupp, donat.nagy,
Szelethus, mikhail.ramalho, a.sidorin, szepet, baloghadamsoftware, xazax.hun.
Herald added a project: clang.
Previous patches were about the infrastructure of the
