Use int and bfd_signed_vma in x32 addend overflow check

bfd/

	* elf64-x86-64.c (elf_x86_64_relocate_section): Use int in x32
	addend overflow check.

gas/

	* config/tc-i386.c (tc_gen_reloc): Use bfd_signed_vma in x32
	addend overflow check.

diff --git a/bfd/ChangeLog b/bfd/ChangeLog
index 806db7f..e776f41 100644 (file)
--- a/bfd/ChangeLog
+++ b/bfd/ChangeLog
@@ -1,3 +1,8 @@
+2012-05-10  H.J. Lu  <hongjiu.lu@intel.com>
+
+       * elf64-x86-64.c (elf_x86_64_relocate_section): Use int in x32
+       addend overflow check.
+
 2012-05-10  DJ Delorie  <dj@redhat.com>
 
        * elf32-rx.c (rx_elf_object_p): Ignore empty segments.

diff --git a/bfd/elf64-x86-64.c b/bfd/elf64-x86-64.c
index e98c325..1111d77 100644 (file)
--- a/bfd/elf64-x86-64.c
+++ b/bfd/elf64-x86-64.c
@@ -3686,7 +3686,7 @@ elf_x86_64_relocate_section (bfd *output_bfd,
                          != (rel->r_addend & 0x80000000))
                        {
                          const char *name;
-                         long addend = rel->r_addend;
+                         int addend = rel->r_addend;
                          if (h && h->root.root.string)
                            name = h->root.root.string;
                          else
@@ -3694,7 +3694,7 @@ elf_x86_64_relocate_section (bfd *output_bfd,
                                                     sym, NULL);
                          if (addend < 0)
                            (*_bfd_error_handler)
-                             (_("%B: addend -0x%lx in relocation %s against "
+                             (_("%B: addend -0x%x in relocation %s against "
                                 "symbol `%s' at 0x%lx in section `%A' is "
                                 "out of range"),
                               input_bfd, input_section, addend,
@@ -3702,7 +3702,7 @@ elf_x86_64_relocate_section (bfd *output_bfd,
                               name, (unsigned long) rel->r_offset);
                          else
                            (*_bfd_error_handler)
-                             (_("%B: addend 0x%lx in relocation %s against "
+                             (_("%B: addend 0x%x in relocation %s against "
                                 "symbol `%s' at 0x%lx in section `%A' is "
                                 "out of range"),
                               input_bfd, input_section, addend,

diff --git a/gas/ChangeLog b/gas/ChangeLog
index dfe45d2..c9048e3 100644 (file)
--- a/gas/ChangeLog
+++ b/gas/ChangeLog
@@ -1,5 +1,10 @@
 2012-05-10  H.J. Lu  <hongjiu.lu@intel.com>
 
+       * config/tc-i386.c (tc_gen_reloc): Use bfd_signed_vma in x32
+       addend overflow check.
+
+2012-05-10  H.J. Lu  <hongjiu.lu@intel.com>
+
        * config/tc-i386.c (tc_gen_reloc): Display signed hex number in
        x32 addend overflow check.
 

diff --git a/gas/config/tc-i386.c b/gas/config/tc-i386.c
index e3440f0..e33fb6c 100644 (file)
--- a/gas/config/tc-i386.c
+++ b/gas/config/tc-i386.c
@@ -9177,16 +9177,18 @@ tc_gen_reloc (asection *section ATTRIBUTE_UNUSED, fixS *fixp)
            /* Check addend overflow.  */
            if (!fits_in_signed_long (fixp->fx_offset))
              {
-               long long addend = fixp->fx_offset;
+               bfd_signed_vma addend = fixp->fx_offset;
                if (addend < 0)
                  as_bad_where (fixp->fx_file, fixp->fx_line,
                                _("cannot represent relocation %s with "
-                                 "addend -0x%llx in x32 mode"),
+                                 "addend -0x%" BFD_VMA_FMT "x in x32 "
+                                 "mode"),
                                bfd_get_reloc_code_name (code), -addend);
                else
                  as_bad_where (fixp->fx_file, fixp->fx_line,
                                _("cannot represent relocation %s with "
-                                 "addend 0x%llx in x32 mode"),
+                                 "addend 0x%" BFD_VMA_FMT "x in x32 "
+                                 "mode"),
                                bfd_get_reloc_code_name (code), addend);
              }
            break;