a_sidorin accepted this revision. a_sidorin added a comment. This revision is now accepted and ready to land.
Hi Balasz, I cannot find any problems with the latest changes. I suggest you to commit the patch to make further reviews easier. ================ Comment at: lib/AST/ASTImporter.cpp:8197 -void ASTImporter::ImportDefinition(Decl *From) { +Error ASTImporter::ImportDefinition_New(Decl *From) { Decl *To = Import(From); ---------------- ImportDefinitionOrError? Repository: rC Clang https://reviews.llvm.org/D51633 _______________________________________________ cfe-commits mailing list cfe-commits@lists.llvm.org http://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits