On Tue, Nov 19, 2013 at 10:09 AM, bin.cheng <bin.ch...@arm.com> wrote: > > >> -----Original Message----- >> From: Bernd Schmidt [mailto:ber...@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.ch...@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? >
Got approval and committed as r205015. Thanks, bin -- Best Regards.