That patch I just approved

Mark Mitchell mark@codesourcery.com
Fri Mar 21 17:36:00 GMT 2003


On Fri, 2003-03-21 at 08:09, Gabriel Dos Reis wrote:
> Mark Mitchell <mark@codesourcery.com> writes:
> 
> | Nathan --
> | 
> | Actually, I'd feel much better if make_node didn't set TREE_CONSTANT for
> | a CONST_DECL.  It makes sense -- but it still might cause problems. 
> 
> Mark, would mind elaborating on why setting TREE_CNSTANT would be
> problematic?   Maybe TREE_UNCHANGING?

In theory, it should be fine.

In pratice, I have no idea what all the front ends do, and what the
constant folders do, and so forth.  DECL_INITIAL for a CONST_DECL is not
always a constant -- sometimes it is an expression that will become a
constant later.

I think messing with this at this point is asking for trouble.  If we
want to make that change, we should do it later, as a separate patch.

-- 
Mark Mitchell <mark@codesourcery.com>
CodeSourcery, LLC



More information about the Gcc-patches mailing list