RE: [patch] [arm] ARM Cortex-M3/M4 tuning
Sorry about this. I should have run x86 make check.
- Joey
> -Original Message-
> From: Richard Biener [mailto:richard.guent...@gmail.com]
> Sent: Thursday, November 14, 2013 22:16
> To: H.J. Lu
> Cc: Joey Ye; Janis Johnson; GCC Patches; Ramana Radhakrishnan
> Subject: Re: [patch] [arm] ARM Cortex-M3/M4 tuning
>
> On Thu, Nov 14, 2013 at 1:35 PM, H.J. Lu wrote:
> > On Thu, Nov 14, 2013 at 2:24 AM, Joey Ye wrote:
> >> In mainline and arm/embedded-4_8-branch now.
> >>
> >>> -Original Message-
> >>> From: Janis Johnson [mailto:janis_john...@mentor.com]
> >>> Sent: Thursday, November 14, 2013 1:45
> >>> To: Joey Ye; jani...@codesourcery.com
> >>> Cc: gcc-patches@gcc.gnu.org; Ramana Radhakrishnan
> >>> Subject: Re: [patch] [arm] ARM Cortex-M3/M4 tuning
> >>>
> >>> 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.
> >>>
> >
> > I got
> >
> > ERROR: (DejaGnu) proc "{ scan-tree-dump-times "Threaded" 1 "vrp1" } ||
> > { arm_cortex_m }" does not exist.
> >
> > on Linux/x86-64.
>
> me too, this stops testing tree-ssa.exp at this point which is bad. FIxed
as
> attached.
>
> Richard.
>
> >
> >
> >
> >
> >
> > --
> > H.J.
Re: [patch] [arm] ARM Cortex-M3/M4 tuning
On Thu, Nov 14, 2013 at 1:35 PM, H.J. Lu wrote:
> On Thu, Nov 14, 2013 at 2:24 AM, Joey Ye wrote:
>> In mainline and arm/embedded-4_8-branch now.
>>
>>> -Original Message-
>>> From: Janis Johnson [mailto:janis_john...@mentor.com]
>>> Sent: Thursday, November 14, 2013 1:45
>>> To: Joey Ye; jani...@codesourcery.com
>>> Cc: gcc-patches@gcc.gnu.org; Ramana Radhakrishnan
>>> Subject: Re: [patch] [arm] ARM Cortex-M3/M4 tuning
>>>
>>> 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.
>>>
>
> I got
>
> ERROR: (DejaGnu) proc "{ scan-tree-dump-times "Threaded" 1 "vrp1" } ||
> { arm_cortex_m }" does not exist.
>
> on Linux/x86-64.
me too, this stops testing tree-ssa.exp at this point which is bad. FIxed
as attached.
Richard.
>
>
>
>
>
> --
> H.J.
p
Description: Binary data
Re: [patch] [arm] ARM Cortex-M3/M4 tuning
On Thu, Nov 14, 2013 at 2:24 AM, Joey Ye wrote:
> In mainline and arm/embedded-4_8-branch now.
>
>> -Original Message-
>> From: Janis Johnson [mailto:janis_john...@mentor.com]
>> Sent: Thursday, November 14, 2013 1:45
>> To: Joey Ye; jani...@codesourcery.com
>> Cc: gcc-patches@gcc.gnu.org; Ramana Radhakrishnan
>> Subject: Re: [patch] [arm] ARM Cortex-M3/M4 tuning
>>
>> 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.
>>
I got
ERROR: (DejaGnu) proc "{ scan-tree-dump-times "Threaded" 1 "vrp1" } ||
{ arm_cortex_m }" does not exist.
on Linux/x86-64.
--
H.J.
RE: [patch] [arm] ARM Cortex-M3/M4 tuning
In mainline and arm/embedded-4_8-branch now. > -Original Message- > From: Janis Johnson [mailto:janis_john...@mentor.com] > Sent: Thursday, November 14, 2013 1:45 > To: Joey Ye; jani...@codesourcery.com > Cc: gcc-patches@gcc.gnu.org; Ramana Radhakrishnan > Subject: Re: [patch] [arm] ARM Cortex-M3/M4 tuning > > 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 >
Re: [patch] [arm] ARM Cortex-M3/M4 tuning
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 >>>> Joey Ye >>>> >>>> * 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 >>>> >>>> * 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.
RE: [patch] [arm] ARM Cortex-M3/M4 tuning
Janis, can you please take a look at test case changes. Thanks, Joey > -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 > >> Joey Ye > >> > >> * 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 > >> > >> * 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. v7m_tune_allin1-131016.patch Description: Binary data
Re: [patch] [arm] ARM Cortex-M3/M4 tuning
ChangeLog: 2013-11-01 Julian Brown Joey Ye * 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 * 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.
RE: [patch] [arm] ARM Cortex-M3/M4 tuning
Ping > -Original Message- > From: Joey Ye [mailto:joey...@arm.com] > Sent: Friday, November 01, 2013 1:00 > To: gcc-patches@gcc.gnu.org > Subject: [patch] [arm] ARM Cortex-M3/M4 tuning > > Based on Julian's http://gcc.gnu.org/ml/gcc-patches/2012-07/msg01006.html > and > > * Merged with latest mainline and applied test > * Only apply to Cortex-M3 and M4 > * Set LOGICAL_OP_NON_SHORT_CIRCUIT to false > > Test: > - Coremark on M3/M4 gained 5% > - GCC make check with qemu Cortex-M3 resulting following expected > regressions, which will be fixed by following-up condition compare patches > * gcc.target/arm/thumb2-cond-cmp-1.c: Expected fail for now. > * gcc.target/arm/thumb2-cond-cmp-2.c: Likewise. > * gcc.target/arm/thumb2-cond-cmp-3.c: Likewise. > * gcc.target/arm/thumb2-cond-cmp-4.c: Likewise. > > ChangeLog: > > 2013-11-01 Julian Brown > Joey Ye > > * config/arm/arm.c (arm_cortex_m_branch_cost): New. > (arm_v7m_tune): New. > (arm_*_tune): Add comments for Sched adj cost. > * config/arm/arm-cores.def (cortex-m4, cortex-m3): > Use arm_v7m_tune tuning. > > testsuite: > 2013-11-01 Joey Ye > > * 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. v7m_tune_allin1-131016.patch Description: Binary data
