Re: [PATCH] Fix PR84859

2018-03-20 Thread Jakub Jelinek
On Tue, Mar 20, 2018 at 01:42:04PM +0100, Richard Biener wrote: > 2018-03-20 Richard Biener > > * testsuite/libgomp.graphite/force-parallel-4.c: XFAIL one > parallelizable loop. > > Index: libgomp/testsuite/libgomp.graphite/force-parallel-4.c > =

Re: [PATCH] Fix PR84859

2018-03-20 Thread Richard Biener
On Tue, 20 Mar 2018, Jakub Jelinek wrote: > On Tue, Mar 20, 2018 at 01:42:04PM +0100, Richard Biener wrote: > > 2018-03-20 Richard Biener > > > > * testsuite/libgomp.graphite/force-parallel-4.c: XFAIL one > > parallelizable loop. > > > > Index: libgomp/testsuite/libgomp.graphite/force

Re: [PATCH] Fix PR84859

2018-03-20 Thread Richard Biener
On Tue, 20 Mar 2018, Richard Biener wrote: > On Tue, 20 Mar 2018, Bin.Cheng wrote: > > > On Mon, Mar 19, 2018 at 11:57 AM, Jakub Jelinek wrote: > > > On Fri, Mar 16, 2018 at 03:08:17PM +0100, Richard Biener wrote: > > >> Bootstrapped and tested on x86_64-unknown-linux-gnu, ok? > > > > > > Ok, th

Re: [PATCH] Fix PR84859

2018-03-20 Thread Richard Biener
On Tue, 20 Mar 2018, Bin.Cheng wrote: > On Mon, Mar 19, 2018 at 11:57 AM, Jakub Jelinek wrote: > > On Fri, Mar 16, 2018 at 03:08:17PM +0100, Richard Biener wrote: > >> Bootstrapped and tested on x86_64-unknown-linux-gnu, ok? > > > > Ok, thanks. > Hi Richard, > > This change causes below test fai

Re: [PATCH] Fix PR84859

2018-03-20 Thread Bin.Cheng
On Mon, Mar 19, 2018 at 11:57 AM, Jakub Jelinek wrote: > On Fri, Mar 16, 2018 at 03:08:17PM +0100, Richard Biener wrote: >> Bootstrapped and tested on x86_64-unknown-linux-gnu, ok? > > Ok, thanks. Hi Richard, This change causes below test failure on aarch64/arm linux: FAIL: libgomp.graphite/force

Re: [PATCH] Fix PR84859

2018-03-19 Thread Jakub Jelinek
On Fri, Mar 16, 2018 at 03:08:17PM +0100, Richard Biener wrote: > Bootstrapped and tested on x86_64-unknown-linux-gnu, ok? Ok, thanks. Jakub

[PATCH] Fix PR84859

2018-03-16 Thread Richard Biener
This fixes a bogus warning by performing MIN_EXPR detection on memory which avoids bogus threading and isolating of a path with a bogus memcpy. I've gone back in time and at some point we had no MAX_STORES_TO_SINK param but handled just the last_and_only_stmt case of sinking two stores. The para