This is the mail archive of the
gcc-patches@gcc.gnu.org
mailing list for the GCC project.
Re: [PATCH, ARM] New CPU support for Marvell PJ4 cores
- From: Chung-Lin Tang <cltang at codesourcery dot com>
- To: Ramana Radhakrishnan <ramana dot radhakrishnan at linaro dot org>
- Cc: Yi-Hsiu Hsu <ahsu at marvell dot com>, "gcc-patches at gcc dot gnu dot org" <gcc-patches at gcc dot gnu dot org>
- Date: Thu, 14 Jun 2012 14:29:54 +0800
- Subject: Re: [PATCH, ARM] New CPU support for Marvell PJ4 cores
- References: <689C75E0D210324F8BE88961219F681624B5949520@SC-VEXCH2.marvell.com> <CACUk7=WiZr-GEbHPjBOZ8=OP9jr2XqMnOWfrPSOANDkqs771ZA@mail.gmail.com>
On 2012/6/14 02:18 AM, Ramana Radhakrishnan wrote:
> On 29 May 2012 10:07, Yi-Hsiu Hsu <ahsu@marvell.com> wrote:
>> Hi,
>>
>> This patch maintains Marvell PJ4 cores pipeline description.
>> Run arm testsuite on arm-linux-gnueabi and no extra regressions are found.
>>
>> * config/arm/marvell-pj4.md: New marvell-pj4 pipeline description.
>> * config/arm/arm.c (arm_issue_rate): Add marvell_pj4.
>> * config/arm/arm-cores.def: Add core marvell-pj4.
>> * config/arm/arm-tune.md: Regenerated.
>> * config/arm/arm-tables.opt: Regenerated.
>> * doc/invoke.texi: Added entry for marvell-pj4.
>
> This command line option should also be added to BE8_LINK_SPEC similar
> to what's done for the other v7-a cores.
>
> Ok with that change.
I take the blame for not doing this back then, but I suggest the
resource names be properly qualified, similar to most recently added
pipeline descriptions, e.g. prepend resource/reservation names with
"pj4_" ("is" to "pj4_is", etc.)
Chung-Lin