Re: [PATCH] Improve SCEV for array element

2012-03-12 Thread Richard Guenther
On Thu, Mar 8, 2012 at 8:13 AM, Jiangning Liu jiangning@arm.com wrote: -Original Message- From: Richard Guenther [mailto:richard.guent...@gmail.com] Sent: Tuesday, March 06, 2012 9:12 PM To: Jiangning Liu Cc: gcc-patches@gcc.gnu.org Subject: Re: [PATCH] Improve SCEV for array

RE: [PATCH] Improve SCEV for array element

2012-03-07 Thread Jiangning Liu
-Original Message- From: Richard Guenther [mailto:richard.guent...@gmail.com] Sent: Tuesday, March 06, 2012 9:12 PM To: Jiangning Liu Cc: gcc-patches@gcc.gnu.org Subject: Re: [PATCH] Improve SCEV for array element On Fri, Jan 20, 2012 at 10:06 AM, Jiangning Liu jiangning

Re: [PATCH] Improve SCEV for array element

2012-03-06 Thread Richard Guenther
On Fri, Jan 20, 2012 at 10:06 AM, Jiangning Liu jiangning@arm.com wrote: It's definitely not ok at this stage but at most for next stage1. OK. I may wait until next stage1. This is a very narrow pattern-match.  It doesn't allow for a[i].x for example, even if a[i] is a one-element

RE: [PATCH] Improve SCEV for array element

2012-02-13 Thread Jiangning Liu
-Original Message- From: gcc-patches-ow...@gcc.gnu.org [mailto:gcc-patches- ow...@gcc.gnu.org] On Behalf Of Jiangning Liu Sent: Friday, January 20, 2012 5:07 PM To: 'Richard Guenther' Cc: gcc-patches@gcc.gnu.org Subject: RE: [PATCH] Improve SCEV for array element It's

Re: [PATCH] Improve SCEV for array element

2012-02-13 Thread Richard Guenther
@gcc.gnu.org Subject: RE: [PATCH] Improve SCEV for array element It's definitely not ok at this stage but at most for next stage1. OK. I may wait until next stage1. This is a very narrow pattern-match.  It doesn't allow for a[i].x for example, even if a[i] is a one-element structure.  I think

RE: [PATCH] Improve SCEV for array element

2012-01-20 Thread Jiangning Liu
It's definitely not ok at this stage but at most for next stage1. OK. I may wait until next stage1. This is a very narrow pattern-match. It doesn't allow for a[i].x for example, even if a[i] is a one-element structure. I think the canonical way of handling ADDR_EXPR is to use sth like

Re: [PATCH] Improve SCEV for array element

2012-01-18 Thread Richard Guenther
On Wed, Jan 18, 2012 at 8:41 AM, Jiangning Liu jiangning@arm.com wrote: This code change intends to improve scev for array element and reduce the common sub-expressions in loop, which may be introduced by multiple reference of expression like a[i]. With this optimization the register