r.stahl accepted this revision. r.stahl added a comment. LGTM!
I'm not really too confident to approve changes yet, but with a second opinion it should be fine. ================ Comment at: lib/AST/ASTImporter.cpp:2139 + CXXRecordDecl *Injected = nullptr; + for (NamedDecl *Found : D2CXX->noload_lookup(Name)) { + auto *Record = dyn_cast<CXXRecordDecl>(Found); ---------------- The only thing I'm wondering is whether the Decls looked up here are always known to be already imported. Repository: rC Clang https://reviews.llvm.org/D47450 _______________________________________________ cfe-commits mailing list cfe-commits@lists.llvm.org http://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits