[cfe-commits] r60298 - in /cfe/trunk: include/clang/AST/Expr.h lib/AST/ExprConstant.cpp

[Daniel Dunbar]

On Sun, Nov 30, 2008 at 8:58 AM, Anders Carlsson <andersca at mac.com> wrote:
> +  bool Evaluate(EvalResult &Result, ASTContext &Ctx) const;
> +
>   // FIXME: We should come up with a better API for the isEvaluated case.
>   bool Evaluate(APValue& Result, ASTContext &Ctx, bool *isEvaluated = 0) const;

Could we just kill off the second variant of Evaluate? I think this
API is more confusing than useful, and I don't believe the second
variant is really used that many places?

 - Daniel