Fix compile time warnings from a GCC 4.0 compiler

diff --git a/bfd/ChangeLog b/bfd/ChangeLog
index e7cdd85..67a8e79 100644 (file)
--- a/bfd/ChangeLog
+++ b/bfd/ChangeLog
@@ -1,3 +1,7 @@
+2005-07-05  Nick Clifton  <nickc@redhat.com>
+
+       * elf64-ppc.c (ppc64_elf_info_to_howto): Fix typo.
+
 2005-05-07  Paul Brook  <paul@codesourcery.com>
 
        * config.bfd: Add separate case for ppc-vxworks.

diff --git a/bfd/elf64-ppc.c b/bfd/elf64-ppc.c
index 656e822..f05262b 100644 (file)
--- a/bfd/elf64-ppc.c
+++ b/bfd/elf64-ppc.c
@@ -2121,8 +2121,8 @@ ppc64_elf_info_to_howto (bfd *abfd ATTRIBUTE_UNUSED, arelent *cache_ptr,
               / sizeof (ppc64_elf_howto_table[0])))
     {
       (*_bfd_error_handler) (_("%B: invalid relocation type %d"),
-                            abfd, (int) r_type);
-      r_type = R_PPC64_NONE;
+                            abfd, (int) type);
+      type = R_PPC64_NONE;
     }
   cache_ptr->howto = ppc64_elf_howto_table[type];
 }

diff --git a/cpu/ChangeLog b/cpu/ChangeLog
index 0186195..7facd5a 100644 (file)
--- a/cpu/ChangeLog
+++ b/cpu/ChangeLog
@@ -1,3 +1,13 @@
+2005-07-05  Nick Clifton  <nickc@redhat.com>
+
+       * iq2000.opc (parse_lo16, parse_mlo16): Make value parameter
+       unsigned in order to avoid compile time warnings about sign
+       conflicts.
+
+       * ms1.opc (parse_*): Likewise.
+       (parse_imm16): Use a "void *" as it is passed both signed and
+       unsigned arguments.
+
 2005-07-01  Nick Clifton  <nickc@redhat.com>
 
        * frv.opc: Update to ISO C90 function declaration style.

