diff mbox

[ARM] Add Cortex-A17 support

Message ID 54770654.7050803@arm.com
State New
Headers show

Commit Message

Kyrylo Tkachov Nov. 27, 2014, 11:09 a.m. UTC
On 27/11/14 08:52, Ramana Radhakrishnan wrote:
> On Thu, Nov 13, 2014 at 5:25 PM, Kyrill Tkachov <kyrylo.tkachov@arm.com> wrote:
>> Hi all,
>>
>> This patch adds support for the Cortex-A17 processor to the arm backend.
>> Cortex-A17 is an ARMv7ve core with the same architectural features as the
>> Cortex-A7, A12 and A15 cores.
>> The -m{tune, cpu}=cortex-a17 option is added and a pipeline description for
>> instruction scheduling is provided.
>> This has given an uplift over -mcpu=cortex-a15 tuning on a number of
>> benchmarks.
>>
>> The patch is fairly self-contained with the bulk of the diffstat being the
>> pipeline description files.
>>
>> Bootstrapped and tested on arm-none-linux-gnueabihf.
>>
>> Ok for trunk?
>>
> Ok with documentation update in invoke.texi and an entry for changes.html

Hi Ramana,

Thanks for the review. The invoke.texi changes were posted and approved 
at https://gcc.gnu.org/ml/gcc-patches/2014-11/msg02683.html
and I'm attaching the proposed changes.html patch.

Kyrill

>
> Ramana
>
>> Thanks,
>> Kyrill
>>
>> 2014-11-13  Kyrylo Tkachov  <kyrylo.tkachov@arm.com>
>>
>>      * config/arm/arm.md (generic_sched): Specify cortexa17 in 'no' list.
>>      Include cortex-a17.md.
>>      * config/arm/arm.c (arm_issue_rate): Specify 2 for cortexa17.
>>      * config/arm/arm-cores.def (cortex-a17): New entry.
>>      * config/arm/arm-tables.opt: Regenerate.
>>      * config/arm/arm-tune.md: Regenerate.
>>      * config/arm/bpabi.h (BE8_LINK_SPEC): Specify mcpu=cortex-a17.
>>      * config/arm/cortex-a17.md: New file.
>>      * config/arm/cortex-a17-neon.md: New file.
>>      * config/arm/driver-arm.c (arm_cpu_table): Add entry for cortex-a17.
>>      * config/arm/t-aprofile: Add cortex-a17 entries to MULTILIB_MATCHES.

Comments

Ramana Radhakrishnan Nov. 27, 2014, 11:15 a.m. UTC | #1
On 27/11/14 11:09, Kyrill Tkachov wrote:
>
> On 27/11/14 08:52, Ramana Radhakrishnan wrote:
>> On Thu, Nov 13, 2014 at 5:25 PM, Kyrill Tkachov <kyrylo.tkachov@arm.com> wrote:
>>> Hi all,
>>>
>>> This patch adds support for the Cortex-A17 processor to the arm backend.
>>> Cortex-A17 is an ARMv7ve core with the same architectural features as the
>>> Cortex-A7, A12 and A15 cores.
>>> The -m{tune, cpu}=cortex-a17 option is added and a pipeline description for
>>> instruction scheduling is provided.
>>> This has given an uplift over -mcpu=cortex-a15 tuning on a number of
>>> benchmarks.
>>>
>>> The patch is fairly self-contained with the bulk of the diffstat being the
>>> pipeline description files.
>>>
>>> Bootstrapped and tested on arm-none-linux-gnueabihf.
>>>
>>> Ok for trunk?
>>>
>> Ok with documentation update in invoke.texi and an entry for changes.html
>
> Hi Ramana,
>
> Thanks for the review. The invoke.texi changes were posted and approved
> at https://gcc.gnu.org/ml/gcc-patches/2014-11/msg02683.html
> and I'm attaching the proposed changes.html patch.

Ok thanks.

Ramana

>
> Kyrill
>
>>
>> Ramana
>>
>>> Thanks,
>>> Kyrill
>>>
>>> 2014-11-13  Kyrylo Tkachov  <kyrylo.tkachov@arm.com>
>>>
>>>       * config/arm/arm.md (generic_sched): Specify cortexa17 in 'no' list.
>>>       Include cortex-a17.md.
>>>       * config/arm/arm.c (arm_issue_rate): Specify 2 for cortexa17.
>>>       * config/arm/arm-cores.def (cortex-a17): New entry.
>>>       * config/arm/arm-tables.opt: Regenerate.
>>>       * config/arm/arm-tune.md: Regenerate.
>>>       * config/arm/bpabi.h (BE8_LINK_SPEC): Specify mcpu=cortex-a17.
>>>       * config/arm/cortex-a17.md: New file.
>>>       * config/arm/cortex-a17-neon.md: New file.
>>>       * config/arm/driver-arm.c (arm_cpu_table): Add entry for cortex-a17.
>>>       * config/arm/t-aprofile: Add cortex-a17 entries to MULTILIB_MATCHES.
>
diff mbox

Patch

Index: htdocs/gcc-5/changes.html
===================================================================
RCS file: /cvs/gcc/wwwdocs/htdocs/gcc-5/changes.html,v
retrieving revision 1.43
diff -U 3 -r1.43 changes.html
--- htdocs/gcc-5/changes.html	27 Nov 2014 09:50:04 -0000	1.43
+++ htdocs/gcc-5/changes.html	27 Nov 2014 09:57:19 -0000
@@ -398,6 +398,14 @@ 
       configure option with either of <code>--with-tune</code> or
       <code>--with-arch</code>.
       </li>
+      <li> Support for the Cortex-A17 processor has been added through the
+      <code>-mcpu=cortex-a17</code> and <code>-mtune=cortex-a17</code> options.
+      </li>
+      <li> Initial big.LITTLE tuning support for the combination of Cortex-A17
+       and Cortex-A7 was added through the <code>-mcpu=cortex-a17.cortex-a7
+       </code> and <code>-mtune=cortex-a17.cortex-a7</code> options.
+      </li>
+
      </ul>
 
 <h3 id="x86">IA-32/x86-64</h3>