[analyzer] Remove unnecessary code.
This code has been added a while ago and removing it does not trigger any test failures. The false positives it was trying to suppress are probably handled by other logic (ex: special handling of delegates). llvm-svn: 162529
This commit is contained in:
parent
5e52c2862a
commit
6fb4b055fb
|
@ -1358,22 +1358,6 @@ RetainSummaryManager::updateSummaryFromAnnotations(const RetainSummary *&Summ,
|
|||
const RetainSummary *
|
||||
RetainSummaryManager::getStandardMethodSummary(const ObjCMethodDecl *MD,
|
||||
Selector S, QualType RetTy) {
|
||||
|
||||
if (MD) {
|
||||
// Scan the method decl for 'void*' arguments. These should be treated
|
||||
// as 'StopTracking' because they are often used with delegates.
|
||||
// Delegates are a frequent form of false positives with the retain
|
||||
// count checker.
|
||||
unsigned i = 0;
|
||||
for (ObjCMethodDecl::param_const_iterator I = MD->param_begin(),
|
||||
E = MD->param_end(); I != E; ++I, ++i)
|
||||
if (const ParmVarDecl *PD = *I) {
|
||||
QualType Ty = Ctx.getCanonicalType(PD->getType());
|
||||
if (Ty.getLocalUnqualifiedType() == Ctx.VoidPtrTy)
|
||||
ScratchArgs = AF.add(ScratchArgs, i, StopTracking);
|
||||
}
|
||||
}
|
||||
|
||||
// Any special effects?
|
||||
ArgEffect ReceiverEff = DoNothing;
|
||||
RetEffect ResultEff = RetEffect::MakeNoRet();
|
||||
|
|
Loading…
Reference in New Issue