This is the mail archive of the
gcc-patches@gcc.gnu.org
mailing list for the GCC project.
Re: [PATCH GCC]Compute, cache and use cost of auto-increment rtx patterns in IVOPT
- From: "Bin.Cheng" <amker dot cheng at gmail dot com>
- To: gcc-patches List <gcc-patches at gcc dot gnu dot org>
- Cc: Bernd Schmidt <bernds at codesourcery dot com>, Richard Biener <richard dot guenther at gmail dot com>
- Date: Tue, 19 Nov 2013 17:49:19 +0800
- Subject: Re: [PATCH GCC]Compute, cache and use cost of auto-increment rtx patterns in IVOPT
- Authentication-results: sourceware.org; auth=none
- References: <001001ced90e$4ecd9af0$ec68d0d0$ at arm dot com> <528A0269 dot 7070503 at codesourcery dot com> <002701cee4cc$64d90a90$2e8b1fb0$ at arm dot com>
On Tue, Nov 19, 2013 at 10:09 AM, bin.cheng <bin.cheng@arm.com> wrote:
>
>
>> -----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?
>
Got approval and committed as r205015.
Thanks,
bin
--
Best Regards.