This is the mail archive of the
gcc-patches@gcc.gnu.org
mailing list for the GCC project.
Re: [patch] [arm] ARM Cortex-M3/M4 tuning
- From: Janis Johnson <janis_johnson at mentor dot com>
- To: Joey Ye <joey dot ye at arm dot com>, <janisjo at codesourcery dot com>
- Cc: <gcc-patches at gcc dot gnu dot org>, Ramana Radhakrishnan <Ramana dot Radhakrishnan at arm dot com>
- Date: Wed, 13 Nov 2013 09:45:23 -0800
- Subject: Re: [patch] [arm] ARM Cortex-M3/M4 tuning
- Authentication-results: sourceware.org; auth=none
- References: <000001cedc28$98152c10$c83f8430$ at arm dot com> <527CAA9B dot 3090601 at arm dot com> <000401cee038$79132010$6b396030$ at arm dot com>
- Reply-to: <janisjo at codesourcery dot com>
On 11/12/2013 10:20 PM, Joey Ye wrote:
> Janis, can you please take a look at test case changes.
>
> Thanks,
> Joey
They look fine.
Janis
>> -----Original Message-----
>> From: Ramana Radhakrishnan
>> Sent: Friday, November 08, 2013 17:11
>> To: Joey Ye
>> Cc: gcc-patches@gcc.gnu.org; janisjo@codesourcery.com
>> Subject: Re: [patch] [arm] ARM Cortex-M3/M4 tuning
>>
>>>> ChangeLog:
>>>>
>>>> 2013-11-01 Julian Brown <julian@codesourcery.com>
>>>> Joey Ye <joey.ye@arm.com>
>>>>
>>>> * config/arm/arm.c (arm_cortex_m_branch_cost): New.
>>>> (arm_v7m_tune): New.
>>>> (arm_*_tune): Add comments for Sched adj cost.
>>
>> List all names here please rather than a regexp.
>>
>>>> * config/arm/arm-cores.def (cortex-m4, cortex-m3):
>>>> Use arm_v7m_tune tuning.
>>>>
>>
>> The ARM parts are ok but I'd like a testsuite maintainer to look at the
>> testsuite changes before committing.
>>
>> regards
>> Ramana
>>
>>>> testsuite:
>>>> 2013-11-01 Joey Ye <joey.ye@arm.com>
>>>>
>>>> * gcc.dg/tree-ssa/forwprop-28.c: Disable for cortex_m.
>>>> * gcc.dg/tree-ssa/vrp47.c: Likewise.
>>>> * gcc.dg/tree-ssa/vrp87.c: Likewise.
>>>> * gcc.dg/tree-ssa/ssa-dom-thread-4.c: Ignore for cortex_m.
>>>> * gcc.dg/tree-ssa/ssa-vrp-thread-1.c: Likewise.