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.tkac...@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. >>> >> >> >
