Patchwork [committed] Move MAX_BITS_PER_WORD into expmed.h

login
register
mail settings
Submitter Ramana Radhakrishnan
Date July 13, 2010, 9:48 a.m.
Message ID <1279014514.32159.54.camel@e102325-lin.cambridge.arm.com>
Download mbox | patch
Permalink /patch/58728/
State New
Headers show

Comments

Ramana Radhakrishnan - July 13, 2010, 9:48 a.m.
Hi,

This patch http://gcc.gnu.org/ml/gcc-patches/2010-07/msg00601.html broke
bootstrap on arm-linux-gnueabi because while it moved references to
MAX_BITS_PER_WORD into expmed.h , it didn't move it's definition up
there. 

Verified that bootstrap proceeds beyond building expmed.o and complete
bootstrap and test on arm-linux-gnueabi in progress. Committed.


Cheers
Ramana

2010-07-13  Ramana Radhakrishnan  <ramana.radhakrishnan@arm.com>

	* expmed.c (MAX_BITS_PER_WORD): Moved to expmed.h.
	* expmed.h (MAX_BITS_PER_WORD): Moved from expmed.c.
Steven Bosscher - July 13, 2010, 10 a.m.
On Tue, Jul 13, 2010 at 11:48 AM, Ramana Radhakrishnan
<ramana.radhakrishnan@arm.com> wrote:
> Hi,
>
> This patch http://gcc.gnu.org/ml/gcc-patches/2010-07/msg00601.html broke
> bootstrap on arm-linux-gnueabi because while it moved references to
> MAX_BITS_PER_WORD into expmed.h , it didn't move it's definition up
> there.
>
> Verified that bootstrap proceeds beyond building expmed.o and complete
> bootstrap and test on arm-linux-gnueabi in progress. Committed.

Thanks, this also fixes bootstrap on ia64-linux.

Ciao!
Steven

Patch

Index: gcc/expmed.c
===================================================================
--- gcc/expmed.c	(revision 162131)
+++ gcc/expmed.c	(working copy)
@@ -69,13 +69,6 @@  static rtx expand_sdiv_pow2 (enum machin
 #define SLOW_UNALIGNED_ACCESS(MODE, ALIGN) STRICT_ALIGNMENT
 #endif
 
-/* For compilers that support multiple targets with different word sizes,
-   MAX_BITS_PER_WORD contains the biggest value of BITS_PER_WORD.  An example
-   is the H8/300(H) compiler.  */
-
-#ifndef MAX_BITS_PER_WORD
-#define MAX_BITS_PER_WORD BITS_PER_WORD
-#endif
 
 /* Reduce conditional compilation elsewhere.  */
 #ifndef HAVE_insv
Index: gcc/expmed.h
===================================================================
--- gcc/expmed.h	(revision 162131)
+++ gcc/expmed.h	(working copy)
@@ -65,6 +65,14 @@  struct mult_cost {
 				 || ((X)->cost == (Y)->cost	\
 				     && (X)->latency < (Y)->latency))
 
+/* For compilers that support multiple targets with different word sizes,
+   MAX_BITS_PER_WORD contains the biggest value of BITS_PER_WORD.  An example
+   is the H8/300(H) compiler.  */
+
+#ifndef MAX_BITS_PER_WORD
+#define MAX_BITS_PER_WORD BITS_PER_WORD
+#endif
+
 /* This structure records a sequence of operations.
    `ops' is the number of operations recorded.
    `cost' is their total cost.