Re: build_function_call and TREE_ADDRESSABLE

2011-03-04 Thread Richard Guenther
On Fri, Mar 4, 2011 at 12:29 AM, Alan Modra wrote: > On Thu, Mar 03, 2011 at 04:43:15PM +0100, Richard Guenther wrote: >> On Thu, Mar 3, 2011 at 4:42 PM, Richard Guenther >> wrote: >> > On Thu, Mar 3, 2011 at 3:33 PM, Alan Modra wrote: >> >> TREE_ADDRESSABLE comment says "In a FUNCTION_DECL, non

Re: build_function_call and TREE_ADDRESSABLE

2011-03-04 Thread Richard Guenther
On Fri, Mar 4, 2011 at 5:58 AM, Alan Modra wrote: > Warn off anyone trying to do as I did, and the bit about inlines is > no longer relevant.  OK to apply? > >        * tree.h (TREE_ADDRESSABLE): Note that direct calls set the >        flag on FUNCTION_DECLs. > > Index: gcc/tree.h > ==

Re: build_function_call and TREE_ADDRESSABLE

2011-03-03 Thread Alan Modra
Warn off anyone trying to do as I did, and the bit about inlines is no longer relevant. OK to apply? * tree.h (TREE_ADDRESSABLE): Note that direct calls set the flag on FUNCTION_DECLs. Index: gcc/tree.h === --- gcc/t

Re: build_function_call and TREE_ADDRESSABLE

2011-03-03 Thread Alan Modra
On Thu, Mar 03, 2011 at 04:43:15PM +0100, Richard Guenther wrote: > On Thu, Mar 3, 2011 at 4:42 PM, Richard Guenther > wrote: > > On Thu, Mar 3, 2011 at 3:33 PM, Alan Modra wrote: > >> TREE_ADDRESSABLE comment says "In a FUNCTION_DECL, nonzero means its > >> address is needed".  However, as I poi

Re: build_function_call and TREE_ADDRESSABLE

2011-03-03 Thread Richard Guenther
On Thu, Mar 3, 2011 at 4:42 PM, Richard Guenther wrote: > On Thu, Mar 3, 2011 at 3:33 PM, Alan Modra wrote: >> TREE_ADDRESSABLE comment says "In a FUNCTION_DECL, nonzero means its >> address is needed".  However, as I point out in >> http://gcc.gnu.org/ml/gcc-patches/2011-02/msg01525.html, this f

Re: build_function_call and TREE_ADDRESSABLE

2011-03-03 Thread Richard Guenther
On Thu, Mar 3, 2011 at 3:33 PM, Alan Modra wrote: > TREE_ADDRESSABLE comment says "In a FUNCTION_DECL, nonzero means its > address is needed".  However, as I point out in > http://gcc.gnu.org/ml/gcc-patches/2011-02/msg01525.html, this flag > gets set when making normal calls.  It wasn't always lik