Re: [PATCH] Be careful about combined chain with length == 0 (PR, tree-optimization/70754).

2017-01-23 Thread Richard Biener
On Fri, Jan 20, 2017 at 3:30 PM, Bin.Cheng wrote: > On Thu, Jan 19, 2017 at 12:07 PM, Bin.Cheng wrote: >> On Thu, Jan 19, 2017 at 11:22 AM, Richard Biener >> wrote: >>> On Thu, Jan 19, 2017 at 11:25 AM, Bin.Cheng wrote: On Thu, Jan 19, 2017 at 9:42 AM, Richard Biener >>> >>> Or a later pas

Re: [PATCH] Be careful about combined chain with length == 0 (PR, tree-optimization/70754).

2017-01-20 Thread Bin.Cheng
On Thu, Jan 19, 2017 at 12:07 PM, Bin.Cheng wrote: > On Thu, Jan 19, 2017 at 11:22 AM, Richard Biener > wrote: >> On Thu, Jan 19, 2017 at 11:25 AM, Bin.Cheng wrote: >>> On Thu, Jan 19, 2017 at 9:42 AM, Richard Biener >> >> Or a later pass introduced it (in this case, the vectorizer). >> >>> Pred

Re: [PATCH] Be careful about combined chain with length == 0 (PR, tree-optimization/70754).

2017-01-19 Thread Bin.Cheng
On Thu, Jan 19, 2017 at 11:22 AM, Richard Biener wrote: > On Thu, Jan 19, 2017 at 11:25 AM, Bin.Cheng wrote: >> On Thu, Jan 19, 2017 at 9:42 AM, Richard Biener >> wrote: >>> On Wed, Jan 18, 2017 at 4:32 PM, Bin.Cheng wrote: On Wed, Jan 18, 2017 at 2:54 PM, Richard Biener wrote: >

Re: [PATCH] Be careful about combined chain with length == 0 (PR, tree-optimization/70754).

2017-01-19 Thread Richard Biener
On Thu, Jan 19, 2017 at 11:25 AM, Bin.Cheng wrote: > On Thu, Jan 19, 2017 at 9:42 AM, Richard Biener > wrote: >> On Wed, Jan 18, 2017 at 4:32 PM, Bin.Cheng wrote: >>> On Wed, Jan 18, 2017 at 2:54 PM, Richard Biener >>> wrote: On Wed, Jan 18, 2017 at 11:10 AM, Martin Liška wrote: > Hel

Re: [PATCH] Be careful about combined chain with length == 0 (PR, tree-optimization/70754).

2017-01-19 Thread Bin.Cheng
On Thu, Jan 19, 2017 at 9:42 AM, Richard Biener wrote: > On Wed, Jan 18, 2017 at 4:32 PM, Bin.Cheng wrote: >> On Wed, Jan 18, 2017 at 2:54 PM, Richard Biener >> wrote: >>> On Wed, Jan 18, 2017 at 11:10 AM, Martin Liška wrote: Hello. Patch can bootstrap on ppc64le-redhat-linu

Re: [PATCH] Be careful about combined chain with length == 0 (PR, tree-optimization/70754).

2017-01-19 Thread Richard Biener
On Wed, Jan 18, 2017 at 4:32 PM, Bin.Cheng wrote: > On Wed, Jan 18, 2017 at 2:54 PM, Richard Biener > wrote: >> On Wed, Jan 18, 2017 at 11:10 AM, Martin Liška wrote: >>> Hello. >>> >>> >>> Patch can bootstrap on ppc64le-redhat-linux and survives regression tests. >>> >>> Ready to be installed? >

Re: [PATCH] Be careful about combined chain with length == 0 (PR, tree-optimization/70754).

2017-01-18 Thread Bin.Cheng
On Wed, Jan 18, 2017 at 2:54 PM, Richard Biener wrote: > On Wed, Jan 18, 2017 at 11:10 AM, Martin Liška wrote: >> Hello. >> >> >> Patch can bootstrap on ppc64le-redhat-linux and survives regression tests. >> >> Ready to be installed? > > I'm not sure. If we have such zero distance refs in the IL

Re: [PATCH] Be careful about combined chain with length == 0 (PR, tree-optimization/70754).

2017-01-18 Thread Vidya Praveen
among loop iterations (which is main motivation of the > pass), it's doing PRE > if I'm right. > > Patch can bootstrap on ppc64le-redhat-linux and survives regression tests. I could bootstrap on aarch64-none-linux-gnu without any issues, regression tests are fine and the

Re: [PATCH] Be careful about combined chain with length == 0 (PR, tree-optimization/70754).

2017-01-18 Thread Richard Biener
On Wed, Jan 18, 2017 at 11:10 AM, Martin Liška wrote: > Hello. > > After basic understanding of loop predictive commoning, the problematic > combined chain is: > > Loads-only chain 0x38b6730 (combined) > max distance 0 > references: > MEM[(real(kind=8) *)vectp_a.29_81] (id 1) > offs

Re: [PATCH] Be careful about combined chain with length == 0 (PR, tree-optimization/70754).

2017-01-18 Thread Bin.Cheng
On Wed, Jan 18, 2017 at 10:10 AM, Martin Liška wrote: > Hello. > > After basic understanding of loop predictive commoning, the problematic > combined chain is: > > Loads-only chain 0x38b6730 (combined) > max distance 0 > references: > MEM[(real(kind=8) *)vectp_a.29_81] (id 1) > offs

[PATCH] Be careful about combined chain with length == 0 (PR, tree-optimization/70754).

2017-01-18 Thread Martin Liška
le-redhat-linux and survives regression tests. Ready to be installed? Martin >From 41b153cf975374fff48419ec8ac5991ac134735f Mon Sep 17 00:00:00 2001 From: marxin Date: Tue, 17 Jan 2017 14:22:40 +0100 Subject: [PATCH] Be careful about combined chain with length == 0 (PR tree-optimization