rnk added a comment.

Is it possible to fix this in assignInheritanceModel instead? I'd imagine we'd 
get the most recent decl. If that's not the issue, maybe you're fixing the bug 
in the right spot, but we need to find out where other class template 
attributes are moved from instantiation to real declaration.



================
Comment at: lib/Sema/SemaTemplate.cpp:7079
 
+void Sema::CheckCXXRecordDecl(CXXRecordDecl *Record, CXXRecordDecl *Prev)
+{
----------------
This seems like an overly-generic name for what it does, which currently is 
specific to class template instantiations.


Repository:
  rC Clang

https://reviews.llvm.org/D46664



_______________________________________________
cfe-commits mailing list
cfe-commits@lists.llvm.org
http://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits

Reply via email to