This revision was automatically updated to reflect the committed changes.
Closed by commit rL332588: [ASTImporter] Fix missing implict CXXRecordDecl
(authored by martong, committed by ).
Herald added a subscriber: llvm-commits.
Repository:
rL LLVM
https://reviews.llvm.org/D46958
Files:
cfe/
martong added a comment.
In https://reviews.llvm.org/D46958#1101570, @a.sidorin wrote:
> So, we fail to add injected name to a CXXRecordDecl that has a described
> class template?
Yes, we failed to import the implicit `CXXRecordDecl`.
Here is how it looked before the fix:
From:
ClassTempl
a.sidorin accepted this revision.
a.sidorin added a comment.
This revision is now accepted and ready to land.
So, we fail to add injected name to a CXXRecordDecl that has a described class
template? Nice catch! LGTM.
Repository:
rC Clang
https://reviews.llvm.org/D46958
___
martong created this revision.
martong added reviewers: xazax.hun, a.sidorin, szepet.
Herald added subscribers: cfe-commits, dkrupp, rnkovacs.
Implicit CXXRecordDecl is not added to its DeclContext during import, but in
the original AST it is. This patch fixes this.
Repository:
rC Clang
https