r290773 - Wdocumentation fix

Richard Smith via cfe-commits cfe-commits at lists.llvm.org
Fri Dec 30 15:12:32 PST 2016


On 30 Dec 2016 3:06 pm, "Simon Pilgrim via cfe-commits" <
cfe-commits at lists.llvm.org> wrote:

Author: rksimon
Date: Fri Dec 30 16:55:33 2016
New Revision: 290773

URL: http://llvm.org/viewvc/llvm-project?rev=290773&view=rev
Log:
Wdocumentation fix


Thanks, but please be careful you don't introduce trailing whitespace
changes to unrelated code in the future.

Also perhaps we should turn this warning on by default for regular
selfhosted builds so we don't need to wait for a buildbot to find issues
like this?

Modified:
    cfe/trunk/lib/Sema/SemaTemplate.cpp

Modified: cfe/trunk/lib/Sema/SemaTemplate.cpp
URL: http://llvm.org/viewvc/llvm-project/cfe/trunk/lib/Sema/
SemaTemplate.cpp?rev=290773&r1=290772&r2=290773&view=diff
============================================================
==================
--- cfe/trunk/lib/Sema/SemaTemplate.cpp (original)
+++ cfe/trunk/lib/Sema/SemaTemplate.cpp Fri Dec 30 16:55:33 2016
@@ -88,14 +88,14 @@ static NamedDecl *isAcceptableTemplateNa
   return nullptr;
 }

