Re: THUMB interlibrary linking may be broken

[Nick Clifton]

Hi Fred,

> Is this a bug, warning about an unimplemented feature, or something 
> that simply isn't supported?

It appears to be a case of a missing case. :-)

The patch below should fix this.  (Let me know if it doesn't).  I will
be checking it in a moment.

Cheers
        Nick

PS.  Thanks for a nice, small, reproducible test case.  I wish all bug
reports could be like this.


2001-10-12  Nick Clifton  <address@hidden>

        * elf32-arm.h (elf32_arm_relocate_section): Treat R_ARM_THM_PC22
        like R_ARM_PC24.  Include reloc number in error message about
        unresolvable relocs.

Index: bfd/elf32-arm.h
===================================================================
RCS file: /cvs/src/src/bfd/elf32-arm.h,v
retrieving revision 1.61
diff -p -r1.61 elf32-arm.h
*** elf32-arm.h 2001/09/27 13:49:27     1.61
--- elf32-arm.h 2001/10/12 11:11:32
*************** elf32_arm_relocate_section (output_bfd, 
*** 1883,1888 ****
--- 1883,1889 ----
                {
                case R_ARM_PC24:
                case R_ARM_ABS32:
+               case R_ARM_THM_PC22:
                  if (info->shared
                      && (
                  (!info->symbolic && h->dynindx != -1)
*************** elf32_arm_relocate_section (output_bfd, 
*** 1923,1930 ****
                  if (sec->output_section == NULL)
                    {
                      (*_bfd_error_handler)
!                       (_("%s: warning: unresolvable relocation against symbol 
`%s' from %s section"),
!                        bfd_archive_filename (input_bfd), h->root.root.string,
                         bfd_get_section_name (input_bfd, input_section));
                      relocation_needed = 0;
                    }
--- 1924,1933 ----
                  if (sec->output_section == NULL)
                    {
                      (*_bfd_error_handler)
!                       (_("%s: warning: unresolvable relocation %d against 
symbol `%s' from %s section"),
!                        bfd_archive_filename (input_bfd),
!                        r_type,
!                        h->root.root.string,
                         bfd_get_section_name (input_bfd, input_section));
                      relocation_needed = 0;
                    }