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

Daniel Dunbar daniel at zuster.org
Mon Dec 1 13:01:48 PST 2008

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

More information about the cfe-commits mailing list