This is the mail archive of the
gcc-patches@gcc.gnu.org
mailing list for the GCC project.
Re: [PATCH][ARM] Add support for -mcpu=cortex-a72
- From: James Greenhalgh <james dot greenhalgh at arm dot com>
- To: Ramana Radhakrishnan <Ramana dot Radhakrishnan at arm dot com>
- Cc: Matthew Wahab <Matthew dot Wahab at arm dot com>, gcc-patches <gcc-patches at gcc dot gnu dot org>
- Date: Wed, 4 Feb 2015 13:37:12 +0000
- Subject: Re: [PATCH][ARM] Add support for -mcpu=cortex-a72
- Authentication-results: sourceware.org; auth=none
- References: <54D1F640 dot 3070109 at arm dot com> <CAJA7tRbyYkA-f_qGaDJGbuSZ0hBgXLpoZfP1FgURi1S6=gHL6g at mail dot gmail dot com>
On Wed, Feb 04, 2015 at 01:23:06PM +0000, Ramana Radhakrishnan wrote:
> On Wed, Feb 4, 2015 at 10:36 AM, Matthew Wahab <matthew.wahab@arm.com> wrote:
> > Hello,
> >
> > The Cortex-A72 is an ARMv8 core with the same architectural features as the
> > Cortex-A57. This patch adds support for the command line option
> > -mcpu=cortex-a72 with the same effect as the -mcpu=cortex-a57 option, with
> > only the name being different. It also adds support for the
> > -mcpu=cortex-a72.cortex-a53 big-little variant.
> >
> > Tested with check-gcc for arm-none-linux-gnueabihf. Tested the new cpu
> > options from the command line.
> >
> > Matthew
> >
> > gcc/
> > 2015-02-04 Matthew Wahab <matthew.wahab@arm.com>
> >
> > * config/arm/arm-cores.def: Add cortex-a72 and
> > cortex-a72.cortex-a53.
> > * config/arm/bpabi.h (BE8_LINK_SPEC): Likewise.
> > * config/arm/t-aprofile (MULTILIB_MATCHES): Likewise.
> > * config/arm/arm-tune.md: Regenerate.
> > * config/arm/arm-tables.opt: Add entries for "cortex-a72" and
> > "cortex-a72.cortex-a53".
> > * doc/invoke.texi (ARM Options/-mtune): Likewise.
>
>
> This is OK - it's purely wiring up a new name and treating that the
> same as an existing tune. No new functionality and no point in waiting
> for a year for this cpu option to be available for folks.
On Matthew's (offline) request, I've installed this as r220399 with the
following minor change:
> +@samp{cortex-a15.cortex-a7}, @samp{cortex-a57.cortex-a53}, @samp{cortex-a72.cortex-a53}.
Split this long line:
> +@samp{cortex-a15.cortex-a7}, @samp{cortex-a57.cortex-a53},
> +@samp{cortex-a72.cortex-a53}.
As so.
Cheers,
James