Re: Updated no_reorder patchkit

2014-09-24 Thread Jakub Jelinek
On Wed, Sep 24, 2014 at 04:16:44PM +0200, Martin Liška wrote: > On 09/16/2014 05:15 AM, Andi Kleen wrote: > >This version addresses earlier comments and has an updated testsuite > >(still no LTO tests however). The assembler statements also > >no stay in order with ordered statements. > > > >It doe

Re: Updated no_reorder patchkit

2014-09-24 Thread Martin Liška
On 09/24/2014 04:17 PM, Jan Hubicka wrote: There's missing DECL_ATTRIBUTES in varpool.c in lookup_attribute call. Ready for trunk? OK, thanks Honza Ah, it has been fixed in r215552. Martin Martin diff --git a/gcc/varpool.c b/gcc/varpool.c index 8001c93..3761f14 100644 --- a/gcc/varpool

Re: Updated no_reorder patchkit

2014-09-24 Thread Jan Hubicka
> There's missing DECL_ATTRIBUTES in varpool.c in lookup_attribute call. > > Ready for trunk? OK, thanks Honza > > Martin > diff --git a/gcc/varpool.c b/gcc/varpool.c > index 8001c93..3761f14 100644 > --- a/gcc/varpool.c > +++ b/gcc/varpool.c > @@ -449,7 +449,7 @@ varpool_node::add (tree decl) >

Re: Updated no_reorder patchkit

2014-09-24 Thread Martin Liška
On 09/16/2014 05:15 AM, Andi Kleen wrote: This version addresses earlier comments and has an updated testsuite (still no LTO tests however). The assembler statements also no stay in order with ordered statements. It doesn't disable sorting of paritions with ordered symbols. I think that's an exi

Re: Updated no_reorder patchkit

2014-09-22 Thread Jan Hubicka
> This version addresses earlier comments and has an updated testsuite > (still no LTO tests however). The assembler statements also > no stay in order with ordered statements. > > It doesn't disable sorting of paritions with ordered symbols. I think > that's an existing bug and is best addressed

Updated no_reorder patchkit

2014-09-15 Thread Andi Kleen
This version addresses earlier comments and has an updated testsuite (still no LTO tests however). The assembler statements also no stay in order with ordered statements. It doesn't disable sorting of paritions with ordered symbols. I think that's an existing bug and is best addressed separately.