Re: [reviewed] qsort comparator consistency checking

2017-09-29 Thread Steve Ellcey
On Fri, 2017-09-29 at 21:43 +0200, Christophe Lyon wrote: > On 29 September 2017 at 21:39, Steve Ellcey > wrote: > > > > On Fri, 2017-09-29 at 21:14 +0300, Alexander Monakov wrote: > > > > > > On Fri, 29 Sep 2017, Andrew Pinski wrote: > > > > > > > > > > > > > > > > > >

Re: [reviewed] qsort comparator consistency checking

2017-09-29 Thread Christophe Lyon
On 29 September 2017 at 21:39, Steve Ellcey wrote: > On Fri, 2017-09-29 at 21:14 +0300, Alexander Monakov wrote: >> On Fri, 29 Sep 2017, Andrew Pinski wrote: >> > >> > > >> > > This patch (r253295) breaks the gcc build for aarch64-linux-gnu: >> > I was just about to report the

Re: [reviewed] qsort comparator consistency checking

2017-09-29 Thread Steve Ellcey
On Fri, 2017-09-29 at 21:14 +0300, Alexander Monakov wrote: > On Fri, 29 Sep 2017, Andrew Pinski wrote: > > > > > > > > This patch (r253295) breaks the gcc build for aarch64-linux-gnu: > > I was just about to report the same thing. > I think autoprefetch ranking heuristic is still wrong if >

Re: [reviewed] qsort comparator consistency checking

2017-09-29 Thread Alexander Monakov
On Fri, 29 Sep 2017, Andrew Pinski wrote: > > This patch (r253295) breaks the gcc build for aarch64-linux-gnu: > > I was just about to report the same thing. I think autoprefetch ranking heuristic is still wrong if multi_mem_insn_p may be true; please try this patch. * haifa-sched.c

Re: [reviewed] qsort comparator consistency checking

2017-09-29 Thread Andrew Pinski
On Fri, Sep 29, 2017 at 10:46 AM, Christophe Lyon wrote: > Hi, > > > On 29 September 2017 at 15:29, Alexander Monakov wrote: >> Hello, >> >> I'm going to install the following patch on trunk in the next few hours. >> This revision doesn't offer

Re: [reviewed] qsort comparator consistency checking

2017-09-29 Thread Christophe Lyon
Hi, On 29 September 2017 at 15:29, Alexander Monakov wrote: > Hello, > > I'm going to install the following patch on trunk in the next few hours. > This revision doesn't offer per-callsite opt-out anymore as suggested by > Richi on the Cauldron (made possible by fixing all

[reviewed] qsort comparator consistency checking

2017-09-29 Thread Alexander Monakov
Hello, I'm going to install the following patch on trunk in the next few hours. This revision doesn't offer per-callsite opt-out anymore as suggested by Richi on the Cauldron (made possible by fixing all known issues on trunk). Thus this patch has a few minor differences compared to the previous