diff --git a/cpu/iq2000.opc b/cpu/iq2000.opc
index 5da6911..63ef076 100644 (file)
--- a/cpu/iq2000.opc
+++ b/cpu/iq2000.opc
@@ -259,7 +259,7 @@ static const char *
 parse_lo16 (CGEN_CPU_DESC cd,
            const char **strp,
            int opindex,
-           long *valuep)
+           unsigned long *valuep)
 {
   if (strncasecmp (*strp, "%lo(", 4) == 0)
     {
@@ -291,7 +291,7 @@ static const char *
 parse_mlo16 (CGEN_CPU_DESC cd,
             const char **strp,
             int opindex,
-            long *valuep)
+            unsigned long *valuep)
 {
   if (strncasecmp (*strp, "%lo(", 4) == 0)
     {

diff --git a/cpu/ms1.opc b/cpu/ms1.opc
index 57c2913..f61de6c 100644 (file)
--- a/cpu/ms1.opc
+++ b/cpu/ms1.opc
@@ -89,16 +89,6 @@ ms1_asm_hash (const char* insn)
 
 \f
 /* -- asm.c */
-static int signed_out_of_bounds (long);
-static const char * parse_imm16 (CGEN_CPU_DESC, const char **, int, long *);
-static const char * parse_dup   (CGEN_CPU_DESC, const char **, int, long *);
-static const char * parse_ball  (CGEN_CPU_DESC, const char **, int, long *);
-static const char * parse_xmode (CGEN_CPU_DESC, const char **, int, long *);
-static const char * parse_rc    (CGEN_CPU_DESC, const char **, int, long *);
-static const char * parse_cbrb  (CGEN_CPU_DESC, const char **, int, long *);
-static const char * parse_rbbc  (CGEN_CPU_DESC, const char **, int, long *);
-static const char * parse_type  (CGEN_CPU_DESC, const char **, int, long *);
-
 /* Range checking for signed numbers.  Returns 0 if acceptable
    and 1 if the value is out of bounds for a signed quantity.  */
 
@@ -114,8 +104,9 @@ static const char *
 parse_imm16 (CGEN_CPU_DESC cd,
             const char **strp,
             int opindex,
-            long *valuep)
+            void *arg)
 {
+  signed long * valuep = (signed long *) arg;
   const char *errmsg;
   enum cgen_parse_operand_result result_type;
   bfd_reloc_code_real_type code = BFD_RELOC_NONE;
@@ -218,9 +209,9 @@ parse_imm16 (CGEN_CPU_DESC cd,
       else  
        {
           /* MS1_OPERAND_IMM16Z.  Parse as an unsigned integer.  */
-          errmsg = cgen_parse_unsigned_integer (cd, strp, opindex, valuep);
+          errmsg = cgen_parse_unsigned_integer (cd, strp, opindex, (unsigned long *) valuep);
 
-         if (opindex == (CGEN_OPERAND_TYPE)MS1_OPERAND_IMM16
+         if (opindex == (CGEN_OPERAND_TYPE) MS1_OPERAND_IMM16
              && *valuep >= 0x8000
              && *valuep <= 0xffff)
            *valuep -= 0x10000;
@@ -235,7 +226,7 @@ static const char *
 parse_dup (CGEN_CPU_DESC cd,
           const char **strp,
           int opindex,
-          long *valuep)
+          unsigned long *valuep)
 {
   const char *errmsg = NULL;
 
@@ -260,7 +251,7 @@ static const char *
 parse_ball (CGEN_CPU_DESC cd,
            const char **strp,
            int opindex,
-           long *valuep)
+           unsigned long *valuep)
 {
   const char *errmsg = NULL;
 
@@ -284,7 +275,7 @@ static const char *
 parse_xmode (CGEN_CPU_DESC cd,
             const char **strp,
             int opindex,
-            long *valuep)
+            unsigned long *valuep)
 {
   const char *errmsg = NULL;
 
@@ -308,7 +299,7 @@ static const char *
 parse_rc (CGEN_CPU_DESC cd,
          const char **strp,
          int opindex,
-         long *valuep)
+         unsigned long *valuep)
 {
   const char *errmsg = NULL;
 
@@ -332,7 +323,7 @@ static const char *
 parse_cbrb (CGEN_CPU_DESC cd,
            const char **strp,
            int opindex,
-           long *valuep)
+           unsigned long *valuep)
 {
   const char *errmsg = NULL;
 
@@ -356,7 +347,7 @@ static const char *
 parse_rbbc (CGEN_CPU_DESC cd,
            const char **strp,
            int opindex,
-           long *valuep)
+           unsigned long *valuep)
 {
   const char *errmsg = NULL;
 
@@ -390,7 +381,7 @@ static const char *
 parse_type (CGEN_CPU_DESC cd,
            const char **strp,
            int opindex,
-           long *valuep)
+           unsigned long *valuep)
 {
   const char *errmsg = NULL;
 

diff --git a/gas/ChangeLog b/gas/ChangeLog
index 80a8b7f..c34b4e2 100644 (file)
--- a/gas/ChangeLog
+++ b/gas/ChangeLog
@@ -1,3 +1,9 @@
+2005-07-05  Nick Clifton  <nickc@redhat.com>
+
+       * config/tc-pdp11.c (md_apply_fix): Cast first argument to
+       md_chars_to_numbers to an unsigned pointer in order to avoid a
+       compile time warning.
+
 2005-07-05  Paul Brook  <paul@codesourcery.com>
 
        * config/tc-ppc.c (ppc_target_format): Add VxWorks.

diff --git a/gas/config/tc-pdp11.c b/gas/config/tc-pdp11.c
index a0057f1..cf103df 100644 (file)
--- a/gas/config/tc-pdp11.c
+++ b/gas/config/tc-pdp11.c
@@ -243,7 +243,7 @@ md_apply_fix (fixS *fixP,
 
   buf = fixP->fx_where + fixP->fx_frag->fr_literal;
   size = fixP->fx_size;
-  code = md_chars_to_number (buf, size);
+  code = md_chars_to_number ((unsigned char *) buf, size);
 
   switch (fixP->fx_r_type)
     {

diff --git a/opcodes/ChangeLog b/opcodes/ChangeLog
index f78ecf2..87b3942 100644 (file)
--- a/opcodes/ChangeLog
+++ b/opcodes/ChangeLog
@@ -1,3 +1,8 @@
+2005-07-05  Nick Clifton  <nickc@redhat.com>
+
+       * iq2000-asm.c: Regenerate.
+       * ms1-asm.c: Regenerate.
+
 2005-07-05  Jan Beulich  <jbeulich@novell.com>
 
        * i386-dis.c (SVME_Fixup): New.

diff --git a/opcodes/iq2000-asm.c b/opcodes/iq2000-asm.c
index 0bbb4ae..caab54f 100644 (file)
--- a/opcodes/iq2000-asm.c
+++ b/opcodes/iq2000-asm.c
@@ -255,7 +255,7 @@ static const char *
 parse_lo16 (CGEN_CPU_DESC cd,
            const char **strp,
            int opindex,
-           long *valuep)
+           unsigned long *valuep)
 {
   if (strncasecmp (*strp, "%lo(", 4) == 0)
     {
@@ -287,7 +287,7 @@ static const char *
 parse_mlo16 (CGEN_CPU_DESC cd,
             const char **strp,
             int opindex,
-            long *valuep)
+            unsigned long *valuep)
 {
   if (strncasecmp (*strp, "%lo(", 4) == 0)
     {

diff --git a/opcodes/ms1-asm.c b/opcodes/ms1-asm.c
index 7647c1b..528a0d5 100644 (file)
--- a/opcodes/ms1-asm.c
+++ b/opcodes/ms1-asm.c
@@ -49,16 +49,6 @@ static const char * parse_insn_normal
 /* -- assembler routines inserted here.  */
 
 /* -- asm.c */
-static int signed_out_of_bounds (long);
-static const char * parse_imm16 (CGEN_CPU_DESC, const char **, int, long *);
-static const char * parse_dup   (CGEN_CPU_DESC, const char **, int, long *);
-static const char * parse_ball  (CGEN_CPU_DESC, const char **, int, long *);
-static const char * parse_xmode (CGEN_CPU_DESC, const char **, int, long *);
-static const char * parse_rc    (CGEN_CPU_DESC, const char **, int, long *);
-static const char * parse_cbrb  (CGEN_CPU_DESC, const char **, int, long *);
-static const char * parse_rbbc  (CGEN_CPU_DESC, const char **, int, long *);
-static const char * parse_type  (CGEN_CPU_DESC, const char **, int, long *);
-
 /* Range checking for signed numbers.  Returns 0 if acceptable
    and 1 if the value is out of bounds for a signed quantity.  */
 
@@ -74,8 +64,9 @@ static const char *
 parse_imm16 (CGEN_CPU_DESC cd,
             const char **strp,
             int opindex,
-            long *valuep)
+            void *arg)
 {
+  signed long * valuep = (signed long *) arg;
   const char *errmsg;
   enum cgen_parse_operand_result result_type;
   bfd_reloc_code_real_type code = BFD_RELOC_NONE;
@@ -178,9 +169,9 @@ parse_imm16 (CGEN_CPU_DESC cd,
       else  
        {
           /* MS1_OPERAND_IMM16Z.  Parse as an unsigned integer.  */
-          errmsg = cgen_parse_unsigned_integer (cd, strp, opindex, valuep);
+          errmsg = cgen_parse_unsigned_integer (cd, strp, opindex, (unsigned long *) valuep);
 
-         if (opindex == (CGEN_OPERAND_TYPE)MS1_OPERAND_IMM16
+         if (opindex == (CGEN_OPERAND_TYPE) MS1_OPERAND_IMM16
              && *valuep >= 0x8000
              && *valuep <= 0xffff)
            *valuep -= 0x10000;
@@ -195,7 +186,7 @@ static const char *
 parse_dup (CGEN_CPU_DESC cd,
           const char **strp,
           int opindex,
-          long *valuep)
+          unsigned long *valuep)
 {
   const char *errmsg = NULL;
 
@@ -220,7 +211,7 @@ static const char *
 parse_ball (CGEN_CPU_DESC cd,
            const char **strp,
            int opindex,
-           long *valuep)
+           unsigned long *valuep)
 {
   const char *errmsg = NULL;
 
@@ -244,7 +235,7 @@ static const char *
 parse_xmode (CGEN_CPU_DESC cd,
             const char **strp,
             int opindex,
-            long *valuep)
+            unsigned long *valuep)
 {
   const char *errmsg = NULL;
 
@@ -268,7 +259,7 @@ static const char *
 parse_rc (CGEN_CPU_DESC cd,
          const char **strp,
          int opindex,
-         long *valuep)
+         unsigned long *valuep)
 {
   const char *errmsg = NULL;
 
@@ -292,7 +283,7 @@ static const char *
 parse_cbrb (CGEN_CPU_DESC cd,
            const char **strp,
            int opindex,
-           long *valuep)
+           unsigned long *valuep)
 {
   const char *errmsg = NULL;
 
@@ -316,7 +307,7 @@ static const char *
 parse_rbbc (CGEN_CPU_DESC cd,
            const char **strp,
            int opindex,
-           long *valuep)
+           unsigned long *valuep)
 {
   const char *errmsg = NULL;
 
@@ -350,7 +341,7 @@ static const char *
 parse_type (CGEN_CPU_DESC cd,
            const char **strp,
            int opindex,
-           long *valuep)
+           unsigned long *valuep)
 {
   const char *errmsg = NULL;