This is the mail archive of the
gcc-patches@gcc.gnu.org
mailing list for the GCC project.
Re: [PATCH/AARCH64] Add rtx_costs routine for vulcan.
- From: James Greenhalgh <james dot greenhalgh at arm dot com>
- To: Virendra Pathak <virendra dot pathak at broadcom dot com>
- Cc: <gcc-patches at gcc dot gnu dot org>, Marcus Shawcroft <marcus dot shawcroft at arm dot com>, Julian Brown <julian at codesourcery dot com>, <nd at arm dot com>
- Date: Mon, 11 Jul 2016 09:58:56 +0100
- Subject: Re: [PATCH/AARCH64] Add rtx_costs routine for vulcan.
- Authentication-results: sourceware.org; auth=none
- Nodisclaimer: True
- References: <CANx+fTCYNfZRAdeSMJn40oYDQovN9A=TPgrHjveAc9ohFHPbEg@mail.gmail.com> <20160629104135.GB1138@arm.com> <CANx+fTCV-1ceE5-uY_94O2Euj02Suo+T91HQi+1EFWWUbm+Y1A@mail.gmail.com> <CANx+fTB37r8uFNE8D14hyJEqhPfEG2VnJr-HeOJeNXssrLACUw@mail.gmail.com>
- Spamdiagnosticmetadata: NSPM
- Spamdiagnosticoutput: 1:99
On Fri, Jul 08, 2016 at 04:01:33PM +0530, Virendra Pathak wrote:
> Hi James,
>
> Please find the patch after taking care of your comments.
>
>
> > Did you see those patches, and did you consider whether there would be a
> > benefit to doing the same for Vulcan?
> In our simulation environment, we did not observe any performance gain
> for specfp2006.
> However, we did it to keep the cost strategy same as cortexa-57/53.
>
> Please review and merge to trunk.
Hi Virendra,
This patch is OK.
Julian has commit rights and is on the ChangeLog, so I'll let him
commit the patch on your behalf.
Thanks,
James
> gcc/ChangeLog:
>
> Virendra Pathak <virendra.pathak@broadcom.com>
> Julian Brown <julian@codesourcery.com>
>
> * config/aarch64/aarch64-cores.def: Update vulcan COSTS.
> * config/aarch64/aarch64-cost-tables.h
> (vulcan_extra_costs): New variable.
> * config/aarch64/aarch64.c
> (vulcan_addrcost_table): Likewise.
> (vulcan_regmove_cost): Likewise.
> (vulcan_vector_cost): Likewise.
> (vulcan_branch_cost): Likewise.
> (vulcan_tunings): Likewise.