Re: [PATCH][AArch64] Add support for missing CPUs

[Kyrill Tkachov <kyrylo dot tkachov at foss dot arm dot com>]

Hi James,

On 9/2/19 6:30 PM, James Greenhalgh wrote:
> On Thu, Aug 22, 2019 at 12:03:33PM +0100, Kyrill Tkachov wrote:
> > Hi Dennis,
> >
> > On 8/21/19 10:27 AM, Dennis Zhang wrote:
> > > Hi all,
> > >
> > > This patch adds '-mcpu' options for following CPUs:
> > > Cortex-A77, Cortex-A76AE, Cortex-A65, Cortex-A65AE, and Cortex-A34.
> > >
> > > Related specifications are as following:
> > > https://developer.arm.com/ip-products/processors/cortex-a
> > >
> > > Bootstraped/regtested for aarch64-none-linux-gnu.
> > >
> > > Please help to check if it's ready.
> > This looks ok to me but you'll need maintainer approval.
> At this point Kyrill, I fully trust your OK without looking at the
> patch in any more detail...
>
> I think at Cauldron we ought to add some time during the Arm/AArch64 BoF
> to discuss what the community would like us to do about maintainership in
> AArch64. It seems clear to me that I'm just slowing you and others down now
> by rubberstamping your decisions.

Ok, judging by the schedule there should be plenty of opportunity to 
discuss innovative maintainership schemes at Cauldron!


> To be clear, this particular patch is OK for trunk - but I think it is
> time to have a conversation about how we can make this experience easier
> for everyone.

Thanks, I've committed this on Dennis' behalf with r275333.

Kyrill

> Thanks,
> James
>
> > Thanks,
> >
> > Kyrill
> >
> >
> > > Many thanks!
> > > Dennis
> > >
> > > gcc/ChangeLog:
> > >
> > > 2019-08-21  Dennis Zhang  <dennis.zhang@arm.com>
> > >
> > >          * config/aarch64/aarch64-cores.def (AARCH64_CORE): New entries
> > >          for Cortex-A77, Cortex-A76AE, Cortex-A65, Cortex-A65AE, and
> > >          Cortex-A34.
> > >          * config/aarch64/aarch64-tune.md: Regenerated.
> > >          * doc/invoke.texi: Document the new processors.