This is the mail archive of the
gcc-patches@gcc.gnu.org
mailing list for the GCC project.
Re: Patch: fix to sparc.c:arith_4096_operand
- From: "Kaveh R. Ghazi" <ghazi at caip dot rutgers dot edu>
- To: jakub at redhat dot com
- Cc: aoliva at redhat dot com, gcc-patches at gcc dot gnu dot org
- Date: Fri, 8 Mar 2002 20:31:57 -0500 (EST)
- Subject: Re: Patch: fix to sparc.c:arith_4096_operand
> From: Jakub Jelinek <jakub@redhat.com>
>
> Use
> return INTVAL (op) == 4096;
> instead.
> Ok with that change.
> Jakub
Done, thanks.
Index: ChangeLog
===================================================================
RCS file: /cvs/gcc/gcc/gcc/ChangeLog,v
retrieving revision 1.13281
diff -u -p -r1.13281 ChangeLog
--- ChangeLog 2002/03/09 01:13:22 1.13281
+++ ChangeLog 2002/03/09 01:29:11
@@ -1,3 +1,7 @@
+2002-03-08 Kaveh R. Ghazi <ghazi@caip.rutgers.edu>
+
+ * sparc.c (arith_4096_operand): Fix error in last change.
+
2002-03-08 Alexandre Oliva <aoliva@redhat.com>
* config/mips/mips.h (SUBTARGET_CPP_SIZE_SPEC): Remove duplicate
Index: config/sparc/sparc.c
===================================================================
RCS file: /cvs/gcc/gcc/gcc/config/sparc/sparc.c,v
retrieving revision 1.189
diff -u -p -r1.189 sparc.c
--- sparc.c 2002/03/03 15:37:31 1.189
+++ sparc.c 2002/03/09 01:29:12
@@ -964,10 +964,10 @@ arith_4096_operand (op, mode)
rtx op;
enum machine_mode mode ATTRIBUTE_UNUSED;
{
- int val;
if (GET_CODE (op) != CONST_INT)
return 0;
- return val == 4096;
+ else
+ return INTVAL (op) == 4096;
}
/* Return true if OP is suitable as second operand for add/sub */