[PATCH] D47450: [ASTImporter] Use InjectedClassNameType at import of templated record.

Balázs Kéri via Phabricator via cfe-commits cfe-commits at lists.llvm.org
Mon May 28 06:58:30 PDT 2018


balazske added inline comments.


================
Comment at: lib/AST/ASTImporter.cpp:2139
+          CXXRecordDecl *Injected = nullptr;
+          for (NamedDecl *Found : D2CXX->noload_lookup(Name)) {
+            auto *Record = dyn_cast<CXXRecordDecl>(Found);
----------------
r.stahl wrote:
> The only thing I'm wondering is whether the Decls looked up here are always known to be already imported.
These are in the 'To' context. It may be that the `Injected` is not found here, probably because not yet imported (in this case the import may be part of a not completed recursive process).


Repository:
  rC Clang

https://reviews.llvm.org/D47450





More information about the cfe-commits mailing list