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

Reply via email to