Re: [patch 8/8] cfgloop.h includes basic-block.h

2013-10-18 Thread Andrew MacLeod
On 10/18/2013 12:06 PM, Jeff Law wrote: On 10/18/13 07:45, Andrew MacLeod wrote: bootstraps on x86_64-unknown-linux-gnu with no new regressions. OK? OK. jeff All 8 patches were committed at once, revision 203833. I'll submit another patch if any recommended followups surface. Andrew

Re: [patch 8/8] cfgloop.h includes basic-block.h

2013-10-18 Thread Jeff Law
On 10/18/13 07:45, Andrew MacLeod wrote: Another basoc-block inclusion. cfgloop.h was including basic-block.h, which meant lots of other things were getting it from here too. The only routine in cfgloop.h which uses anything from basic-block.h was bb_loop_depth(). By moving that to cfgloop.c,

[patch 8/8] cfgloop.h includes basic-block.h

2013-10-18 Thread Andrew MacLeod
Another basoc-block inclusion. cfgloop.h was including basic-block.h, which meant lots of other things were getting it from here too. The only routine in cfgloop.h which uses anything from basic-block.h was bb_loop_depth(). By moving that to cfgloop.c, its no longer required by the .h file.