[cfe-commits] r63983 - in /cfe/trunk: include/clang/AST/Expr.h include/clang/AST/StmtVisitor.h include/clang/Basic/DiagnosticSemaKinds.def lib/AST/Expr.cpp lib/Parse/ParseExpr.cpp lib/Sema/Sema.h lib/Sema/SemaExpr.cpp lib/Sema/SemaInherit.h lib/Sema/SemaLookup.cpp test/SemaCXX/member-pointer.cpp www/cxx_status.html

[Chris Lattner]

On Feb 6, 2009, at 4:15 PM, Sebastian Redl wrote:

> +++ cfe/trunk/lib/Sema/SemaExpr.cpp Fri Feb  6 18:15:38 2009
> +
> +inline QualType Sema::CheckPointerToMemberOperands(
> +  Expr *&lex, Expr *&rex, SourceLocation Loc, bool isIndirect)
> +{

Should this go in SemaExprCXX?

> +++ cfe/trunk/lib/Sema/SemaInherit.h Fri Feb  6 18:15:38 2009
> @@ -16,6 +16,7 @@
> #ifndef LLVM_CLANG_SEMA_INHERIT_H
> #define LLVM_CLANG_SEMA_INHERIT_H
>
> +#include "Sema.h"
> #include "clang/AST/DeclarationName.h"
> #include "clang/AST/DeclBase.h"
> #include "clang/AST/Type.h"
> @@ -25,7 +26,6 @@
> #include <map>
>
> namespace clang {
> -  class Sema;

Why is this needed?

Thanks for working on this Sebastian!

-Chris