Hi James/Julian, Kindly merge this patch to the trunk. Thanks.
>> 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. > with regards, Virendra Pathak On Mon, Jul 11, 2016 at 2:50 PM, Virendra Pathak <virendra.pat...@broadcom.com> wrote: > 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. >>
