On Fri, Jan 16, 2015 at 11:14:42AM +0000, Ramana Radhakrishnan wrote: > > > On 16/01/15 10:20, Marcus Shawcroft wrote: > > On 15 January 2015 at 09:50, James Greenhalgh wrote: > > > >> 2015-01-15 James Greenhalgh > >> > >> * config/arm/cortex-a57.md: New. > >> * config/aarch64/aarch64.md: Include it. > >> * config/aarch64/aarch64-cores.def (cortex-a57): Tune for it. > >> * config/aarch64/aarch64-tune.md: Regenerate. > > > > This is low risk and only affects aarch64 backend. I think we should > > take this for gcc 5, so OK. > > > > Ramana do you want this wired up for aarch32 in a followup? > > Yes please. I think as long as it works, let's just take it for 5.0. I'd > rather not have divergence between the backends as this just creates > problems for us elsewhere. Hi, OK, that patch looks like this. I know I've missed the stage 3 deadline, but I agree that the two back ends should not diverge too much - and this patch is small and gives a nice little boost to performance on some popular benchmarks. I've regression tested it over the weekend with no issues. OK? Cheers, James --- 2014-01-19 James Greenhalgh * config/arm/arm-cores.def (cortex-a57): Use the new Cortex-A57 pipeline model. config/arm/arm.md: Include the new Cortex-A57 model. (generic_sched): Don't use generic_sched when tuning for Cortex-A57.