Re: [PATCH 3/4] bb-reorder: Add -freorder-blocks-algorithm= and wire it up

2015-11-02 Thread Alan Lawrence
On 02/11/15 14:38, Alan Lawrence wrote: > I'm a bit puzzled as to why nobody else has been seeing this, as it's been happening to me as part of building gcc on x86_64, but since this patch I've been seeing an ICE in vec::operator[] in reorder_basic_blocks_simple, building libitm/beginend.cc.

Re: [PATCH 3/4] bb-reorder: Add -freorder-blocks-algorithm= and wire it up

2015-11-02 Thread Segher Boessenkool
On Mon, Nov 02, 2015 at 02:38:33PM +, Alan Lawrence wrote: > On 23/09/15 23:06, Segher Boessenkool wrote: > >This adds an -freorder-blocks-algorithm=[simple|stc] flag, with "simple" > >as default. For -O2 and up (except -Os) it is switched to "stc" instead. > >Targets that never want STC can

Re: [PATCH 3/4] bb-reorder: Add -freorder-blocks-algorithm= and wire it up

2015-09-24 Thread Bernd Schmidt
On 09/24/2015 12:06 AM, Segher Boessenkool wrote: This adds an -freorder-blocks-algorithm=[simple|stc] flag, with "simple" as default. For -O2 and up (except -Os) it is switched to "stc" instead. Targets that never want STC can override this. This changes -freorder-blocks to be on at -O1 and

Re: [PATCH 3/4] bb-reorder: Add -freorder-blocks-algorithm= and wire it up

2015-09-24 Thread Segher Boessenkool
On Thu, Sep 24, 2015 at 12:28:08PM +0200, Bernd Schmidt wrote: > On 09/24/2015 12:06 AM, Segher Boessenkool wrote: > >This adds an -freorder-blocks-algorithm=[simple|stc] flag, with "simple" > >as default. For -O2 and up (except -Os) it is switched to "stc" instead. > >Targets that never want STC

Re: [PATCH 3/4] bb-reorder: Add -freorder-blocks-algorithm= and wire it up

2015-09-24 Thread Andi Kleen
Segher Boessenkool writes: > > In effect, the changes are for -O1 (which now gets "simple" instead of > nothing), -Os (which now gets "simple" instead of "stc", since STC results > in much bigger code), and for targets that wish to never use STC (not in > this patch

Re: [PATCH 3/4] bb-reorder: Add -freorder-blocks-algorithm= and wire it up

2015-09-24 Thread Segher Boessenkool
On Thu, Sep 24, 2015 at 08:12:55AM -0700, Andi Kleen wrote: > Segher Boessenkool writes: > > > > In effect, the changes are for -O1 (which now gets "simple" instead of > > nothing), -Os (which now gets "simple" instead of "stc", since STC results > > in much bigger

[PATCH 3/4] bb-reorder: Add -freorder-blocks-algorithm= and wire it up

2015-09-23 Thread Segher Boessenkool
This adds an -freorder-blocks-algorithm=[simple|stc] flag, with "simple" as default. For -O2 and up (except -Os) it is switched to "stc" instead. Targets that never want STC can override this. This changes -freorder-blocks to be on at -O1 and up (was -O2 and up). In effect, the changes are for