Index: lib/AST/ASTContext.cpp =================================================================== --- lib/AST/ASTContext.cpp +++ lib/AST/ASTContext.cpp @@ -7999,10 +7999,9 @@ // We never need to emit an uninstantiated function template. if (FD->getTemplatedKind() == FunctionDecl::TK_FunctionTemplate) return false; - } else if (isa(D)) - return true; - else - return false; + } else { + return isa(D); + } // If this is a member of a class template, we do not need to emit it. if (D->getDeclContext()->isDependentContext()) Index: lib/AST/DeclBase.cpp =================================================================== --- lib/AST/DeclBase.cpp +++ lib/AST/DeclBase.cpp @@ -491,13 +491,9 @@ return true; // Objective-C classes, if this is the non-fragile runtime. - } else if (isa(this) && - getASTContext().getLangOpts().ObjCRuntime.hasWeakClassImport()) { - return true; - - // Nothing else. } else { - return false; + return isa(this) && + getASTContext().getLangOpts().ObjCRuntime.hasWeakClassImport(); } }