[PATCH/AARCH64] Add issue_rate tuning field

Richard Earnshaw rearnsha@arm.com
Tue Jan 14 10:15:00 GMT 2014


On 14/01/14 01:49, Andrew Pinski wrote:
> Hi,
>   While writing a scheduler for Cavium's aarch64 processor (Thunder),
> I found there was no way currently to change the issue rate in
> back-end.  This patch adds a field (issue_rate) to tune_params and
> creates a new function that the middle-end calls.  I updated the
> current two tuning variables (generic_tunings and cortexa53_tunings)
> to be 1 which was the default before.
> 
> OK?  Built and tested for aarch64-elf with no regressions.
> 
> Thanks,
> Andrew Pinski
> 
> ChangeLog:
> * config/aarch64/aarch64-protos.h (tune_params): Add issue_rate.
> * config/aarch64/aarch64.c (generic_tunings): Add issue rate of 1.
>  (cortexa53_tunings): Likewise.
> (aarch64_sched_issue_rate): New function.
> (TARGET_SCHED_ISSUE_RATE): Define.
> 
> 

Ug, I'd missed that we weren't setting this.

I think the value should be 2 for both generic and Cortex-A53 (I can't
really envisage single-issue AArch64 systems).

OK with that change (yes I know that wasn't the default before)...

R.




More information about the Gcc-patches mailing list