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: 3.3.4 status, and some questions


On Fri, 2004-03-12 at 17:12, Joe Buck wrote:
> OK.  I just filed it as target/14558.  Could you please mark your
> backport as closing this bug?

Here's what I checked in.

p.

2004-03-12  Philip Blundell  <philb@gnu.org>

	PR target/14558
	Backport from trunk:
	2004-02-25  Richard Earnshaw  <rearnsha@arm.com>
	* arm.h (ARM_GO_IF_LEGITIMATE_INDEX): For QImode the range of
	an offset is -4095...+4095 inclusive.

Index: config/arm/arm.h
===================================================================
RCS file: /cvs/gcc/gcc/gcc/config/arm/arm.h,v
retrieving revision 1.168.2.2
diff -u -r1.168.2.2 arm.h
--- config/arm/arm.h	26 Feb 2004 14:10:01 -0000	1.168.2.2
+++ config/arm/arm.h	12 Mar 2004 19:19:08 -0000
@@ -2003,10 +2003,17 @@
 		  && INTVAL (op) <= 31)					\
 		goto LABEL;						\
 	    }								\
-	  /* NASTY: Since this limits the addressing of unsigned	\
-	     byte loads.  */						\
-	  range = ((MODE) == HImode || (MODE) == QImode)		\
-	    ? (arm_arch4 ? 256 : 4095) : 4096;				\
+  	  /* XXX For ARM v4 we may be doing a sign-extend operation	\
+	     during the load, but that has a restricted addressing	\
+	     range and we are unable to tell here whether that is the	\
+	     case.  To be safe we restrict all loads to that		\
+	     range.  */							\
+          if (arm_arch4)						\
+	    range = (mode == HImode || mode == QImode) ? 256 : 4096;	\
+	  else if (mode == HImode)					\
+	    range = 4095;						\
+	  else								\
+	    range = 4096;						\
 	  if (code == CONST_INT && INTVAL (INDEX) < range		\
 	      && INTVAL (INDEX) > -range)				\
 	    goto LABEL;							\

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