This is the mail archive of the gcc-patches@gcc.gnu.org mailing list for the GCC project.

Index Nav:	[Date Index] [Subject Index] [Author Index] [Thread Index]
Message Nav:	[Date Prev] [Date Next]	[Thread Prev] [Thread Next]
Other format:	[Raw text]

Re: [patch] Performance patch for MIPS conditional move in expr.c

From: Richard Sandiford <rdsandiford at googlemail dot com>
To: Andrew Pinski <andrew dot pinski at caviumnetworks dot com>
Cc: Steve Ellcey <sellcey at mips dot com>, gcc-patches at gcc dot gnu dot org
Date: Thu, 15 Nov 2012 20:58:52 +0000
Subject: Re: [patch] Performance patch for MIPS conditional move in expr.c
References: <27d19005-b82d-4ecc-a81e-14208937ce0f@EXCHHUB01.MIPS.com> <CA+=Sn1k=eD2=Suc5VwoNkvToKEoyL_cuwmsh4iDjoGQCMa1P9w@mail.gmail.com> <1352921235.22862.6.camel@ubuntu-sellcey> <CA+=Sn1nn=PSNWVLEHLKwaqsyDPDYtNv4JudLFmCi4=E4+jTYXQ@mail.gmail.com> <1352929552.22862.13.camel@ubuntu-sellcey> <CA+=Sn1n6RTs4GpMF-h_Hj5XP3mDqSghHyfSYZZ_WmWzQxCFxUQ@mail.gmail.com> <CA+=Sn1nNxA+D1GQAbG84YWKmZc8eu=MbD91aiJHTmy2gT4S_Vw@mail.gmail.com>

Andrew Pinski <andrew.pinski@caviumnetworks.com> writes:
>     2012-07-26  Andrew Pinski  <apinski@cavium.com>
>     
>     	Bug #3261
>     	* config/mips/mips.md (*mov<GPR:mode>_on_<MOVECC:mode>):
>     	Remove mode check from comparisons.
>     	(*mov<SCALARF:mode>_on_<MOVECC:mode>): Likewise.
>     	(*mov<GPR:mode>_on_<GPR2:mode>_ne): New pattern to match
>     	when (ne A 0) can be just A.
>     
>     	* testsuite/gcc.target/mips/movcc-4.c: New testcase.

OK, thanks (but remember to remove the internal bug reference :-)).
I think this is early enough during stage 3 for the usual target
flexibility to apply.

Richard

Follow-Ups:
- Re: [patch] Performance patch for MIPS conditional move in expr.c
  - From: Andrew Pinski

References:
- [patch] Performance patch for MIPS conditional move in expr.c
  - From: Steve Ellcey
- Re: [patch] Performance patch for MIPS conditional move in expr.c
  - From: Andrew Pinski
- Re: [patch] Performance patch for MIPS conditional move in expr.c
  - From: Steve Ellcey
- Re: [patch] Performance patch for MIPS conditional move in expr.c
  - From: Andrew Pinski
- Re: [patch] Performance patch for MIPS conditional move in expr.c
  - From: Steve Ellcey
- Re: [patch] Performance patch for MIPS conditional move in expr.c
  - From: Andrew Pinski
- Re: [patch] Performance patch for MIPS conditional move in expr.c
  - From: Andrew Pinski

Index Nav:	[Date Index] [Subject Index] [Author Index] [Thread Index]
Message Nav:	[Date Prev] [Date Next]	[Thread Prev] [Thread Next]