diff options
| author | George Karpenkov <ekarpenkov@apple.com> | 2018-10-25 23:38:24 +0000 |
|---|---|---|
| committer | George Karpenkov <ekarpenkov@apple.com> | 2018-10-25 23:38:24 +0000 |
| commit | 7cdccb14594bc877dfe1153b73b31849630542bb (patch) | |
| tree | 3a877ddeff0975ffaeeff647f16647681f37a2c3 /clang/lib | |
| parent | 3c2ed8f3386eebae956267998c2d9aaec9105181 (diff) | |
| download | bcm5719-llvm-7cdccb14594bc877dfe1153b73b31849630542bb.tar.gz bcm5719-llvm-7cdccb14594bc877dfe1153b73b31849630542bb.zip | |
[analyzer] Remove custom rule for OSIterator in RetainCountChecker
Differential Revision: https://reviews.llvm.org/D53628
llvm-svn: 345339
Diffstat (limited to 'clang/lib')
| -rw-r--r-- | clang/lib/StaticAnalyzer/Core/RetainSummaryManager.cpp | 8 |
1 files changed, 0 insertions, 8 deletions
diff --git a/clang/lib/StaticAnalyzer/Core/RetainSummaryManager.cpp b/clang/lib/StaticAnalyzer/Core/RetainSummaryManager.cpp index e9333266ce8..1a2dc28a571 100644 --- a/clang/lib/StaticAnalyzer/Core/RetainSummaryManager.cpp +++ b/clang/lib/StaticAnalyzer/Core/RetainSummaryManager.cpp @@ -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); |

