diff mbox

MIPS: Do not generate micromips code for the no-smartmips-lwxs.c testcase

Message ID FD3DCEAC5B03E9408544A1E416F112420192C3C0D4@NA-MBX-01.mgc.mentorg.com
State New
Headers show

Commit Message

Moore, Catherine July 6, 2015, 8:44 p.m. UTC 
> -----Original Message-----
> From: Andrew Bennett [mailto:Andrew.Bennett@imgtec.com]
> Sent: Monday, July 06, 2015 9:34 AM
> To: gcc-patches@gcc.gnu.org
> Cc: Matthew Fortune; Moore, Catherine
> Subject: [PATCH] MIPS: Do not generate micromips code for the no-
> smartmips-lwxs.c testcase
> 
> Hi,
> 
> The LWXS instruction is part of the micromips ISA which means it is valid to
> generate it for the no-smartmips-lwxs.c testcase.  I have updated the dg-
> options for the test to ensure that it does not generate micromips code.
> 
> I have tested this on the mips-mti-elf target using mips32r2/{-mno-
> micromips/-mmicromips} test options and there are no new regressions.
> 
> The patch and ChangeLog are below.
> 
> Ok to commit?
> 
> 
> 
> Many thanks,
> 
> 
> Andrew
> 
> 
> 
> testsuite/
> 	* gcc.target/mips/no-smartmips-lwxs.c: Add -mno-micromips to dg-
> options.
> 
> 
> diff --git a/gcc/testsuite/gcc.target/mips/no-smartmips-lwxs.c
> b/gcc/testsuite/gcc.target/mips/no-smartmips-lwxs.c
> index ecf856e..6701a1c 100644
> --- a/gcc/testsuite/gcc.target/mips/no-smartmips-lwxs.c
> +++ b/gcc/testsuite/gcc.target/mips/no-smartmips-lwxs.c
> @@ -1,5 +1,5 @@
>  /* { dg-do compile } */
> -/* { dg-options "-mno-smartmips" } */
> +/* { dg-options "-mno-smartmips -mno-micromips" } */
> 
>  NOMIPS16 int scaled_indexed_word_load (int a[], int b)  {
> 

Hi Andrew,

Instead of adding the -mno-micromips option to dg-options, please change the MIPS16 attribute to NOCOMPRESSION.


OK with that change.
Catherine

Comments

Andrew Bennett July 7, 2015, 4:08 p.m. UTC | #1 
> Hi Andrew,
> 
> Instead of adding the -mno-micromips option to dg-options, please change the
> MIPS16 attribute to NOCOMPRESSION.
> 
> Index: gcc.target/mips/no-smartmips-lwxs.c
> ===================================================================
> --- gcc.target/mips/no-smartmips-lwxs.c (revision 452061)
> +++ gcc.target/mips/no-smartmips-lwxs.c (working copy)
> @@ -1,7 +1,7 @@
>  /* { dg-do compile } */
>  /* { dg-options "-mno-smartmips" } */
> 
> -NOMIPS16 int scaled_indexed_word_load (int a[], int b)
> +NOCOMPRESSION int scaled_indexed_word_load (int a[], int b)
>  {
>    return a[b];
>  }
> 
> OK with that change.
> Catherine

Committed as SVN 225519. 

Regards,



Andrew
diff mbox

Patch

Index: gcc.target/mips/no-smartmips-lwxs.c
===================================================================
--- gcc.target/mips/no-smartmips-lwxs.c (revision 452061)
+++ gcc.target/mips/no-smartmips-lwxs.c (working copy)
@@ -1,7 +1,7 @@ 
 /* { dg-do compile } */
 /* { dg-options "-mno-smartmips" } */

-NOMIPS16 int scaled_indexed_word_load (int a[], int b)
+NOCOMPRESSION int scaled_indexed_word_load (int a[], int b)
 {
   return a[b];
 }