Re: [PATCH][ARM] Make issue rate part of per-core tuning structs

[Kyrill Tkachov <kyrylo dot tkachov at arm dot com>]

Ping (after the macro fusion patch)...
https://gcc.gnu.org/ml/gcc-patches/2014-11/msg02706.html

Thanks,
Kyrill
On 20/11/14 16:48, Kyrill Tkachov wrote:
> I should say that the patch context depends on the macro fusion hook
> implementation posted here:
> https://gcc.gnu.org/ml/gcc-patches/2014-11/msg00958.html
>
> Kyrill
>
> On 20/11/14 16:43, Kyrill Tkachov wrote:
> > Hi all,
> >
> > This patch makes the arm_issue_rate function lookup the issue rate of
> > the process from the tuning structs.
> > This makes it look more like the aarch64 mechanism and centralises a
> > processor-specific construct to the
> > tuning structs, thus not forcing us to remember to update the
> > arm_issue_rate function every time a new core
> > is added.
> >
> > A new tuning struct is added for the marvell-pj4 in order to decouple it
> > from the 9e tuning struct and
> > enable us to set it's correct issue rate to 2.
> >
> > Bootstrapped and tested on arm-none-gnueabihf.
> >
> > Ok for trunk?
> >
> > Thanks,
> > Kyrill
> >
> > 2014-11-19  Kyrylo Tkachov  <kyrylo.tkachov@arm.com>
> >
> >        * config/arm/arm-protos.h (struct tune_params): Add issue_rate field.
> >        * config/arm/arm.c (arm_slowmul_tune, arm_fastmul_tune,
> >        arm_strongarm_tune, arm_xscale_tune, arm_9e_tune, arm_v6t2_tune,
> >        arm_cortex_tune, arm_cortex_a8_tune, arm_cortex_a7_tune,
> >        arm_cortex_a15_tune, arm_cortex_a53_tune, arm_cortex_a57_tune,
> >        arm_cortex_a9_tune, arm_cortex_a12_tune, arm_v7m_tune, arm_v6m_tune,
> >        arm_fa726te_tune arm_cortex_a5_tune): Specify issue_rate value.
> >        (arm_issue_rate): Look up issue rate from tuning structs. Remove
> >        large switch statement.
> >        (arm_marvell_pj4_tune): New struct.
> >        * config/arm/arm-cores.def (marvell-pj4): Use arm_marvell_pj4_tune
> >        struct.