[cfe-dev] [PATCH] add constant expression evaluation to the AST and fix PR2413
clattner at apple.com
Sat Jun 21 16:14:12 PDT 2008
On Jun 9, 2008, at 10:46 PM, Eli Friedman wrote:
> On Sun, Jun 8, 2008 at 7:05 PM, Eli Friedman
> <eli.friedman at gmail.com> wrote:
>> Attached patch adds some capability to the AST to evaluate constant
>> expressions more general than integer constant expressions. It then
>> uses that capability to remove the hack added for
>> Sema/darwin-align-cast.c, which fixes PR2413.
> This is an updated version with a few minor fixes.
I think this is a great idea, and a huge improvement over
In addition to the discriminated union, it would also be nice to
change this to use a StmtVisitor if possible, just to simplify the
code and make it more modular. It would be natural to move the
implementation to its own file if it is large, even though the
entrypoint would stay a method on Expr.
More information about the cfe-dev