Re: [tuples] meaning of DECL_SAVED_TREE while analyzing cgraph

2007-07-31 Thread Diego Novillo
On 7/31/07 6:10 PM, Jan Hubicka wrote: > There are also few other occurences in cgraph.c and ipa-inline. Those > are used to test whether function body is present. When you have body > elsewhere now, you can check something else (such as > DECL_STRUCT_FUNCTION(cfun)->cfg) Currently, in tuples,

Re: [tuples] meaning of DECL_SAVED_TREE while analyzing cgraph

2007-07-31 Thread Jan Hubicka
> > The test there is sort of hack, I would just remove it at this stage and > > we can work out better fix for that testcase later. I hope that with my > > plans for declaration merging pass we can get round such weird side > > effects of in place declaration replacement. > > Will do. > > How a

Re: [tuples] meaning of DECL_SAVED_TREE while analyzing cgraph

2007-07-27 Thread Aldy Hernandez
> The test there is sort of hack, I would just remove it at this stage and > we can work out better fix for that testcase later. I hope that with my > plans for declaration merging pass we can get round such weird side > effects of in place declaration replacement. Will do. How about all the oth

Re: [tuples] meaning of DECL_SAVED_TREE while analyzing cgraph

2007-07-26 Thread Jan Hubicka
> Hi Jan. > > What do you expect DECL_SAVED_TREE to have in cgraph_analyze_functions: > > /* ??? It is possible to create extern inline function and later using >weak alias attribute to kill its body. See >gcc.c-torture/compile/2009-1.c */ > if (!DECL_SAVED_TREE (

[tuples] meaning of DECL_SAVED_TREE while analyzing cgraph

2007-07-26 Thread Aldy Hernandez
Hi Jan. What do you expect DECL_SAVED_TREE to have in cgraph_analyze_functions: /* ??? It is possible to create extern inline function and later using weak alias attribute to kill its body. See gcc.c-torture/compile/2009-1.c */ if (!DECL_SAVED_TREE (decl))