Re: [patch] Remove fold-const.h as a dependency for gimple.h

2015-06-26 Thread Jeff Law
On 06/26/2015 11:59 AM, Andrew MacLeod wrote: On 06/26/2015 05:40 AM, Richard Biener wrote: On Thu, Jun 25, 2015 at 10:57 PM, Jeff Law wrote: On 06/25/2015 02:31 PM, Andrew MacLeod wrote: I proposed including fold-const.h into gimple.h, but didn't notice that its actually only needed by one i

Re: [patch] Remove fold-const.h as a dependency for gimple.h

2015-06-26 Thread Andrew MacLeod
On 06/26/2015 05:40 AM, Richard Biener wrote: On Thu, Jun 25, 2015 at 10:57 PM, Jeff Law wrote: On 06/25/2015 02:31 PM, Andrew MacLeod wrote: I proposed including fold-const.h into gimple.h, but didn't notice that its actually only needed by one inline function. Moving that function to gimple.

Re: [patch] Remove fold-const.h as a dependency for gimple.h

2015-06-26 Thread Richard Biener
On Thu, Jun 25, 2015 at 10:57 PM, Jeff Law wrote: > On 06/25/2015 02:31 PM, Andrew MacLeod wrote: >> >> I proposed including fold-const.h into gimple.h, but didn't notice that >> its actually only needed by one inline function. Moving that function to >> gimple.c means its no longer required. >> >

Re: [patch] Remove fold-const.h as a dependency for gimple.h

2015-06-25 Thread Jeff Law
On 06/25/2015 02:31 PM, Andrew MacLeod wrote: I proposed including fold-const.h into gimple.h, but didn't notice that its actually only needed by one inline function. Moving that function to gimple.c means its no longer required. 160 places need cgraph.h, so by removing this dependency, the numb

[patch] Remove fold-const.h as a dependency for gimple.h

2015-06-25 Thread Andrew MacLeod
I proposed including fold-const.h into gimple.h, but didn't notice that its actually only needed by one inline function. Moving that function to gimple.c means its no longer required. 160 places need cgraph.h, so by removing this dependency, the number of those files which still need fold-cons