Re: [ARM] Wire up the new scheduler description for the ARM Cortex-A57 processor

[Ramana Radhakrishnan <ramana.gcc@googlemail.com>]

On Mon, Jan 19, 2015 at 5:44 PM, James Greenhalgh
<james.greenhalgh@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.greenhalgh@arm.com> wrote:
>> >
>> >> 2015-01-15  James Greenhalgh  <james.greenhalgh@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.greenhalgh@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.