diff options
Diffstat (limited to 'clang/lib')
| -rw-r--r-- | clang/lib/AST/ASTDumper.cpp | 4 | ||||
| -rw-r--r-- | clang/lib/AST/ASTImporter.cpp | 2 | ||||
| -rw-r--r-- | clang/lib/AST/CXXInheritance.cpp | 8 | ||||
| -rw-r--r-- | clang/lib/Basic/FileManager.cpp | 2 | ||||
| -rw-r--r-- | clang/lib/Driver/Driver.cpp | 2 | ||||
| -rw-r--r-- | clang/lib/Parse/ParseAST.cpp | 2 | ||||
| -rw-r--r-- | clang/lib/Parse/ParseDeclCXX.cpp | 2 | ||||
| -rw-r--r-- | clang/lib/Rewrite/Frontend/RewriteModernObjC.cpp | 16 | ||||
| -rw-r--r-- | clang/lib/Rewrite/Frontend/RewriteObjC.cpp | 16 | ||||
| -rw-r--r-- | clang/lib/Sema/SemaDecl.cpp | 2 | ||||
| -rw-r--r-- | clang/lib/Sema/SemaExpr.cpp | 2 | ||||
| -rw-r--r-- | clang/lib/Sema/SemaInit.cpp | 6 | ||||
| -rw-r--r-- | clang/lib/Sema/SemaLambda.cpp | 8 | ||||
| -rw-r--r-- | clang/lib/Sema/SemaLookup.cpp | 4 | ||||
| -rw-r--r-- | clang/lib/Sema/SemaStmt.cpp | 2 | ||||
| -rw-r--r-- | clang/lib/Sema/SemaTemplateInstantiate.cpp | 18 | ||||
| -rw-r--r-- | clang/lib/Serialization/ASTWriter.cpp | 8 | ||||
| -rw-r--r-- | clang/lib/StaticAnalyzer/Checkers/MallocChecker.cpp | 4 | ||||
| -rw-r--r-- | clang/lib/StaticAnalyzer/Checkers/SimpleStreamChecker.cpp | 4 | ||||
| -rw-r--r-- | clang/lib/StaticAnalyzer/Core/BugReporter.cpp | 2 | 
20 files changed, 57 insertions, 57 deletions
diff --git a/clang/lib/AST/ASTDumper.cpp b/clang/lib/AST/ASTDumper.cpp index dddc4d4c18f..c66676f1ad4 100644 --- a/clang/lib/AST/ASTDumper.cpp +++ b/clang/lib/AST/ASTDumper.cpp @@ -335,8 +335,8 @@ void ASTDumper::indent() {      OS << "\n";    ColorScope Color(*this, IndentColor); -  for (llvm::SmallVector<IndentType, 32>::const_iterator I = Indents.begin(), -                                                         E = Indents.end(); +  for (SmallVectorImpl<IndentType>::const_iterator I = Indents.begin(), +                                                   E = Indents.end();         I != E; ++I) {      switch (*I) {      case IT_Child: diff --git a/clang/lib/AST/ASTImporter.cpp b/clang/lib/AST/ASTImporter.cpp index 84904580162..deac0c5dfbe 100644 --- a/clang/lib/AST/ASTImporter.cpp +++ b/clang/lib/AST/ASTImporter.cpp @@ -4410,7 +4410,7 @@ Decl *ASTImporter::Import(Decl *FromD) {    } else if (TypedefNameDecl *FromTypedef = dyn_cast<TypedefNameDecl>(FromD)) {      // When we've finished transforming a typedef, see whether it was the      // typedef for an anonymous tag. -    for (SmallVector<TagDecl *, 4>::iterator +    for (SmallVectorImpl<TagDecl *>::iterator                 FromTag = AnonTagsWithPendingTypedefs.begin(),               FromTagEnd = AnonTagsWithPendingTypedefs.end();           FromTag != FromTagEnd; ++FromTag) { diff --git a/clang/lib/AST/CXXInheritance.cpp b/clang/lib/AST/CXXInheritance.cpp index 0e0b35d92ad..6195a7dc68e 100644 --- a/clang/lib/AST/CXXInheritance.cpp +++ b/clang/lib/AST/CXXInheritance.cpp @@ -447,7 +447,7 @@ FindNestedNameSpecifierMember(const CXXBaseSpecifier *Specifier,  void OverridingMethods::add(unsigned OverriddenSubobject,                               UniqueVirtualMethod Overriding) { -  SmallVector<UniqueVirtualMethod, 4> &SubobjectOverrides +  SmallVectorImpl<UniqueVirtualMethod> &SubobjectOverrides      = Overrides[OverriddenSubobject];    if (std::find(SubobjectOverrides.begin(), SubobjectOverrides.end(),                   Overriding) == SubobjectOverrides.end()) @@ -650,11 +650,11 @@ CXXRecordDecl::getFinalOverriders(CXXFinalOverriderMap &FinalOverriders) const {                                    SOEnd = OM->second.end();           SO != SOEnd;            ++SO) { -      SmallVector<UniqueVirtualMethod, 4> &Overriding = SO->second; +      SmallVectorImpl<UniqueVirtualMethod> &Overriding = SO->second;        if (Overriding.size() < 2)          continue; -      for (SmallVector<UniqueVirtualMethod, 4>::iterator  +      for (SmallVectorImpl<UniqueVirtualMethod>::iterator               Pos = Overriding.begin(), PosEnd = Overriding.end();             Pos != PosEnd;             /* increment in loop */) { @@ -669,7 +669,7 @@ CXXRecordDecl::getFinalOverriders(CXXFinalOverriderMap &FinalOverriders) const {          // in a base class subobject that hides the virtual base class          // subobject.          bool Hidden = false; -        for (SmallVector<UniqueVirtualMethod, 4>::iterator +        for (SmallVectorImpl<UniqueVirtualMethod>::iterator                 OP = Overriding.begin(), OPEnd = Overriding.end();               OP != OPEnd && !Hidden;                ++OP) { diff --git a/clang/lib/Basic/FileManager.cpp b/clang/lib/Basic/FileManager.cpp index 9cc59027ab6..d8deac72c46 100644 --- a/clang/lib/Basic/FileManager.cpp +++ b/clang/lib/Basic/FileManager.cpp @@ -610,7 +610,7 @@ void FileManager::GetUniqueIDMapping(        UIDToFiles[FE->getValue()->getUID()] = FE->getValue();    // Map virtual file entries -  for (SmallVector<FileEntry*, 4>::const_iterator  +  for (SmallVectorImpl<FileEntry *>::const_iterator           VFE = VirtualFileEntries.begin(), VFEEnd = VirtualFileEntries.end();         VFE != VFEEnd; ++VFE)      if (*VFE && *VFE != NON_EXISTENT_FILE) diff --git a/clang/lib/Driver/Driver.cpp b/clang/lib/Driver/Driver.cpp index 135af0404b1..3f3d0014491 100644 --- a/clang/lib/Driver/Driver.cpp +++ b/clang/lib/Driver/Driver.cpp @@ -1077,7 +1077,7 @@ void Driver::BuildActions(const ToolChain &TC, const DerivedArgList &Args,      // Build the pipeline for this file.      OwningPtr<Action> Current(new InputAction(*InputArg, InputType)); -    for (llvm::SmallVector<phases::ID, phases::MaxNumberOfPhases>::iterator +    for (SmallVectorImpl<phases::ID>::iterator             i = PL.begin(), e = PL.end(); i != e; ++i) {        phases::ID Phase = *i; diff --git a/clang/lib/Parse/ParseAST.cpp b/clang/lib/Parse/ParseAST.cpp index 7cd8a21ac45..5678ece0c82 100644 --- a/clang/lib/Parse/ParseAST.cpp +++ b/clang/lib/Parse/ParseAST.cpp @@ -145,7 +145,7 @@ void clang::ParseAST(Sema &S, bool PrintStats, bool SkipFunctionBodies) {    }    // Process any TopLevelDecls generated by #pragma weak. -  for (SmallVector<Decl*,2>::iterator +  for (SmallVectorImpl<Decl *>::iterator         I = S.WeakTopLevelDecls().begin(),         E = S.WeakTopLevelDecls().end(); I != E; ++I)      Consumer->HandleTopLevelDecl(DeclGroupRef(*I)); diff --git a/clang/lib/Parse/ParseDeclCXX.cpp b/clang/lib/Parse/ParseDeclCXX.cpp index d630879390f..67f9c3c7a7c 100644 --- a/clang/lib/Parse/ParseDeclCXX.cpp +++ b/clang/lib/Parse/ParseDeclCXX.cpp @@ -2243,7 +2243,7 @@ void Parser::ParseCXXClassMemberDeclaration(AccessSpecifier AS,        SmallVector<SourceRange, 4> Ranges;        DeclaratorInfo.getCXX11AttributeRanges(Ranges);        if (!Ranges.empty()) { -        for (SmallVector<SourceRange, 4>::iterator I = Ranges.begin(),  +        for (SmallVectorImpl<SourceRange>::iterator I = Ranges.begin(),               E = Ranges.end(); I != E; ++I) {            Diag((*I).getBegin(), diag::err_attributes_not_allowed)               << *I; diff --git a/clang/lib/Rewrite/Frontend/RewriteModernObjC.cpp b/clang/lib/Rewrite/Frontend/RewriteModernObjC.cpp index 0e59b113c96..fd1f26347ac 100644 --- a/clang/lib/Rewrite/Frontend/RewriteModernObjC.cpp +++ b/clang/lib/Rewrite/Frontend/RewriteModernObjC.cpp @@ -4262,7 +4262,7 @@ std::string RewriteModernObjC::SynthesizeBlockFunc(BlockExpr *CE, int i,    // Create local declarations to avoid rewriting all closure decl ref exprs.    // First, emit a declaration for all "by ref" decls. -  for (SmallVector<ValueDecl*,8>::iterator I = BlockByRefDecls.begin(), +  for (SmallVectorImpl<ValueDecl *>::iterator I = BlockByRefDecls.begin(),         E = BlockByRefDecls.end(); I != E; ++I) {      S += "  ";      std::string Name = (*I)->getNameAsString(); @@ -4273,7 +4273,7 @@ std::string RewriteModernObjC::SynthesizeBlockFunc(BlockExpr *CE, int i,      S += Name + " = __cself->" + (*I)->getNameAsString() + "; // bound by ref\n";    }    // Next, emit a declaration for all "by copy" declarations. -  for (SmallVector<ValueDecl*,8>::iterator I = BlockByCopyDecls.begin(), +  for (SmallVectorImpl<ValueDecl *>::iterator I = BlockByCopyDecls.begin(),         E = BlockByCopyDecls.end(); I != E; ++I) {      S += "  ";      // Handle nested closure invocation. For example: @@ -4374,7 +4374,7 @@ std::string RewriteModernObjC::SynthesizeBlockImpl(BlockExpr *CE, std::string Ta    if (BlockDeclRefs.size()) {      // Output all "by copy" declarations. -    for (SmallVector<ValueDecl*,8>::iterator I = BlockByCopyDecls.begin(), +    for (SmallVectorImpl<ValueDecl *>::iterator I = BlockByCopyDecls.begin(),           E = BlockByCopyDecls.end(); I != E; ++I) {        S += "  ";        std::string FieldName = (*I)->getNameAsString(); @@ -4403,7 +4403,7 @@ std::string RewriteModernObjC::SynthesizeBlockImpl(BlockExpr *CE, std::string Ta        S += FieldName + ";\n";      }      // Output all "by ref" declarations. -    for (SmallVector<ValueDecl*,8>::iterator I = BlockByRefDecls.begin(), +    for (SmallVectorImpl<ValueDecl *>::iterator I = BlockByRefDecls.begin(),           E = BlockByRefDecls.end(); I != E; ++I) {        S += "  ";        std::string FieldName = (*I)->getNameAsString(); @@ -4422,7 +4422,7 @@ std::string RewriteModernObjC::SynthesizeBlockImpl(BlockExpr *CE, std::string Ta      Constructor += ", int flags=0)";      // Initialize all "by copy" arguments.      bool firsTime = true; -    for (SmallVector<ValueDecl*,8>::iterator I = BlockByCopyDecls.begin(), +    for (SmallVectorImpl<ValueDecl *>::iterator I = BlockByCopyDecls.begin(),           E = BlockByCopyDecls.end(); I != E; ++I) {        std::string Name = (*I)->getNameAsString();          if (firsTime) { @@ -4437,7 +4437,7 @@ std::string RewriteModernObjC::SynthesizeBlockImpl(BlockExpr *CE, std::string Ta            Constructor += Name + "(_" + Name + ")";      }      // Initialize all "by ref" arguments. -    for (SmallVector<ValueDecl*,8>::iterator I = BlockByRefDecls.begin(), +    for (SmallVectorImpl<ValueDecl *>::iterator I = BlockByRefDecls.begin(),           E = BlockByRefDecls.end(); I != E; ++I) {        std::string Name = (*I)->getNameAsString();        if (firsTime) { @@ -5520,7 +5520,7 @@ Stmt *RewriteModernObjC::SynthBlockInitExpr(BlockExpr *Exp,    if (BlockDeclRefs.size()) {      Expr *Exp;      // Output all "by copy" declarations. -    for (SmallVector<ValueDecl*,8>::iterator I = BlockByCopyDecls.begin(), +    for (SmallVectorImpl<ValueDecl *>::iterator I = BlockByCopyDecls.begin(),           E = BlockByCopyDecls.end(); I != E; ++I) {        if (isObjCType((*I)->getType())) {          // FIXME: Conform to ABI ([[obj retain] autorelease]). @@ -5554,7 +5554,7 @@ Stmt *RewriteModernObjC::SynthBlockInitExpr(BlockExpr *Exp,        InitExprs.push_back(Exp);      }      // Output all "by ref" declarations. -    for (SmallVector<ValueDecl*,8>::iterator I = BlockByRefDecls.begin(), +    for (SmallVectorImpl<ValueDecl *>::iterator I = BlockByRefDecls.begin(),           E = BlockByRefDecls.end(); I != E; ++I) {        ValueDecl *ND = (*I);        std::string Name(ND->getNameAsString()); diff --git a/clang/lib/Rewrite/Frontend/RewriteObjC.cpp b/clang/lib/Rewrite/Frontend/RewriteObjC.cpp index 2f5cd0f6c6c..662efaa63a7 100644 --- a/clang/lib/Rewrite/Frontend/RewriteObjC.cpp +++ b/clang/lib/Rewrite/Frontend/RewriteObjC.cpp @@ -3366,7 +3366,7 @@ std::string RewriteObjC::SynthesizeBlockFunc(BlockExpr *CE, int i,    // Create local declarations to avoid rewriting all closure decl ref exprs.    // First, emit a declaration for all "by ref" decls. -  for (SmallVector<ValueDecl*,8>::iterator I = BlockByRefDecls.begin(), +  for (SmallVectorImpl<ValueDecl *>::iterator I = BlockByRefDecls.begin(),         E = BlockByRefDecls.end(); I != E; ++I) {      S += "  ";      std::string Name = (*I)->getNameAsString(); @@ -3377,7 +3377,7 @@ std::string RewriteObjC::SynthesizeBlockFunc(BlockExpr *CE, int i,      S += Name + " = __cself->" + (*I)->getNameAsString() + "; // bound by ref\n";    }    // Next, emit a declaration for all "by copy" declarations. -  for (SmallVector<ValueDecl*,8>::iterator I = BlockByCopyDecls.begin(), +  for (SmallVectorImpl<ValueDecl *>::iterator I = BlockByCopyDecls.begin(),         E = BlockByCopyDecls.end(); I != E; ++I) {      S += "  ";      // Handle nested closure invocation. For example: @@ -3478,7 +3478,7 @@ std::string RewriteObjC::SynthesizeBlockImpl(BlockExpr *CE, std::string Tag,    if (BlockDeclRefs.size()) {      // Output all "by copy" declarations. -    for (SmallVector<ValueDecl*,8>::iterator I = BlockByCopyDecls.begin(), +    for (SmallVectorImpl<ValueDecl *>::iterator I = BlockByCopyDecls.begin(),           E = BlockByCopyDecls.end(); I != E; ++I) {        S += "  ";        std::string FieldName = (*I)->getNameAsString(); @@ -3507,7 +3507,7 @@ std::string RewriteObjC::SynthesizeBlockImpl(BlockExpr *CE, std::string Tag,        S += FieldName + ";\n";      }      // Output all "by ref" declarations. -    for (SmallVector<ValueDecl*,8>::iterator I = BlockByRefDecls.begin(), +    for (SmallVectorImpl<ValueDecl *>::iterator I = BlockByRefDecls.begin(),           E = BlockByRefDecls.end(); I != E; ++I) {        S += "  ";        std::string FieldName = (*I)->getNameAsString(); @@ -3526,7 +3526,7 @@ std::string RewriteObjC::SynthesizeBlockImpl(BlockExpr *CE, std::string Tag,      Constructor += ", int flags=0)";      // Initialize all "by copy" arguments.      bool firsTime = true; -    for (SmallVector<ValueDecl*,8>::iterator I = BlockByCopyDecls.begin(), +    for (SmallVectorImpl<ValueDecl *>::iterator I = BlockByCopyDecls.begin(),           E = BlockByCopyDecls.end(); I != E; ++I) {        std::string Name = (*I)->getNameAsString();          if (firsTime) { @@ -3541,7 +3541,7 @@ std::string RewriteObjC::SynthesizeBlockImpl(BlockExpr *CE, std::string Tag,            Constructor += Name + "(_" + Name + ")";      }      // Initialize all "by ref" arguments. -    for (SmallVector<ValueDecl*,8>::iterator I = BlockByRefDecls.begin(), +    for (SmallVectorImpl<ValueDecl *>::iterator I = BlockByRefDecls.begin(),           E = BlockByRefDecls.end(); I != E; ++I) {        std::string Name = (*I)->getNameAsString();        if (firsTime) { @@ -4548,7 +4548,7 @@ Stmt *RewriteObjC::SynthBlockInitExpr(BlockExpr *Exp,    if (BlockDeclRefs.size()) {      Expr *Exp;      // Output all "by copy" declarations. -    for (SmallVector<ValueDecl*,8>::iterator I = BlockByCopyDecls.begin(), +    for (SmallVectorImpl<ValueDecl *>::iterator I = BlockByCopyDecls.begin(),           E = BlockByCopyDecls.end(); I != E; ++I) {        if (isObjCType((*I)->getType())) {          // FIXME: Conform to ABI ([[obj retain] autorelease]). @@ -4582,7 +4582,7 @@ Stmt *RewriteObjC::SynthBlockInitExpr(BlockExpr *Exp,        InitExprs.push_back(Exp);      }      // Output all "by ref" declarations. -    for (SmallVector<ValueDecl*,8>::iterator I = BlockByRefDecls.begin(), +    for (SmallVectorImpl<ValueDecl *>::iterator I = BlockByRefDecls.begin(),           E = BlockByRefDecls.end(); I != E; ++I) {        ValueDecl *ND = (*I);        std::string Name(ND->getNameAsString()); diff --git a/clang/lib/Sema/SemaDecl.cpp b/clang/lib/Sema/SemaDecl.cpp index 162b5b8c091..389655686c9 100644 --- a/clang/lib/Sema/SemaDecl.cpp +++ b/clang/lib/Sema/SemaDecl.cpp @@ -5810,7 +5810,7 @@ static NamedDecl* DiagnoseInvalidRedeclaration(    if (CXXMethodDecl *NewMD = dyn_cast<CXXMethodDecl>(NewFD))      NewFDisConst = NewMD->isConst(); -  for (SmallVector<std::pair<FunctionDecl *, unsigned>, 1>::iterator +  for (SmallVectorImpl<std::pair<FunctionDecl *, unsigned>>::iterator         NearMatch = NearMatches.begin(), NearMatchEnd = NearMatches.end();         NearMatch != NearMatchEnd; ++NearMatch) {      FunctionDecl *FD = NearMatch->first; diff --git a/clang/lib/Sema/SemaExpr.cpp b/clang/lib/Sema/SemaExpr.cpp index 6f3997eb368..d2f784c1d5b 100644 --- a/clang/lib/Sema/SemaExpr.cpp +++ b/clang/lib/Sema/SemaExpr.cpp @@ -1350,7 +1350,7 @@ Sema::CreateGenericSelectionExpr(SourceLocation KeyLoc,      Diag(ControllingExpr->getLocStart(), diag::err_generic_sel_multi_match)        << ControllingExpr->getSourceRange() << ControllingExpr->getType()        << (unsigned) CompatIndices.size(); -    for (SmallVector<unsigned, 1>::iterator I = CompatIndices.begin(), +    for (SmallVectorImpl<unsigned>::iterator I = CompatIndices.begin(),           E = CompatIndices.end(); I != E; ++I) {        Diag(Types[*I]->getTypeLoc().getBeginLoc(),             diag::note_compat_assoc) diff --git a/clang/lib/Sema/SemaInit.cpp b/clang/lib/Sema/SemaInit.cpp index 18328553d38..96291923bd2 100644 --- a/clang/lib/Sema/SemaInit.cpp +++ b/clang/lib/Sema/SemaInit.cpp @@ -3383,7 +3383,7 @@ static OverloadingResult TryRefInitWithConversionFunction(Sema &S,      // be changed while iterating (e.g. because of deserialization).      // To be safe we copy the lookup results to a new container.      SmallVector<NamedDecl*, 16> Ctors(R.begin(), R.end()); -    for (SmallVector<NamedDecl*, 16>::iterator +    for (SmallVectorImpl<NamedDecl *>::iterator             CI = Ctors.begin(), CE = Ctors.end(); CI != CE; ++CI) {        NamedDecl *D = *CI;        DeclAccessPair FoundDecl = DeclAccessPair::make(D, D->getAccess()); @@ -4009,7 +4009,7 @@ static void TryUserDefinedConversion(Sema &S,        // be changed while iterating. To be safe we copy the lookup results        // to a new container.        SmallVector<NamedDecl*, 8> CopyOfCon(R.begin(), R.end()); -      for (SmallVector<NamedDecl*, 8>::iterator +      for (SmallVectorImpl<NamedDecl *>::iterator               Con = CopyOfCon.begin(), ConEnd = CopyOfCon.end();             Con != ConEnd; ++Con) {          NamedDecl *D = *Con; @@ -4716,7 +4716,7 @@ static void LookupCopyAndMoveConstructors(Sema &S,    // be changed while iterating (e.g. because of deserialization).    // To be safe we copy the lookup results to a new container.    SmallVector<NamedDecl*, 16> Ctors(R.begin(), R.end()); -  for (SmallVector<NamedDecl*, 16>::iterator +  for (SmallVectorImpl<NamedDecl *>::iterator           CI = Ctors.begin(), CE = Ctors.end(); CI != CE; ++CI) {      NamedDecl *D = *CI;      CXXConstructorDecl *Constructor = 0; diff --git a/clang/lib/Sema/SemaLambda.cpp b/clang/lib/Sema/SemaLambda.cpp index b2dbf0553f4..1e6ef9baf70 100644 --- a/clang/lib/Sema/SemaLambda.cpp +++ b/clang/lib/Sema/SemaLambda.cpp @@ -608,10 +608,10 @@ void Sema::ActOnStartOfLambdaDefinition(LambdaIntroducer &Intro,    // Handle explicit captures.    SourceLocation PrevCaptureLoc      = Intro.Default == LCD_None? Intro.Range.getBegin() : Intro.DefaultLoc; -  for (SmallVector<LambdaCapture, 4>::const_iterator -         C = Intro.Captures.begin(),  -         E = Intro.Captures.end();  -       C != E;  +  for (SmallVectorImpl<LambdaCapture>::const_iterator +         C = Intro.Captures.begin(), +         E = Intro.Captures.end(); +       C != E;         PrevCaptureLoc = C->Loc, ++C) {      if (C->Kind == LCK_This) {        // C++11 [expr.prim.lambda]p8: diff --git a/clang/lib/Sema/SemaLookup.cpp b/clang/lib/Sema/SemaLookup.cpp index d3fd69982fb..2538fad7c53 100644 --- a/clang/lib/Sema/SemaLookup.cpp +++ b/clang/lib/Sema/SemaLookup.cpp @@ -3470,8 +3470,8 @@ void NamespaceSpecifierSet::SortNamespaces() {      std::sort(sortedDistances.begin(), sortedDistances.end());    Specifiers.clear(); -  for (SmallVector<unsigned, 4>::iterator DI = sortedDistances.begin(), -                                       DIEnd = sortedDistances.end(); +  for (SmallVectorImpl<unsigned>::iterator DI = sortedDistances.begin(), +                                        DIEnd = sortedDistances.end();         DI != DIEnd; ++DI) {      SpecifierInfoList &SpecList = DistanceMap[*DI];      Specifiers.append(SpecList.begin(), SpecList.end()); diff --git a/clang/lib/Sema/SemaStmt.cpp b/clang/lib/Sema/SemaStmt.cpp index 7c2ee9dbe77..45162744b1d 100644 --- a/clang/lib/Sema/SemaStmt.cpp +++ b/clang/lib/Sema/SemaStmt.cpp @@ -1411,7 +1411,7 @@ namespace {      // Load SourceRanges into diagnostic if there is room.      // Otherwise, load the SourceRange of the conditional expression.      if (Ranges.size() <= PartialDiagnostic::MaxArguments) -      for (SmallVector<SourceRange, 10>::iterator I = Ranges.begin(), +      for (SmallVectorImpl<SourceRange>::iterator I = Ranges.begin(),                                                    E = Ranges.end();             I != E; ++I)          PDiag << *I; diff --git a/clang/lib/Sema/SemaTemplateInstantiate.cpp b/clang/lib/Sema/SemaTemplateInstantiate.cpp index 9babafedc95..dd9ceae50d5 100644 --- a/clang/lib/Sema/SemaTemplateInstantiate.cpp +++ b/clang/lib/Sema/SemaTemplateInstantiate.cpp @@ -435,7 +435,7 @@ void Sema::PrintInstantiationStack() {    // FIXME: In all of these cases, we need to show the template arguments    unsigned InstantiationIdx = 0; -  for (SmallVector<ActiveTemplateInstantiation, 16>::reverse_iterator +  for (SmallVectorImpl<ActiveTemplateInstantiation>::reverse_iterator           Active = ActiveTemplateInstantiations.rbegin(),           ActiveEnd = ActiveTemplateInstantiations.rend();         Active != ActiveEnd; @@ -615,7 +615,7 @@ Optional<TemplateDeductionInfo *> Sema::isSFINAEContext() const {    if (InNonInstantiationSFINAEContext)      return Optional<TemplateDeductionInfo *>(0); -  for (SmallVector<ActiveTemplateInstantiation, 16>::const_reverse_iterator +  for (SmallVectorImpl<ActiveTemplateInstantiation>::const_reverse_iterator           Active = ActiveTemplateInstantiations.rbegin(),           ActiveEnd = ActiveTemplateInstantiations.rend();         Active != ActiveEnd; @@ -2286,7 +2286,7 @@ Sema::InstantiateClassTemplateSpecialization(    SmallVector<const NamedDecl *, 4> InstantiatedTemplateParameters;    if (Matched.size() >= 1) { -    SmallVector<MatchResult, 4>::iterator Best = Matched.begin(); +    SmallVectorImpl<MatchResult>::iterator Best = Matched.begin();      if (Matched.size() == 1) {        //   -- If exactly one matching specialization is found, the        //      instantiation is generated from that specialization. @@ -2299,8 +2299,8 @@ Sema::InstantiateClassTemplateSpecialization(        //      specialized than all of the other matching        //      specializations, then the use of the class template is        //      ambiguous and the program is ill-formed. -      for (SmallVector<MatchResult, 4>::iterator P = Best + 1, -                                                    PEnd = Matched.end(); +      for (SmallVectorImpl<MatchResult>::iterator P = Best + 1, +                                               PEnd = Matched.end();             P != PEnd; ++P) {          if (getMoreSpecializedPartialSpecialization(P->Partial, Best->Partial,                                                      PointOfInstantiation)  @@ -2311,8 +2311,8 @@ Sema::InstantiateClassTemplateSpecialization(        // Determine if the best partial specialization is more specialized than        // the others.        bool Ambiguous = false; -      for (SmallVector<MatchResult, 4>::iterator P = Matched.begin(), -                                                    PEnd = Matched.end(); +      for (SmallVectorImpl<MatchResult>::iterator P = Matched.begin(), +                                               PEnd = Matched.end();             P != PEnd; ++P) {          if (P != Best &&              getMoreSpecializedPartialSpecialization(P->Partial, Best->Partial, @@ -2330,8 +2330,8 @@ Sema::InstantiateClassTemplateSpecialization(            << ClassTemplateSpec;          // Print the matching partial specializations. -        for (SmallVector<MatchResult, 4>::iterator P = Matched.begin(), -                                                      PEnd = Matched.end(); +        for (SmallVectorImpl<MatchResult>::iterator P = Matched.begin(), +                                                 PEnd = Matched.end();               P != PEnd; ++P)            Diag(P->Partial->getLocation(), diag::note_partial_spec_match)              << getTemplateArgumentBindingsText( diff --git a/clang/lib/Serialization/ASTWriter.cpp b/clang/lib/Serialization/ASTWriter.cpp index bf01e9c2f27..b9061ad7562 100644 --- a/clang/lib/Serialization/ASTWriter.cpp +++ b/clang/lib/Serialization/ASTWriter.cpp @@ -3120,7 +3120,7 @@ public:      // Only emit declarations that aren't from a chained PCH, though.      SmallVector<Decl *, 16> Decls(IdResolver.begin(II),                                    IdResolver.end()); -    for (SmallVector<Decl *, 16>::reverse_iterator D = Decls.rbegin(), +    for (SmallVectorImpl<Decl *>::reverse_iterator D = Decls.rbegin(),                                                  DEnd = Decls.rend();           D != DEnd; ++D)        clang::io::Emit32(Out, Writer.getDeclID(getMostRecentLocalDecl(*D))); @@ -4022,7 +4022,7 @@ void ASTWriter::WriteASTCore(Sema &SemaRef,    // Make sure visible decls, added to DeclContexts previously loaded from    // an AST file, are registered for serialization. -  for (SmallVector<const Decl *, 16>::iterator +  for (SmallVectorImpl<const Decl *>::iterator           I = UpdatingVisibleDecls.begin(),           E = UpdatingVisibleDecls.end(); I != E; ++I) {      GetDeclRef(*I); @@ -4296,8 +4296,8 @@ void ASTWriter::WriteDeclReplacementsBlock() {      return;    RecordData Record; -  for (SmallVector<ReplacedDeclInfo, 16>::iterator -           I = ReplacedDecls.begin(), E = ReplacedDecls.end(); I != E; ++I) { +  for (SmallVectorImpl<ReplacedDeclInfo>::iterator +         I = ReplacedDecls.begin(), E = ReplacedDecls.end(); I != E; ++I) {      Record.push_back(I->ID);      Record.push_back(I->Offset);      Record.push_back(I->Loc); diff --git a/clang/lib/StaticAnalyzer/Checkers/MallocChecker.cpp b/clang/lib/StaticAnalyzer/Checkers/MallocChecker.cpp index c5286d1a508..1229a326fdf 100644 --- a/clang/lib/StaticAnalyzer/Checkers/MallocChecker.cpp +++ b/clang/lib/StaticAnalyzer/Checkers/MallocChecker.cpp @@ -1670,8 +1670,8 @@ void MallocChecker::checkDeadSymbols(SymbolReaper &SymReaper,    if (!Errors.empty()) {      static SimpleProgramPointTag Tag("MallocChecker : DeadSymbolsLeak");      N = C.addTransition(C.getState(), C.getPredecessor(), &Tag); -    for (SmallVector<SymbolRef, 2>::iterator -        I = Errors.begin(), E = Errors.end(); I != E; ++I) { +    for (SmallVectorImpl<SymbolRef>::iterator +           I = Errors.begin(), E = Errors.end(); I != E; ++I) {        reportLeak(*I, N, C);      }    } diff --git a/clang/lib/StaticAnalyzer/Checkers/SimpleStreamChecker.cpp b/clang/lib/StaticAnalyzer/Checkers/SimpleStreamChecker.cpp index c2786e184b3..9ca0ab5d7fb 100644 --- a/clang/lib/StaticAnalyzer/Checkers/SimpleStreamChecker.cpp +++ b/clang/lib/StaticAnalyzer/Checkers/SimpleStreamChecker.cpp @@ -227,8 +227,8 @@ void SimpleStreamChecker::reportLeaks(SymbolVector LeakedStreams,                                                 ExplodedNode *ErrNode) const {    // Attach bug reports to the leak node.    // TODO: Identify the leaked file descriptor. -  for (SmallVector<SymbolRef, 2>::iterator -      I = LeakedStreams.begin(), E = LeakedStreams.end(); I != E; ++I) { +  for (SmallVectorImpl<SymbolRef>::iterator +         I = LeakedStreams.begin(), E = LeakedStreams.end(); I != E; ++I) {      BugReport *R = new BugReport(*LeakBugType,          "Opened file is never closed; potential resource leak", ErrNode);      R->markInteresting(*I); diff --git a/clang/lib/StaticAnalyzer/Core/BugReporter.cpp b/clang/lib/StaticAnalyzer/Core/BugReporter.cpp index 6b110ad28ba..912d5c501d6 100644 --- a/clang/lib/StaticAnalyzer/Core/BugReporter.cpp +++ b/clang/lib/StaticAnalyzer/Core/BugReporter.cpp @@ -2743,7 +2743,7 @@ void BugReporter::FlushReports() {    SmallVector<const BugType*, 16> bugTypes;    for (BugTypesTy::iterator I=BugTypes.begin(), E=BugTypes.end(); I!=E; ++I)      bugTypes.push_back(*I); -  for (SmallVector<const BugType*, 16>::iterator +  for (SmallVectorImpl<const BugType *>::iterator           I = bugTypes.begin(), E = bugTypes.end(); I != E; ++I)      const_cast<BugType*>(*I)->FlushReports(*this);  | 

