Hi James, > This patch is OK. Thanks for the review.
> Julian has commit rights and is on the ChangeLog, so I'll let him > commit the patch on your behalf. Julian, Kindly commit the patch to the trunk. Thanks. with regards, Virendra Pathak On Mon, Jul 11, 2016 at 2:28 PM, James Greenhalgh <james.greenha...@arm.com> wrote: > 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.pat...@broadcom.com> >> Julian Brown <jul...@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. >
