[PATCH] D47450: [ASTImporter] Use InjectedClassNameType at import of templated record.
Rafael Stahl via Phabricator via cfe-commits
cfe-commits at lists.llvm.org
Mon May 28 06:30:21 PDT 2018
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
More information about the cfe-commits
mailing list