Re: [PATCH][2/2] Fix expansion slowness of PR60291

2014-02-24 Thread Richard Biener
On Fri, 21 Feb 2014, Richard Biener wrote: > On Fri, 21 Feb 2014, Richard Biener wrote: > > > On Fri, 21 Feb 2014, Richard Biener wrote: > > > > > On Fri, 21 Feb 2014, Richard Sandiford wrote: > > > > > > > In a thread a few years ago you talked about the possibility of going > > > > further an

Re: [PATCH][2/2] Fix expansion slowness of PR60291

2014-02-21 Thread Richard Biener
On Fri, 21 Feb 2014, Richard Biener wrote: > On Fri, 21 Feb 2014, Richard Biener wrote: > > > On Fri, 21 Feb 2014, Richard Sandiford wrote: > > > > > In a thread a few years ago you talked about the possibility of going > > > further and folding the attributes into the MEM itself, so avoiding >

Re: [PATCH][2/2] Fix expansion slowness of PR60291

2014-02-21 Thread Richard Biener
On Fri, 21 Feb 2014, Richard Biener wrote: > On Fri, 21 Feb 2014, Richard Sandiford wrote: > > > Richard Biener writes: > > > On Fri, 21 Feb 2014, Richard Biener wrote: > > > > > >> On Fri, 21 Feb 2014, Richard Biener wrote: > > >> > > >> > On Fri, 21 Feb 2014, Richard Biener wrote: > > >> > >

Re: [PATCH][2/2] Fix expansion slowness of PR60291

2014-02-21 Thread Richard Biener
On Fri, 21 Feb 2014, Richard Sandiford wrote: > Richard Biener writes: > > On Fri, 21 Feb 2014, Richard Biener wrote: > > > >> On Fri, 21 Feb 2014, Richard Biener wrote: > >> > >> > On Fri, 21 Feb 2014, Richard Biener wrote: > >> > > >> > > > >> > > This fixes the slowness of RTL expansion in

Re: [PATCH][2/2] Fix expansion slowness of PR60291

2014-02-21 Thread Richard Biener
On Fri, 21 Feb 2014, Richard Biener wrote: > On Fri, 21 Feb 2014, Richard Biener wrote: > > > Last statistics: http://gcc.gnu.org/ml/gcc-patches/2011-08/msg01784.html > > With the patch below to get some statistics we see that one important > piece of sharing not covered by above measurements is

Re: [PATCH][2/2] Fix expansion slowness of PR60291

2014-02-21 Thread Richard Sandiford
Richard Biener writes: > On Fri, 21 Feb 2014, Richard Biener wrote: > >> On Fri, 21 Feb 2014, Richard Biener wrote: >> >> > On Fri, 21 Feb 2014, Richard Biener wrote: >> > >> > > >> > > This fixes the slowness of RTL expansion in PR60291 which is caused >> > > by excessive collisions in mem-att

Re: [PATCH][2/2] Fix expansion slowness of PR60291

2014-02-21 Thread Richard Biener
On Fri, 21 Feb 2014, Richard Biener wrote: > On Fri, 21 Feb 2014, Richard Biener wrote: > > > On Fri, 21 Feb 2014, Richard Biener wrote: > > > > > > > > This fixes the slowness of RTL expansion in PR60291 which is caused > > > by excessive collisions in mem-attr sharing. The issue here is > >

Re: [PATCH][2/2] Fix expansion slowness of PR60291

2014-02-21 Thread Richard Biener
On Fri, 21 Feb 2014, Richard Biener wrote: > On Fri, 21 Feb 2014, Richard Biener wrote: > > > > > This fixes the slowness of RTL expansion in PR60291 which is caused > > by excessive collisions in mem-attr sharing. The issue here is > > that sharing attempts happens across functions and we have

Re: [PATCH][2/2] Fix expansion slowness of PR60291

2014-02-21 Thread Richard Biener
On Fri, 21 Feb 2014, Richard Biener wrote: > > This fixes the slowness of RTL expansion in PR60291 which is caused > by excessive collisions in mem-attr sharing. The issue here is > that sharing attempts happens across functions and we have a _lot_ > of functions in this testcase referencing the

[PATCH][2/2] Fix expansion slowness of PR60291

2014-02-21 Thread Richard Biener
This fixes the slowness of RTL expansion in PR60291 which is caused by excessive collisions in mem-attr sharing. The issue here is that sharing attempts happens across functions and we have a _lot_ of functions in this testcase referencing the same lexically equivalent memory, for example MEM[(St