On Mon, Jan 19, 2015 at 5:44 PM, James Greenhalgh <james.greenha...@arm.com> wrote: > > 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 <james.greenha...@arm.com> >> > wrote: >> > >> >> 2015-01-15 James Greenhalgh <james.greenha...@arm.com> >> >> >> >> * 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?
This is OK as this is a tuning patch that was posted before the stage3 deadline and only wires in a pipeline description. Ramana > > Cheers, > James > > --- > 2014-01-19 James Greenhalgh <james.greenha...@arm.com> > > * 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.