Re: [PATCH] Propagate nonfreeing_call_p using ipa-pure-const (take 2)

2014-11-14 Thread Jan Hubicka
> On Thu, Nov 13, 2014 at 12:52:21PM +0100, Jakub Jelinek wrote: > > On Thu, Nov 13, 2014 at 09:39:42AM +0100, Jakub Jelinek wrote: > > > What about the: > > > > > I wonder if the nonfreeing_call_p function shouldn't be moved > > > > > elsewhere > > > > > though (suggestion where), so that gimple.

Re: [PATCH] Propagate nonfreeing_call_p using ipa-pure-const (take 2)

2014-11-14 Thread Jakub Jelinek
On Thu, Nov 13, 2014 at 12:52:21PM +0100, Jakub Jelinek wrote: > On Thu, Nov 13, 2014 at 09:39:42AM +0100, Jakub Jelinek wrote: > > What about the: > > > > I wonder if the nonfreeing_call_p function shouldn't be moved elsewhere > > > > though (suggestion where), so that gimple.c doesn't need the cg

Re: [PATCH] Propagate nonfreeing_call_p using ipa-pure-const (take 2)

2014-11-13 Thread Jakub Jelinek
On Thu, Nov 13, 2014 at 09:39:42AM +0100, Jakub Jelinek wrote: > What about the: > > > I wonder if the nonfreeing_call_p function shouldn't be moved elsewhere > > > though (suggestion where), so that gimple.c doesn't need the cgraph > > > includes. > question though (maybe it is more on Richard)?

Re: [PATCH] Propagate nonfreeing_call_p using ipa-pure-const (take 2)

2014-11-13 Thread Richard Biener
On Thu, Nov 13, 2014 at 9:39 AM, Jakub Jelinek wrote: > Hi! > > On Thu, Nov 13, 2014 at 12:11:08AM +0100, Jan Hubicka wrote: > >> Actually I think you want to do this for can_throw, too. >> We probably do not have throwing internal calls, but it is better to be safe. > > I'll leave that change to

[PATCH] Propagate nonfreeing_call_p using ipa-pure-const (take 2)

2014-11-13 Thread Jakub Jelinek
Hi! On Thu, Nov 13, 2014 at 12:11:08AM +0100, Jan Hubicka wrote: > Actually I think you want to do this for can_throw, too. > We probably do not have throwing internal calls, but it is better to be safe. I'll leave that change to you ;), as I said in my last mail, it isn't immediately clear to m