RE: BB reorder forced off for -Os

2010-03-30 Thread Ian Bolton
> We're not able to enable BB reordering with -Os. The behaviour is > hard-coded via this if statement in rest_of_handle_reorder_blocks(): > > if ((flag_reorder_blocks || flag_reorder_blocks_and_partition) > /* Don't reorder blocks when optimizing for size because extra > jump insns may >

Re: BB reorder forced off for -Os

2010-03-24 Thread Dave Hudson
On 23 Mar 2010, at 22:30, Steven Bosscher wrote: > On Tue, Mar 23, 2010 at 7:05 PM, Ian Bolton wrote: >> Is there any reason why BB reorder has been disabled >> in bb-reorder.c for -Os, such that you can't even >> turn it on with -freorder-blocks? > > No, you should have the option to turn it on

Re: BB reorder forced off for -Os

2010-03-23 Thread Steven Bosscher
On Tue, Mar 23, 2010 at 7:05 PM, Ian Bolton wrote: > Is there any reason why BB reorder has been disabled > in bb-reorder.c for -Os, such that you can't even > turn it on with -freorder-blocks? No, you should have the option to turn it on if you wish to do so. If that is not possible, I consider

Re: BB reorder forced off for -Os

2010-03-23 Thread Joe Buck
> From: Ian Bolton [mailto:bol...@icerasemi.com] > > Is there any reason why BB reorder has been disabled > > in bb-reorder.c for -Os, such that you can't even > > turn it on with -freorder-blocks? On Tue, Mar 23, 2010 at 12:21:05PM -0700, Paul Koning wrote: > Does -Os mean "optimize even if it ma

RE: BB reorder forced off for -Os

2010-03-23 Thread Paul Koning
Does -Os mean "optimize even if it makes things a bit bigger" or does it mean "optimize only to make it smaller"? If the latter then the current behavior would appear to be the correct one. paul > -Original Message- > From: Ian Bolton [mailto:bol...@icerasemi.com] > Sent: Tuesday