diff mbox

[MIPS] Enable load/store bonding for I6400

Message ID B5E67142681B53468FAF6B7C31356562441AEC41@hhmail02.hh.imgtec.org
State New
Headers show

Commit Message

Robert Suchanek Aug. 5, 2015, 8:30 a.m. UTC 
Hi,

Following up 
https://gcc.gnu.org/ml/gcc-patches/2015-07/msg01730.html

The patch below enables the load-load/store-store bonding for MIPS32/MIPS64 R6.

Ok to apply?

Regards,
Robert

gcc/
	* config/mips/mips.h (ENABLE_LD_ST_PAIRS): Enable load/store pairs for
	I6400.
---
 gcc/config/mips/mips.h | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

Comments

Robert Suchanek Aug. 13, 2015, 7:46 a.m. UTC | #1 
Ping.

> -----Original Message-----
> From: gcc-patches-owner@gcc.gnu.org [mailto:gcc-patches-owner@gcc.gnu.org] On
> Behalf Of Robert Suchanek
> Sent: 05 August 2015 09:31
> To: Catherine_Moore@mentor.com; Matthew Fortune; gcc-patches@gcc.gnu.org
> Subject: [PATCH, MIPS] Enable load/store bonding for I6400
> 
> Hi,
> 
> Following up
> https://gcc.gnu.org/ml/gcc-patches/2015-07/msg01730.html
> 
> The patch below enables the load-load/store-store bonding for MIPS32/MIPS64
> R6.
> 
> Ok to apply?
> 
> Regards,
> Robert
> 
> gcc/
> 	* config/mips/mips.h (ENABLE_LD_ST_PAIRS): Enable load/store pairs for
> 	I6400.
> ---
>  gcc/config/mips/mips.h | 2 +-
>  1 file changed, 1 insertion(+), 1 deletion(-)
> 
> diff --git a/gcc/config/mips/mips.h b/gcc/config/mips/mips.h
> index d17a833..6e262d6 100644
> --- a/gcc/config/mips/mips.h
> +++ b/gcc/config/mips/mips.h
> @@ -3177,5 +3177,5 @@ extern GTY(()) struct target_globals
> *micromips_globals;
>     performance can be degraded for those targets.  Hence, do not bond for
>     micromips or fix_24k.  */
>  #define ENABLE_LD_ST_PAIRS \
> -  (TARGET_LOAD_STORE_PAIRS && TUNE_P5600 \
> +  (TARGET_LOAD_STORE_PAIRS && (TUNE_P5600 || TUNE_I6400) \
>     && !TARGET_MICROMIPS && !TARGET_FIX_24K)
> --
> 2.4.5
Matthew Fortune Aug. 13, 2015, 10:16 a.m. UTC | #2 
> > gcc/
> > 	* config/mips/mips.h (ENABLE_LD_ST_PAIRS): Enable load/store pairs for
> > 	I6400.

Sorry, I missed this one. OK to commit.

Thanks,
Matthew
Robert Suchanek Aug. 13, 2015, 11:46 a.m. UTC | #3 
Hi,

> > > gcc/
> > > 	* config/mips/mips.h (ENABLE_LD_ST_PAIRS): Enable load/store pairs for
> > > 	I6400.
> 
> Sorry, I missed this one. OK to commit.
> 
> Thanks,
> Matthew

Committed as r226860.

Regards,
Robert
diff mbox

Patch

diff --git a/gcc/config/mips/mips.h b/gcc/config/mips/mips.h
index d17a833..6e262d6 100644
--- a/gcc/config/mips/mips.h
+++ b/gcc/config/mips/mips.h
@@ -3177,5 +3177,5 @@  extern GTY(()) struct target_globals *micromips_globals;
    performance can be degraded for those targets.  Hence, do not bond for
    micromips or fix_24k.  */
 #define ENABLE_LD_ST_PAIRS \
-  (TARGET_LOAD_STORE_PAIRS && TUNE_P5600 \
+  (TARGET_LOAD_STORE_PAIRS && (TUNE_P5600 || TUNE_I6400) \
    && !TARGET_MICROMIPS && !TARGET_FIX_24K)