maxjhandsome
/
llvm-project
forked from OSchip/llvm-project
1
0
Fork 0
Code Issues Pull Requests Packages Releases Wiki Activity

After a conversation with Doug. I added a fix me to 

where we build the constructor's initializer list.

llvm-svn: 80735
This commit is contained in:
Fariborz Jahanian 2009-09-01 23:08:16 +00:00
parent efe96918f8
commit 5083e2623b
1 changed files with 13 additions and 0 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

13
clang/lib/AST/DeclCXX.cpp
View File

@ -609,6 +609,19 @@ CXXConstructorDecl::setBaseOrMemberInitializers(
} }
if (HasDependentBaseInit) { if (HasDependentBaseInit) {
// FIXME. This does not preserve the ordering of the initializers.
// Try (with -Wreorder)
// template<class X> struct A {};
// template<class X> struct B : A<X> {
// B() : x1(10), A<X>() {}
// int x1;
// };
// B<int> x;
// On seeing one dependent type, we should essentially exit this routine
// while preserving user-declared initializer list. When this routine is
// called during instantiatiation process, this routine will rebuild the
// oderdered initializer list correctly.
// If we have a dependent base initialization, we can't determine the // If we have a dependent base initialization, we can't determine the
// association between initializers and bases; just dump the known // association between initializers and bases; just dump the known
// initializers into the list, and don't try to deal with other bases. // initializers into the list, and don't try to deal with other bases.