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: Matthias Klose <doko at ubuntu dot com>
- To: ramrad01 at arm dot com
- Cc: ahsu at marvell dot com, gcc-patches at gcc dot gnu dot org, gcc-patches at gcc dot gnu dot org
- Date: Sun, 20 Jan 2013 21:36:24 +0100
- Subject: Re: [PATCH, ARM] New CPU support for Marvell PJ4 cores
- References: <689C75E0D210324F8BE88961219F681624B5949520@SC-VEXCH2.marvell.com> <CACUk7=WiZr-GEbHPjBOZ8=OP9jr2XqMnOWfrPSOANDkqs771ZA@mail.gmail.com> <689C75E0D210324F8BE88961219F681624B5BCD276@SC-VEXCH2.marvell.com> <50F95BFB.1060808@arm.com>
Am 18.01.2013 15:28, schrieb Ramana Radhakrishnan:
> On 06/20/12 03:53, Yi-Hsiu Hsu wrote:
>> marvell-pj4 is added to BE8_LINK_SPEC.
>
> Sorry about the time it's taken to finish this patch up. I seem to have missed
> this one in the review process.
>
> I've now applied the attached patch after taking into account the recent
> attribute changes and treated alu_reg and simple_alu_imm as you have treated
> alu, rebased to trunk to fix up some small issues with BE8_LINK_SPECS.
>
> Additionally I've removed tune_marvell as that seems redundant at this point of
> time - an additional attribute and testing that appears to be unnecessary
> instead of just one more inequality test.
>
> The only part I'm not sure about is how to treat the simple_alu_shift category
> here , so a patch to handle them in the pj4 pipeline description would be welcome.
>
>
> Thanks
> Ramana
>
> 2013-01-18 Yi-Hsiu Hsu <ahsu@marvell.com>
> Ramana Radhakrishnan <ramana.radhakrishnan@arm.com>
>
> * config/arm/marvell-pj4.md: New file.
> * config/arm/arm.c (arm_issue_rate): Add marvell_pj4.
> * config/arm/arm.md (generic_sched): Add marvell_pj4.
> (generic_vfp): Likewise.
> * config/arm/arm-cores.def: Add marvell-pj4.
> * config/arm/arm-tune.md: Regenerate.
> * config/arm/arm-tables.opt: Regenerate.
> * config/arm/bpabi.h (BE8_LINK_SPEC): Add marvell_pj4.
> * doc/invoke.texi: Document marvell-pj4.
>
>>
>> Modified patch is attached.
the patch was missing the new file config/arm/marvell-pj4.md. Checked is as
obvious to restore the bootstrap.
Matthias