-void Sema::FilterAcceptableTemplateNames(LookupResult &R,
+void Sema::FilterAcceptableTemplateNames(LookupResult &R,
                                          bool AllowFunctionTemplates) {
   // The set of class templates we've already seen.
   llvm::SmallPtrSet<ClassTemplateDecl *, 8> ClassTemplates;
   LookupResult::Filter filter = R.makeFilter();
   while (filter.hasNext()) {
     NamedDecl *Orig = filter.next();
-    NamedDecl *Repl = isAcceptableTemplateName(Context, Orig,
+    NamedDecl *Repl = isAcceptableTemplateName(Context, Orig,
                                                AllowFunctionTemplates);
     if (!Repl)
       filter.erase();
@@ -131,7 +131,7 @@ bool Sema::hasAnyAcceptableTemplateNames
   for (LookupResult::iterator I = R.begin(), IEnd = R.end(); I != IEnd;
++I)
     if (isAcceptableTemplateName(Context, *I, AllowFunctionTemplates))
       return true;
-
+
   return false;
 }

@@ -265,7 +265,7 @@ void Sema::LookupTemplateName(LookupResu
     assert((isDependent || !ObjectType->isIncompleteType() ||
             ObjectType->castAs<TagType>()->isBeingDefined()) &&
            "Caller should have completed object type");
-
+
     // Template names cannot appear inside an Objective-C class or object
type.
     if (ObjectType->isObjCObjectOrInterfaceType()) {
       Found.clear();
@@ -312,7 +312,7 @@ void Sema::LookupTemplateName(LookupResu
   } else {
     // Perform unqualified name lookup in the current scope.
     LookupName(Found, S);
-
+
     if (!ObjectType.isNull())
       AllowFunctionTemplatesInLookup = false;
   }
@@ -890,7 +890,7 @@ Decl *Sema::ActOnTemplateTemplateParamet
                                      Depth, Position, IsParameterPack,
                                      Name, Params);
   Param->setAccess(AS_public);
-
+
   // If the template template parameter has a name, then link the
identifier
   // into the scope and lookup mechanisms.
   if (Name) {
@@ -1020,8 +1020,8 @@ Sema::CheckClassTemplate(Scope *S, unsig
     if (RequireCompleteDeclContext(SS, SemanticContext))
       return true;

-    // If we're adding a template to a dependent context, we may need to
-    // rebuilding some of the types used within the template parameter
list,
+    // If we're adding a template to a dependent context, we may need to
+    // rebuilding some of the types used within the template parameter
list,
     // now that we know what the current instantiation is.
     if (SemanticContext->isDependentContext()) {
       ContextRAII SavedContext(*this, SemanticContext);
@@ -1247,10 +1247,10 @@ Sema::CheckClassTemplate(Scope *S, unsig
                                 DeclarationName(Name), TemplateParams,
                                 NewClass, PrevClassTemplate);
   NewClass->setDescribedClassTemplate(NewTemplate);
-
+
   if (ModulePrivateLoc.isValid())
     NewTemplate->setModulePrivate();
-
+
   // Build the type for the class template declaration now.
   QualType T = NewTemplate->getInjectedClassNameSpecialization();
   T = Context.getInjectedClassNameType(NewClass, T);
@@ -1341,7 +1341,7 @@ static bool DiagnoseDefaultTemplateArgum
     //   A default template-argument shall not be specified in a
     //   function template declaration or a function template
     //   definition [...]
-    //   If a friend function template declaration specifies a default
+    //   If a friend function template declaration specifies a default
     //   template-argument, that declaration shall be a definition and
shall be
     //   the only declaration of the function template in the translation
unit.
     // (C++98/03 doesn't have this wording; see DR226).
@@ -1768,10 +1768,10 @@ static SourceRange getRangeOfTypeInNeste
         return NNSLoc.getTypeLoc().getSourceRange();
     } else
       break;
-
+
     NNSLoc = NNSLoc.getPrefix();
   }
-
+
   return SourceRange();
 }

@@ -1814,34 +1814,34 @@ TemplateParameterList *Sema::MatchTempla
     bool &IsExplicitSpecialization, bool &Invalid) {
   IsExplicitSpecialization = false;
   Invalid = false;
-
+
   // The sequence of nested types to which we will match up the template
   // parameter lists. We first build this list by starting with the type
named
   // by the nested-name-specifier and walking out until we run out of
types.
   SmallVector<QualType, 4> NestedTypes;
   QualType T;
   if (SS.getScopeRep()) {
-    if (CXXRecordDecl *Record
+    if (CXXRecordDecl *Record
               = dyn_cast_or_null<CXXRecordDecl>(computeDeclContext(SS,
true)))
       T = Context.getTypeDeclType(Record);
     else
       T = QualType(SS.getScopeRep()->getAsType(), 0);
   }
-
+
   // If we found an explicit specialization that prevents us from needing
   // 'template<>' headers, this will be set to the location of that
   // explicit specialization.
   SourceLocation ExplicitSpecLoc;
-
+
   while (!T.isNull()) {
     NestedTypes.push_back(T);
-
+
     // Retrieve the parent of a record type.
     if (CXXRecordDecl *Record = T->getAsCXXRecordDecl()) {
       // If this type is an explicit specialization, we're done.
       if (ClassTemplateSpecializationDecl *Spec
           = dyn_cast<ClassTemplateSpecializationDecl>(Record)) {
-        if (!isa<ClassTemplatePartialSpecializationDecl>(Spec) &&
+        if (!isa<ClassTemplatePartialSpecializationDecl>(Spec) &&
             Spec->getSpecializationKind() == TSK_ExplicitSpecialization) {
           ExplicitSpecLoc = Spec->getLocation();
           break;
@@ -1851,14 +1851,14 @@ TemplateParameterList *Sema::MatchTempla
         ExplicitSpecLoc = Record->getLocation();
         break;
       }
-
+
       if (TypeDecl *Parent = dyn_cast<TypeDecl>(Record->getParent()))
         T = Context.getTypeDeclType(Parent);
       else
         T = QualType();
       continue;
-    }
-
+    }
+
     if (const TemplateSpecializationType *TST
                                      = T->getAs<TemplateSpecializationType>())
{
       if (TemplateDecl *Template = TST->getTemplateName().getAsTemplateDecl())
{
@@ -1866,10 +1866,10 @@ TemplateParameterList *Sema::MatchTempla
           T = Context.getTypeDeclType(Parent);
         else
           T = QualType();
-        continue;
+        continue;
       }
     }
-
+
     // Look one step prior in a dependent template specialization type.
     if (const DependentTemplateSpecializationType *DependentTST
                           = T->getAs<DependentTemplateSpecializationType>())
{
@@ -1879,7 +1879,7 @@ TemplateParameterList *Sema::MatchTempla
         T = QualType();
       continue;
     }
-
+
     // Look one step prior in a dependent name type.
     if (const DependentNameType *DependentName =
T->getAs<DependentNameType>()){
       if (NestedNameSpecifier *NNS = DependentName->getQualifier())
@@ -1888,18 +1888,18 @@ TemplateParameterList *Sema::MatchTempla
         T = QualType();
       continue;
     }
-
+
     // Retrieve the parent of an enumeration type.
     if (const EnumType *EnumT = T->getAs<EnumType>()) {
       // FIXME: Forward-declared enums require a TSK_ExplicitSpecialization
       // check here.
       EnumDecl *Enum = EnumT->getDecl();
-
+
       // Get to the parent type.
       if (TypeDecl *Parent = dyn_cast<TypeDecl>(Enum->getParent()))
         T = Context.getTypeDeclType(Parent);
       else
-        T = QualType();
+        T = QualType();
       continue;
     }

@@ -1951,21 +1951,21 @@ TemplateParameterList *Sema::MatchTempla
   for (unsigned TypeIdx = 0, NumTypes = NestedTypes.size(); TypeIdx !=
NumTypes;
        ++TypeIdx) {
     T = NestedTypes[TypeIdx];
-
+
     // Whether we expect a 'template<>' header.
     bool NeedEmptyTemplateHeader = false;

     // Whether we expect a template header with parameters.
     bool NeedNonemptyTemplateHeader = false;
-
+
     // For a dependent type, the set of template parameters that we
     // expect to see.
     TemplateParameterList *ExpectedTemplateParams = nullptr;

     // C++0x [temp.expl.spec]p15:
-    //   A member or a member template may be nested within many enclosing
-    //   class templates. In an explicit specialization for such a member,
the
-    //   member declaration shall be preceded by a template<> for each
+    //   A member or a member template may be nested within many enclosing
+    //   class templates. In an explicit specialization for such a member,
the
+    //   member declaration shall be preceded by a template<> for each
     //   enclosing class template that is explicitly specialized.
     if (CXXRecordDecl *Record = T->getAsCXXRecordDecl()) {
       if (ClassTemplatePartialSpecializationDecl *Partial
@@ -1982,38 +1982,38 @@ TemplateParameterList *Sema::MatchTempla
                      = dyn_cast<ClassTemplateSpecializationDecl>(Record)) {
         // C++0x [temp.expl.spec]p4:
         //   Members of an explicitly specialized class template are
defined
-        //   in the same manner as members of normal classes, and not using
-        //   the template<> syntax.
+        //   in the same manner as members of normal classes, and not using
+        //   the template<> syntax.
         if (Spec->getSpecializationKind() != TSK_ExplicitSpecialization)
           NeedEmptyTemplateHeader = true;
         else
           continue;
       } else if (Record->getTemplateSpecializationKind()) {
-        if (Record->getTemplateSpecializationKind()
+        if (Record->getTemplateSpecializationKind()
                                                 !=
TSK_ExplicitSpecialization &&
             TypeIdx == NumTypes - 1)
           IsExplicitSpecialization = true;
-
+
         continue;
       }
     } else if (const TemplateSpecializationType *TST
                                      = T->getAs<TemplateSpecializationType>())
{
       if (TemplateDecl *Template = TST->getTemplateName().getAsTemplateDecl())
{
         ExpectedTemplateParams = Template->getTemplateParameters();
-        NeedNonemptyTemplateHeader = true;
+        NeedNonemptyTemplateHeader = true;
       }
     } else if (T->getAs<DependentTemplateSpecializationType>()) {
       // FIXME:  We actually could/should check the template arguments here
       // against the corresponding template parameter list.
       NeedNonemptyTemplateHeader = false;
-    }
-
+    }
+
     // C++ [temp.expl.spec]p16:
-    //   In an explicit specialization declaration for a member of a class
-    //   template or a member template that ap- pears in namespace scope,
the
-    //   member template and some of its enclosing class templates may
remain
-    //   unspecialized, except that the declaration shall not explicitly
-    //   specialize a class member template if its en- closing class
templates
+    //   In an explicit specialization declaration for a member of a class
+    //   template or a member template that ap- pears in namespace scope,
the
+    //   member template and some of its enclosing class templates may
remain
+    //   unspecialized, except that the declaration shall not explicitly
+    //   specialize a class member template if its en- closing class
templates
     //   are not explicitly specialized as well.
     if (ParamIdx < ParamLists.size()) {
       if (ParamLists[ParamIdx]->size() == 0) {
@@ -2023,7 +2023,7 @@ TemplateParameterList *Sema::MatchTempla
       } else
         SawNonEmptyTemplateParameterList = true;
     }
-
+
     if (NeedEmptyTemplateHeader) {
       // If we're on the last of the types, and we need a 'template<>'
header
       // here, then it's an explicit specialization.
@@ -2033,7 +2033,7 @@ TemplateParameterList *Sema::MatchTempla
       if (ParamIdx < ParamLists.size()) {
         if (ParamLists[ParamIdx]->size() > 0) {
           // The header has template parameters when it shouldn't.
Complain.
-          Diag(ParamLists[ParamIdx]->getTemplateLoc(),
+          Diag(ParamLists[ParamIdx]->getTemplateLoc(),
                diag::err_template_param_list_matches_nontemplate)
             << T
             << SourceRange(ParamLists[ParamIdx]->getLAngleLoc(),
@@ -2065,7 +2065,7 @@ TemplateParameterList *Sema::MatchTempla
         if (ParamIdx < ParamLists.size() &&
             DependsOnTemplateParameters(T, ParamLists[ParamIdx]))
           ExpectedTemplateParams = nullptr;
-        else
+        else
           continue;
       }

@@ -2081,11 +2081,11 @@ TemplateParameterList *Sema::MatchTempla
             CheckTemplateParameterList(ParamLists[ParamIdx], nullptr,
                                        TPC_ClassTemplateMember))
           Invalid = true;
-
+
         ++ParamIdx;
         continue;
       }
-
+
       Diag(DeclLoc, diag::err_template_spec_needs_template_parameters)
         << T
         << getRangeOfTypeInNestedNameSpecifier(Context, T, SS);
@@ -2135,10 +2135,10 @@ TemplateParameterList *Sema::MatchTempla
     // not required, and there were any 'template<>' headers, note where
the
     // specialization occurred.
     if (ExplicitSpecLoc.isValid() && HasAnyExplicitSpecHeader)
-      Diag(ExplicitSpecLoc,
+      Diag(ExplicitSpecLoc,
            diag::note_explicit_template_spec_does_not_need_header)
         << NestedTypes.back();
-
+
     // We have a template parameter list with no corresponding scope, which
     // means that the resulting template declaration can't be instantiated
     // properly (we'll end up with dependent nodes when we shouldn't).
@@ -2147,11 +2147,11 @@ TemplateParameterList *Sema::MatchTempla
   }

   // C++ [temp.expl.spec]p16:
-  //   In an explicit specialization declaration for a member of a class
-  //   template or a member template that ap- pears in namespace scope, the
-  //   member template and some of its enclosing class templates may remain
-  //   unspecialized, except that the declaration shall not explicitly
-  //   specialize a class member template if its en- closing class
templates
+  //   In an explicit specialization declaration for a member of a class
+  //   template or a member template that ap- pears in namespace scope, the
+  //   member template and some of its enclosing class templates may remain
+  //   unspecialized, except that the declaration shall not explicitly
+  //   specialize a class member template if its en- closing class
templates
   //   are not explicitly specialized as well.
   if (ParamLists.back()->size() == 0 &&
       CheckExplicitSpecialization(ParamLists[ParamIdx]->getSourceRange(),
@@ -2176,14 +2176,14 @@ void Sema::NoteAllFoundTemplates(Templat
         << Template->getDeclName();
     return;
   }
-
+
   if (OverloadedTemplateStorage *OST = Name.getAsOverloadedTemplate()) {
-    for (OverloadedTemplateStorage::iterator I = OST->begin(),
+    for (OverloadedTemplateStorage::iterator I = OST->begin(),
                                           IEnd = OST->end();
          I != IEnd; ++I)
       Diag((*I)->getLocation(), diag::note_template_declared_here)
         << 0 << (*I)->getDeclName();
-
+
     return;
   }
 }
@@ -2459,7 +2459,7 @@ Sema::ActOnTemplateIdType(CXXScopeSpec &
       SpecTL.setArgLocInfo(I, TemplateArgs[I].getLocInfo());
     return CreateParsedType(T, TLB.getTypeSourceInfo(Context, T));
   }
-
+
   QualType Result = CheckTemplateIdType(Template, TemplateLoc,
TemplateArgs);

   if (Result.isNull())
@@ -2486,7 +2486,7 @@ Sema::ActOnTemplateIdType(CXXScopeSpec &
     ElabTL.setElaboratedKeywordLoc(SourceLocation());
     ElabTL.setQualifierLoc(SS.getWithLocInContext(Context));
   }
-
+
   return CreateParsedType(Result, TLB.getTypeSourceInfo(Context, Result));
 }

@@ -2501,11 +2501,11 @@ TypeResult Sema::ActOnTagTemplateIdType(
                                         ASTTemplateArgsPtr TemplateArgsIn,
                                         SourceLocation RAngleLoc) {
   TemplateName Template = TemplateD.get();
-
+
   // Translate the parser's template argument list in our AST format.
   TemplateArgumentListInfo TemplateArgs(LAngleLoc, RAngleLoc);
   translateTemplateArguments(TemplateArgsIn, TemplateArgs);
-
+
   // Determine the tag kind
   TagTypeKind TagKind = TypeWithKeyword::getTagTypeKindForTypeSpec(
TagSpec);
   ElaboratedTypeKeyword Keyword
@@ -2513,11 +2513,11 @@ TypeResult Sema::ActOnTagTemplateIdType(

   if (DependentTemplateName *DTN = Template.getAsDependentTemplateName()) {
     QualType T = Context.getDependentTemplateSpecializationType(Keyword,
-
DTN->getQualifier(),
-
DTN->getIdentifier(),
+
DTN->getQualifier(),
+
DTN->getIdentifier(),

 TemplateArgs);
-
-    // Build type-source information.
+
+    // Build type-source information.
     TypeLocBuilder TLB;
     DependentTemplateSpecializationTypeLoc SpecTL
       = TLB.push<DependentTemplateSpecializationTypeLoc>(T);
@@ -2542,18 +2542,18 @@ TypeResult Sema::ActOnTagTemplateIdType(
         << TAT << NTK_TypeAliasTemplate << TagKind;
     Diag(TAT->getLocation(), diag::note_declared_at);
   }
-
+
   QualType Result = CheckTemplateIdType(Template, TemplateLoc,
TemplateArgs);
   if (Result.isNull())
     return TypeResult(true);
-
+
   // Check the tag kind
   if (const RecordType *RT = Result->getAs<RecordType>()) {
     RecordDecl *D = RT->getDecl();
-
+
     IdentifierInfo *Id = D->getIdentifier();
     assert(Id && "templated class must have an identifier");
-
+
     if (!isAcceptableTagRedeclaration(D, TagKind, TUK == TUK_Definition,
                                       TagLoc, Id)) {
       Diag(TagLoc, diag::err_use_with_wrong_tag)
@@ -3409,7 +3409,7 @@ bool Sema::CheckTemplateTypeArgument(Tem

   // Add the converted template type argument.
   ArgType = Context.getCanonicalType(ArgType);
-
+
   // Objective-C ARC:
   //   If an explicitly-specified template argument type is a lifetime type
   //   with no lifetime qualifier, the __strong lifetime qualifier is
inferred.
@@ -3420,7 +3420,7 @@ bool Sema::CheckTemplateTypeArgument(Tem
     Qs.setObjCLifetime(Qualifiers::OCL_Strong);
     ArgType = Context.getQualifiedType(ArgType, Qs);
   }
-
+
   Converted.push_back(TemplateArgument(ArgType));
   return false;
 }
@@ -3550,7 +3550,7 @@ SubstDefaultTemplateArgument(Sema &SemaR
 /// \param Converted the list of template arguments provided for template
 /// parameters that precede \p Param in the template parameter list.
 ///
-/// \param QualifierLoc Will be set to the nested-name-specifier (with
+/// \param QualifierLoc Will be set to the nested-name-specifier (with
 /// source-location information) that precedes the template name.
 ///
 /// \returns the substituted template argument, or NULL if an error
occurred.
@@ -3901,7 +3901,7 @@ bool Sema::CheckTemplateArgument(NamedDe
   return false;
 }

-/// \brief Diagnose an arity mismatch in the
+/// \brief Diagnose an arity mismatch in the
 static bool diagnoseArityMismatch(Sema &S, TemplateDecl *Template,
                                   SourceLocation TemplateLoc,
                                   TemplateArgumentListInfo &TemplateArgs) {
@@ -3911,7 +3911,7 @@ static bool diagnoseArityMismatch(Sema &

   SourceRange Range;
   if (NumArgs > NumParams)
-    Range = SourceRange(TemplateArgs[NumParams].getLocation(),
+    Range = SourceRange(TemplateArgs[NumParams].getLocation(),
                         TemplateArgs.getRAngleLoc());
   S.Diag(TemplateLoc, diag::err_template_arg_list_different_arity)
     << (NumArgs > NumParams)
@@ -4535,20 +4535,20 @@ isNullPointerValueTemplateArgument(Sema

   if (!S.getLangOpts().CPlusPlus11)
     return NPV_NotNullPointer;
-
+
   // Determine whether we have a constant expression.
   ExprResult ArgRV = S.DefaultFunctionArrayConversion(Arg);
   if (ArgRV.isInvalid())
     return NPV_Error;
   Arg = ArgRV.get();
-
+
   Expr::EvalResult EvalResult;
   SmallVector<PartialDiagnosticAt, 8> Notes;
   EvalResult.Diag = &Notes;
   if (!Arg->EvaluateAsRValue(EvalResult, S.Context) ||
       EvalResult.HasSideEffects) {
     SourceLocation DiagLoc = Arg->getExprLoc();
-
+
     // If our only note is the usual "invalid subexpression" note, just
point
     // the caret at its location rather than producing an essentially
     // redundant note.
@@ -4557,21 +4557,21 @@ isNullPointerValueTemplateArgument(Sema
       DiagLoc = Notes[0].first;
       Notes.clear();
     }
-
+
     S.Diag(DiagLoc, diag::err_template_arg_not_address_constant)
       << Arg->getType() << Arg->getSourceRange();
     for (unsigned I = 0, N = Notes.size(); I != N; ++I)
       S.Diag(Notes[I].first, Notes[I].second);
-
+
     S.Diag(Param->getLocation(), diag::note_template_param_here);
     return NPV_Error;
   }
-
+
   // C++11 [temp.arg.nontype]p1:
   //   - an address constant expression of type std::nullptr_t
   if (Arg->getType()->isNullPtrType())
     return NPV_NullPointer;
-
+
   //   - a constant expression that evaluates to a null pointer value
(4.10); or
   //   - a constant expression that evaluates to a null member pointer
value
   //     (4.11); or
@@ -4584,7 +4584,7 @@ isNullPointerValueTemplateArgument(Sema
         S.IsQualificationConversion(Arg->getType(), ParamType, false,
                                      ObjCLifetimeConversion))
       return NPV_NullPointer;
-
+
     // The types didn't match, but we know we got a null pointer; complain,
     // then recover as if the types were correct.
     S.Diag(Arg->getExprLoc(), diag::err_template_arg_
wrongtype_null_constant)
@@ -4604,7 +4604,7 @@ isNullPointerValueTemplateArgument(Sema
     S.Diag(Param->getLocation(), diag::note_template_param_here);
     return NPV_NullPointer;
   }
-
+
   // FIXME: If we ever want to support general, address-constant
expressions
   // as non-type template arguments, we should return the ExprResult here
to
   // be interpreted by the caller.
@@ -5347,7 +5347,7 @@ ExprResult Sema::CheckTemplateArgument(N
     } else if (!Arg->isValueDependent()) {
       class TmplArgICEDiagnoser : public VerifyICEDiagnoser {
         QualType T;
-
+
       public:
         TmplArgICEDiagnoser(QualType T) : T(T) { }

@@ -5409,14 +5409,14 @@ ExprResult Sema::CheckTemplateArgument(N
       Value.setIsSigned(IntegerType->isSignedIntegerOrEnumerationType());
     } else {
       llvm::APSInt OldValue = Value;
-
+
       // Coerce the template argument's value to the value it will have
       // based on the template parameter's type.
       unsigned AllowedBits = Context.getTypeSize(IntegerType);
       if (Value.getBitWidth() != AllowedBits)
         Value = Value.extOrTrunc(AllowedBits);
       Value.setIsSigned(IntegerType->isSignedIntegerOrEnumerationType());
-
+
       // Complain if an unsigned parameter received a negative value.
       if (IntegerType->isUnsignedIntegerOrEnumerationType()
                && (OldValue.isSigned() && OldValue.isNegative())) {
@@ -5425,7 +5425,7 @@ ExprResult Sema::CheckTemplateArgument(N
           << Arg->getSourceRange();
         Diag(Param->getLocation(), diag::note_template_param_here);
       }
-
+
       // Complain if we overflowed the template parameter's type.
       unsigned RequiredBits;
       if (IntegerType->isUnsignedIntegerOrEnumerationType())
@@ -5444,7 +5444,7 @@ ExprResult Sema::CheckTemplateArgument(N
     }

     Converted = TemplateArgument(Context, Value,
-                                 ParamType->isEnumeralType()
+                                 ParamType->isEnumeralType()
                                    ? Context.getCanonicalType(ParamType)
                                    : IntegerType);
     return Arg;
@@ -5556,17 +5556,17 @@ ExprResult Sema::CheckTemplateArgument(N
       Converted = TemplateArgument(Arg);
       return Arg;
     }
-
+
     switch (isNullPointerValueTemplateArgument(*this, Param, ParamType,
Arg)) {
     case NPV_NotNullPointer:
       Diag(Arg->getExprLoc(), diag::err_template_arg_not_convertible)
         << Arg->getType() << ParamType;
       Diag(Param->getLocation(), diag::note_template_param_here);
       return ExprError();
-
+
     case NPV_Error:
       return ExprError();
-
+
     case NPV_NullPointer:
       Diag(Arg->getExprLoc(), diag::warn_cxx98_compat_template_arg_null);
       Converted = TemplateArgument(Context.getCanonicalType(ParamType),
@@ -5813,7 +5813,7 @@ Sema::BuildExpressionFromIntegralTemplat
                                Context.getTrivialTypeSourceInfo(OrigT,
Loc),
                                Loc, Loc);
   }
-
+
   return E;
 }

@@ -6194,7 +6194,7 @@ static bool CheckTemplateSpecializationS
       << Specialized;
     return true;
   }
-
+
   // C++ [temp.class.spec]p6:
   //   A class template partial specialization may be declared or
redeclared
   //   in any namespace scope in which its definition may be defined
(14.5.1
@@ -6384,7 +6384,7 @@ static bool CheckNonTypeTemplatePartialS
 /// partial specialization according to C++ [temp.class.spec]p9.
 ///
 /// \param TemplateNameLoc the location of the template name.
-/// \param TemplateParams the template parameters of the primary class
+/// \param PrimaryTemplate the template parameters of the primary class
 ///        template.
 /// \param NumExplicit the number of explicitly-specified template
arguments.
 /// \param TemplateArgs the template arguments of the class template
@@ -6746,7 +6746,7 @@ Sema::ActOnClassTemplateSpecialization(S
     Diag(Specialization->getLocation(), diag::err_module_private_
specialization)
       << (isPartialSpecialization? 1 : 0)
       << FixItHint::CreateRemoval(ModulePrivateLoc);
-
+
   // Build the fully-sugared type for this class template
   // specialization as the user wrote in the specialization
   // itself. This means that we'll pretty-print the type retrieved
@@ -7226,7 +7226,7 @@ bool Sema::CheckFunctionTemplateSpeciali
                                          SpecInfo->
getPointOfInstantiation(),
                                              HasNoEffect))
     return true;
-
+
   // Mark the prior declaration as an explicit specialization, so that
later
   // clients know that this is an explicit specialization.
   if (!isFriend) {
@@ -8015,18 +8015,18 @@ DeclResult Sema::ActOnExplicitInstantiat
     return true;

   // C++ [dcl.stc]p1:
-  //   A storage-class-specifier shall not be specified in [...] an
explicit
+  //   A storage-class-specifier shall not be specified in [...] an
explicit
   //   instantiation (14.7.2) directive.
   if (D.getDeclSpec().getStorageClassSpec() == DeclSpec::SCS_typedef) {
     Diag(D.getIdentifierLoc(), diag::err_explicit_instantiation_of_typedef)
       << Name;
     return true;
-  } else if (D.getDeclSpec().getStorageClassSpec()
+  } else if (D.getDeclSpec().getStorageClassSpec()
                                                 !=
DeclSpec::SCS_unspecified) {
     // Complain about then remove the storage class specifier.
     Diag(D.getIdentifierLoc(), diag::err_explicit_
instantiation_storage_class)
       << FixItHint::CreateRemoval(D.getDeclSpec().
getStorageClassSpecLoc());
-
+
     D.getMutableDeclSpec().ClearStorageClassSpecs();
   }

@@ -8408,7 +8408,7 @@ Sema::ActOnDependentTag(Scope *S, unsign
   // Create the resulting type.
   ElaboratedTypeKeyword Kwd = TypeWithKeyword::
getKeywordForTagTypeKind(Kind);
   QualType Result = Context.getDependentNameType(Kwd, NNS, Name);
-
+
   // Create type-source location information for this type.
   TypeLocBuilder TLB;
   DependentNameTypeLoc TL = TLB.push<DependentNameTypeLoc>(Result);
@@ -8424,7 +8424,7 @@ Sema::ActOnTypenameType(Scope *S, Source
                         SourceLocation IdLoc) {
   if (SS.isInvalid())
     return true;
-
+
   if (TypenameLoc.isValid() && S && !S->getTemplateParamParent())
     Diag(TypenameLoc,
          getLangOpts().CPlusPlus11 ?
@@ -8470,11 +8470,11 @@ Sema::ActOnTypenameType(Scope *S,
            diag::warn_cxx98_compat_typename_outside_of_template :
            diag::ext_typename_outside_of_template)
       << FixItHint::CreateRemoval(TypenameLoc);
-
+
   // Translate the parser's template argument list in our AST format.
   TemplateArgumentListInfo TemplateArgs(LAngleLoc, RAngleLoc);
   translateTemplateArguments(TemplateArgsIn, TemplateArgs);
-
+
   TemplateName Template = TemplateIn.get();
   if (DependentTemplateName *DTN = Template.getAsDependentTemplateName()) {
     // Construct a dependent template specialization type.
@@ -8484,10 +8484,10 @@ Sema::ActOnTypenameType(Scope *S,

 DTN->getQualifier(),

 DTN->getIdentifier(),

 TemplateArgs);
-
+
     // Create source-location information for this type.
     TypeLocBuilder Builder;
-    DependentTemplateSpecializationTypeLoc SpecTL
+    DependentTemplateSpecializationTypeLoc SpecTL
     = Builder.push<DependentTemplateSpecializationTypeLoc>(T);
     SpecTL.setElaboratedKeywordLoc(TypenameLoc);
     SpecTL.setQualifierLoc(SS.getWithLocInContext(Context));
@@ -8499,11 +8499,11 @@ Sema::ActOnTypenameType(Scope *S,
       SpecTL.setArgLocInfo(I, TemplateArgs[I].getLocInfo());
     return CreateParsedType(T, Builder.getTypeSourceInfo(Context, T));
   }
-
+
   QualType T = CheckTemplateIdType(Template, TemplateNameLoc,
TemplateArgs);
   if (T.isNull())
     return true;
-
+
   // Provide source-location information for the template specialization
type.
   TypeLocBuilder Builder;
   TemplateSpecializationTypeLoc SpecTL
@@ -8514,12 +8514,12 @@ Sema::ActOnTypenameType(Scope *S,
   SpecTL.setRAngleLoc(RAngleLoc);
   for (unsigned I = 0, N = TemplateArgs.size(); I != N; ++I)
     SpecTL.setArgLocInfo(I, TemplateArgs[I].getLocInfo());
-
+
   T = Context.getElaboratedType(ETK_Typename, SS.getScopeRep(), T);
   ElaboratedTypeLoc TL = Builder.push<ElaboratedTypeLoc>(T);
   TL.setElaboratedKeywordLoc(TypenameLoc);
   TL.setQualifierLoc(SS.getWithLocInContext(Context));
-
+
   TypeSourceInfo *TSI = Builder.getTypeSourceInfo(Context, T);
   return CreateParsedType(T, TSI);
 }
@@ -8564,9 +8564,9 @@ static bool isEnableIf(NestedNameSpecifi
 /// \brief Build the type that describes a C++ typename specifier,
 /// e.g., "typename T::type".
 QualType
-Sema::CheckTypenameType(ElaboratedTypeKeyword Keyword,
+Sema::CheckTypenameType(ElaboratedTypeKeyword Keyword,
                         SourceLocation KeywordLoc,
-                        NestedNameSpecifierLoc QualifierLoc,
+                        NestedNameSpecifierLoc QualifierLoc,
                         const IdentifierInfo &II,
                         SourceLocation IILoc) {
   CXXScopeSpec SS;
@@ -8577,8 +8577,8 @@ Sema::CheckTypenameType(ElaboratedTypeKe
     // If the nested-name-specifier is dependent and couldn't be
     // resolved to a type, build a typename type.
     assert(QualifierLoc.getNestedNameSpecifier()->isDependent());
-    return Context.getDependentNameType(Keyword,
-                                        QualifierLoc.
getNestedNameSpecifier(),
+    return Context.getDependentNameType(Keyword,
+                                        QualifierLoc.
getNestedNameSpecifier(),
                                         &II);
   }

@@ -8630,8 +8630,8 @@ Sema::CheckTypenameType(ElaboratedTypeKe

   case LookupResult::NotFoundInCurrentInstantiation:
     // Okay, it's a member of an unknown instantiation.
-    return Context.getDependentNameType(Keyword,
-                                        QualifierLoc.
getNestedNameSpecifier(),
+    return Context.getDependentNameType(Keyword,
+                                        QualifierLoc.
getNestedNameSpecifier(),
                                         &II);

   case LookupResult::Found:
@@ -8639,7 +8639,7 @@ Sema::CheckTypenameType(ElaboratedTypeKe
       // We found a type. Build an ElaboratedType, since the
       // typename-specifier was just sugar.
       MarkAnyDeclReferenced(Type->getLocation(), Type, /*OdrUse=*/false);
-      return Context.getElaboratedType(ETK_Typename,
+      return Context.getElaboratedType(ETK_Typename,
                                        QualifierLoc.
getNestedNameSpecifier(),
                                        Context.getTypeDeclType(Type));
     }
@@ -8706,7 +8706,7 @@ namespace {
       this->Loc = Loc;
       this->Entity = Entity;
     }
-
+
     ExprResult TransformLambdaExpr(LambdaExpr *E) {
       // Lambdas never need to be transformed.
       return E;
@@ -8757,15 +8757,15 @@ ExprResult Sema::RebuildExprInCurrentIns
 }

 bool Sema::RebuildNestedNameSpecifierInCurrentInstantiation(CXXScopeSpec
&SS) {
-  if (SS.isInvalid())
+  if (SS.isInvalid())
     return true;

   NestedNameSpecifierLoc QualifierLoc = SS.getWithLocInContext(Context);
   CurrentInstantiationRebuilder Rebuilder(*this, SS.getRange().getBegin(),
                                           DeclarationName());
-  NestedNameSpecifierLoc Rebuilt
+  NestedNameSpecifierLoc Rebuilt
     = Rebuilder.TransformNestedNameSpecifierLoc(QualifierLoc);
-  if (!Rebuilt)
+  if (!Rebuilt)
     return true;

   SS.Adopt(Rebuilt);
@@ -8778,36 +8778,36 @@ bool Sema::RebuildTemplateParamsInCurren
                                                TemplateParameterList
*Params) {
   for (unsigned I = 0, N = Params->size(); I != N; ++I) {
     Decl *Param = Params->getParam(I);
-
+
     // There is nothing to rebuild in a type parameter.
     if (isa<TemplateTypeParmDecl>(Param))
       continue;
-
+
     // Rebuild the template parameter list of a template template
parameter.
-    if (TemplateTemplateParmDecl *TTP
+    if (TemplateTemplateParmDecl *TTP
         = dyn_cast<TemplateTemplateParmDecl>(Param)) {
       if (RebuildTemplateParamsInCurrentInstantiation(
             TTP->getTemplateParameters()))
         return true;
-
+
       continue;
     }
-
+
     // Rebuild the type of a non-type template parameter.
     NonTypeTemplateParmDecl *NTTP = cast<NonTypeTemplateParmDecl>(Param);
-    TypeSourceInfo *NewTSI
-      = RebuildTypeInCurrentInstantiation(NTTP->getTypeSourceInfo(),
-                                          NTTP->getLocation(),
+    TypeSourceInfo *NewTSI
+      = RebuildTypeInCurrentInstantiation(NTTP->getTypeSourceInfo(),
+                                          NTTP->getLocation(),
                                           NTTP->getDeclName());
     if (!NewTSI)
       return true;
-
+
     if (NewTSI != NTTP->getTypeSourceInfo()) {
       NTTP->setTypeSourceInfo(NewTSI);
       NTTP->setType(NewTSI->getType());
     }
   }
-
+
   return false;
 }



_______________________________________________
cfe-commits mailing list
cfe-commits at lists.llvm.org
http://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.llvm.org/pipermail/cfe-commits/attachments/20161230/f7a13362/attachment-0001.html>


More information about the cfe-commits mailing list