Re: [PATCH] D15366: [Clang] Use autos in lib/AST/ASTImporter.cpp

2015-12-10 Thread Aaron Ballman via cfe-commits
On Thu, Dec 10, 2015 at 2:11 PM, Eugene Zelenko wrote: > Eugene.Zelenko added inline comments. > > > Comment at: lib/AST/ASTImporter.cpp:5855 > @@ -5888,3 +5854,3 @@ >} > - else if (ObjCProtocolDecl *PD = dyn_cast(D)) { > + else if (auto *PD = dyn_cast(D)) { > if (!PD->

Re: [PATCH] D15366: [Clang] Use autos in lib/AST/ASTImporter.cpp

2015-12-10 Thread Eugene Zelenko via cfe-commits
Eugene.Zelenko added inline comments. Comment at: lib/AST/ASTImporter.cpp:5855 @@ -5888,3 +5854,3 @@ } - else if (ObjCProtocolDecl *PD = dyn_cast(D)) { + else if (auto *PD = dyn_cast(D)) { if (!PD->getDefinition()) aaron.ballman wrote: > Can you fix the

Re: [PATCH] D15366: [Clang] Use autos in lib/AST/ASTImporter.cpp

2015-12-10 Thread Aaron Ballman via cfe-commits
aaron.ballman added inline comments. Comment at: lib/AST/ASTImporter.cpp:1676 @@ -1690,3 +1675,3 @@ QualType ASTNodeImporter::VisitTypedefType(const TypedefType *T) { - TypedefNameDecl *ToDecl + const auto *ToDecl = dyn_cast_or_null(Importer.Import(T->getDecl()))