On Jul 22, 2009, at 5:42 PM, Fariborz Jahanian wrote: > Author: fjahanian > Date: Wed Jul 22 19:42:24 2009 > New Revision: 76826 > > URL: http://llvm.org/viewvc/llvm-project?rev=76826&view=rev > Log: > Add constructor used to initialize base/member in > CXXBaseOrMemberInitializer AST node. Needed by > its clients to do the initialization.
Looks good! One little typo mentioned below. > Modified: > cfe/trunk/include/clang/AST/DeclCXX.h > cfe/trunk/lib/AST/DeclCXX.cpp > cfe/trunk/lib/Sema/SemaDeclCXX.cpp > cfe/trunk/test/SemaCXX/constructor-initializer.cpp > > Modified: cfe/trunk/include/clang/AST/DeclCXX.h > URL: > http://llvm.org/viewvc/llvm-project/cfe/trunk/include/clang/AST/DeclCXX.h?rev=76826&r1=76825&r2=76826&view=diff > > = > = > = > = > = > = > = > = > ====================================================================== > --- cfe/trunk/include/clang/AST/DeclCXX.h (original) > +++ cfe/trunk/include/clang/AST/DeclCXX.h Wed Jul 22 19:42:24 2009 > @@ -736,6 +736,10 @@ > Expr **Args; > unsigned NumArgs; > > + /// CtorToCall - For a base or mamber needing a constructor for > their > + /// initialization, this is the constructor to call. > + CXXConstructorDecl *CtorToCall; Typo "mamber". - Doug _______________________________________________ cfe-commits mailing list [email protected] http://lists.cs.uiuc.edu/mailman/listinfo/cfe-commits
