]> granicus.if.org Git - clang/commitdiff
[analyzer] Remove custom rule for OSIterator in RetainCountChecker
authorGeorge Karpenkov <ekarpenkov@apple.com>
Thu, 25 Oct 2018 23:38:24 +0000 (23:38 +0000)
committerGeorge Karpenkov <ekarpenkov@apple.com>
Thu, 25 Oct 2018 23:38:24 +0000 (23:38 +0000)
Differential Revision: https://reviews.llvm.org/D53628

git-svn-id: https://llvm.org/svn/llvm-project/cfe/trunk@345339 91177308-0d34-0410-b5e6-96231b3b80d8

lib/StaticAnalyzer/Core/RetainSummaryManager.cpp

index e9333266ce828f72379d47493adccb146b0590ec..1a2dc28a5713d3520387beafa31899b05329aa32 100644 (file)
@@ -69,10 +69,6 @@ static bool isOSObjectDynamicCast(StringRef S) {
   return S == "safeMetaCast";
 }
 
-static bool isOSIteratorSubclass(const Decl *D) {
-  return isSubclass(D, "OSIterator");
-}
-
 static bool hasRCAnnotation(const Decl *D, StringRef rcAnnotation) {
   for (const auto *Ann : D->specific_attrs<AnnotateAttr>()) {
     if (Ann->getAnnotation() == rcAnnotation)
@@ -240,10 +236,6 @@ RetainSummaryManager::generateSummary(const FunctionDecl *FD,
 
         // All objects returned with functions starting with "get" are getters.
         if (II->getName().startswith("get")) {
-
-          // ...except for iterators.
-          if (isOSIteratorSubclass(PD))
-            return getOSSummaryCreateRule(FD);
           return getOSSummaryGetRule(FD);
         } else {
           return getOSSummaryCreateRule(FD);