[Patch, cilk, C++] Fix cilk testsuite failure

Steve Ellcey sellcey@mips.com
Fri Jan 17 18:12:00 GMT 2014


On Fri, 2014-01-17 at 08:23 +0000, Richard Sandiford wrote:

> 
> I think it'd be more direct to check the register class, since we used
> to store CCmode in GPRs too.  I.e. ST_REGNO_P (XEXP (op, 0)).
> 
> OK with that change, thanks.  Please backport to 4.8 too.
> 
> Richard

I assume you meant ST_REG_P instead of ST_REGNO_P and it looks like
ST_REG_P wants a register number as an argument so the patch I tested
and checked in is:


diff --git a/gcc/config/mips/mips.c b/gcc/config/mips/mips.c
index 617391c..ff28750 100644
--- a/gcc/config/mips/mips.c
+++ b/gcc/config/mips/mips.c
@@ -8184,7 +8184,7 @@ mips_print_operand (FILE *file, rtx op, int letter)
     case 't':
       {
 	int truth = (code == NE) == (letter == 'T');
-	fputc ("zfnt"[truth * 2 + (GET_MODE (op) == CCmode)], file);
+	fputc ("zfnt"[truth * 2 + ST_REG_P (REGNO (XEXP (op, 0)))], file);
       }
       break;
 

If no issues come up over the weekend with this patch I will backport it
to the 4.8 branch next week.

Steve Ellcey
sellcey@mips.com




More information about the Gcc-patches mailing list