[committed] Move MAX_BITS_PER_WORD into expmed.h

Message ID	1279014514.32159.54.camel@e102325-lin.cambridge.arm.com
State	New
Headers	show Return-Path: <gcc-patches-return-268037-incoming=patchwork.ozlabs.org@gcc.gnu.org> Subject: [Patch committed] Move MAX_BITS_PER_WORD into expmed.h From: Ramana Radhakrishnan <ramana.radhakrishnan@arm.com> Reply-To: ramana.radhakrishnan@arm.com To: gcc-patches@gcc.gnu.org Cc: rsandifo@gcc.gnu.org Content-Type: multipart/mixed; boundary="=-EpkKOsXD5ITSE4qViGwz" Date: Tue, 13 Jul 2010 10:48:34 +0100 Message-Id: <1279014514.32159.54.camel@e102325-lin.cambridge.arm.com> Mime-Version: 1.0 Mailing-List: contact gcc-patches-help@gcc.gnu.org; run by ezmlm Precedence: bulk Sender: gcc-patches-owner@gcc.gnu.org

Message ID

1279014514.32159.54.camel@e102325-lin.cambridge.arm.com

State

New

Headers

Subject: [Patch committed] Move MAX_BITS_PER_WORD into expmed.h
From: Ramana Radhakrishnan <ramana.radhakrishnan@arm.com>
Reply-To: ramana.radhakrishnan@arm.com
To: gcc-patches@gcc.gnu.org
Cc: rsandifo@gcc.gnu.org
Content-Type: multipart/mixed; boundary="=-EpkKOsXD5ITSE4qViGwz"
Date: Tue, 13 Jul 2010 10:48:34 +0100
Message-Id: <1279014514.32159.54.camel@e102325-lin.cambridge.arm.com>
Mime-Version: 1.0
Mailing-List: contact gcc-patches-help@gcc.gnu.org; run by ezmlm
Precedence: bulk
Sender: gcc-patches-owner@gcc.gnu.org

Commit Message

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

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.

Comments

Steven Bosscher July 13, 2010, 10 a.m. UTC | #1

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

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.