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; jani...@codesourcery.com
>> Subject: Re: [patch] [arm] ARM Cortex-M3/M4 tuning
>>
>>>> ChangeLog:
>>>>
>>>>      2013-11-01  Julian Brown  <jul...@codesourcery.com>
>>>>      Joey Ye  <joey...@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...@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.

Reply via email to