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

Anders Carlsson andersca at mac.com
Mon Dec 1 21:24:48 PST 2008


1 dec 2008 kl. 13.01 skrev 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?

Yeah, that's on my todo list.

Anders
-------------- next part --------------
A non-text attachment was scrubbed...
Name: smime.p7s
Type: application/pkcs7-signature
Size: 2415 bytes
Desc: not available
URL: <http://lists.llvm.org/pipermail/cfe-commits/attachments/20081201/cc93ebe3/attachment.bin>


More information about the cfe-commits mailing list