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

[Richard Earnshaw <rearnsha at arm dot com>]

On 16/01/15 13:46, Kyrill Tkachov wrote:
> Hi,
> 
> This is a ping and a question.
> Now that the xgene1 support has gone in, is it ok if I update this patch 
> to include the xgene tuning struct? Will it still be acceptable for GCC5?
> 

In principle, that should be OK.  It's just work to make the patch work
with the latest version of the compiler.

Get it re-submitted quickly though.

R.

> Thanks,
> Kyrill
> 
> 
> On 09/01/15 11:32, Kyrill Tkachov wrote:
>> Ping.
>>
>> Thanks,
>> Kyrill
>> On 12/12/14 13:57, Kyrill Tkachov wrote:
>>> 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.
>>>
>>
>>
>