Thanks a lot Kyrill! I will bare this in mind for future patches. Dennis
________________________________ From: Kyrill Tkachov <kyrylo.tkac...@foss.arm.com> Sent: Friday, August 23, 2019 9:27 AM To: Dennis Zhang <dennis.zh...@arm.com>; gcc-patches@gcc.gnu.org <gcc-patches@gcc.gnu.org> Cc: nd <n...@arm.com>; Richard Earnshaw <richard.earns...@arm.com>; Ramana Radhakrishnan <ramana.radhakrish...@arm.com> Subject: Re: [PATCH][Arm] Add support for missing CPUs Hi Dennis, On 8/22/19 4:52 PM, Dennis Zhang wrote: > Hi all, > > This patch adds '-mcpu' options for following CPUs: > Cortex-M35P, Cortex-A77, Cortex-A76AE. > > Related specifications are as following: > https://developer.arm.com/ip-products/processors/cortex-m > https://developer.arm.com/ip-products/processors/cortex-a > > Bootstraped/Regtested for arm-none-linux-gnueabihf. > > Please help to check. > > Cheers > Dennis > > gcc/ChangeLog: > > 2019-07-29 Dennis Zhang <dennis.zh...@arm.com> > > * config/arm/arm-cpus.in: New entries for Cortex-M35P, > Cortex-A76AE and Cortex-A77. > * config/arm/arm-tables.opt: Regenerated. > * config/arm/arm-tune.md: Likewise. > * doc/invoke.texi: Document the added processors. The patch is ok. The ChangeLogshould list each new cpu entry as a separate entity. I've adjusted the ChangeLog to: 2019-08-23 Dennis Zhang <dennis.zh...@arm.com> * config/arm/arm-cpus.in (cortex-m35p): New entry. (cortex-a76ae): Likewise. (cortex-a77): Likewise * config/arm/arm-tables.opt: Regenerate. * config/arm/arm-tune.md: Likewise. * doc/invoke.texi (ARM Options): Document cortex-m35p, cortx-a76ae, cortex-a77 CPU options. and committed it to trunk for you with r274845. Thanks for the patch! Kyrill