RE: [PATCH GCC]Compute, cache and use cost of auto-increment rtx patterns in IVOPT

["bin.cheng" <bin dot cheng at arm dot com>]

> -----Original Message-----
> From: Bernd Schmidt [mailto:bernds@codesourcery.com]
> Sent: Monday, November 18, 2013 8:05 PM
> To: Bin Cheng
> Cc: gcc-patches@gcc.gnu.org
> Subject: Re: [PATCH GCC]Compute, cache and use cost of auto-increment rtx
> patterns in IVOPT
> 
> On 11/04/2013 04:31 AM, bin.cheng wrote:
> > 2013-11-01  Bin Cheng  <bin.cheng@arm.com>
> >
> > 	* tree-ssa-loop-ivopts.c (enum ainc_type): New.
> > 	(address_cost_data): New field.
> > 	(get_address_cost): Compute auto-increment rtx cost in ainc_costs.
> > 	Use ainc_costs for auto-increment rtx patterns.
> > 	Cleanup TWS.
> 
> I think this is fine. I'd just like to see AINC_NUM gone and its use
replaced by
> AIC_NONE, we don't really need two separate enum codes for that.
> 
Thanks for reviewing, I updated patch as suggested.

Hi Richard, is this patch ok for you too?

Thanks,
bin

Attachment: ivopt-ainc_cost-20131119.txt
Description: Text document