martong added inline comments.
================ Comment at: lib/AST/ASTImporter.cpp:1962 TagDecl *Definition = D->getDefinition(); - if (Definition && Definition != D) { + if (!D->isImplicit() && Definition && Definition != D) { Decl *ImportedDef = Importer.Import(Definition); ---------------- a.sidorin wrote: > We are changing import if RecordDecl. Is it possible to add a test that > doesn't require templates? > I tried and found that the implicit CXXRecordDecl of > ClassTemplateSpecializationDecl is its redeclaration. That's not true for > normal CXXRecordDecls, as I see, so this deserves a comment. Yes, I've added a new test which exercises only a RecordDecl. Also, added a comment about the specialization re-declaration. Repository: rC Clang https://reviews.llvm.org/D47057 _______________________________________________ cfe-commits mailing list cfe-commits@lists.llvm.org http://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits