* stabs.texinfo: Clean up stuff about visibility and virtual
[platform/upstream/binutils.git] / bfd / reloc.c
index bfa7c85..0f05cbe 100644 (file)
@@ -22,18 +22,18 @@ Foundation, Inc., 675 Mass Ave, Cambridge, MA 02139, USA.  */
 SECTION
        Relocations
 
-       BFD maintains relocations in much the same was as it maintains
-       symbols; they are left alone until required, then read in
-       en-mass and traslated into an internal form. There is a common
-       routine <<bfd_perform_relocation>> which acts upon the
-       canonical form to to the actual fixup.
+       BFD maintains relocations in much the same way it maintains
+       symbols: they are left alone until required, then read in
+       en-mass and translated into an internal form.  A common
+       routine <<bfd_perform_relocation>> acts upon the
+       canonical form to do the fixup.
 
-       Note that relocations are maintained on a per section basis,
-       whilst symbols are maintained on a per BFD basis.
+       Relocations are maintained on a per section basis,
+       while symbols are maintained on a per BFD basis.
 
-       All a back end has to do to fit the BFD interface is to create
-       as many <<struct reloc_cache_entry>> as there are relocations
-       in a particular section, and fill in the right bits:
+       All that a back end has to do to fit the BFD interface is to create
+       a <<struct reloc_cache_entry>> for each relocation
+       in a particular section, and fill in the right bits of the structures.
 
 @menu
 @* typedef arelent::
@@ -71,10 +71,10 @@ CODE_FRAGMENT
 .       {* Used by special functions *}
 .  bfd_reloc_continue,
 .
-.       {* Unused *}
+.       {* Unsupported relocation size requested. *}
 .  bfd_reloc_notsupported,
 .
-.       {* Unsupported relocation size requested. *}
+.       {* Unused *}
 .  bfd_reloc_other,
 .
 .       {* The symbol to relocate against was undefined. *}
@@ -109,14 +109,14 @@ CODE_FRAGMENT
 /*
 DESCRIPTION
 
-        Here is a description of each of the fields within a relent:
+        Here is a description of each of the fields within an <<arelent>>:
 
-        o sym_ptr_ptr
+        o <<sym_ptr_ptr>>
 
         The symbol table pointer points to a pointer to the symbol
-        associated with the relocation request. This would naturally
-        be the pointer into the table returned by the back end's
-        get_symtab action. @xref{Symbols}. The symbol is referenced
+        associated with the relocation request.  It is
+        the pointer into the table returned by the back end's
+        <<get_symtab>> action. @xref{Symbols}. The symbol is referenced
         through a pointer to a pointer so that tools like the linker
         can fix up all the symbols of the same name by modifying only
         one pointer. The relocation routine looks in the symbol and
@@ -124,19 +124,19 @@ DESCRIPTION
         value of the symbol as the initial relocation offset. If the
         symbol pointer is zero, then the section provided is looked up.
 
-        o address
+        o <<address>>
 
-        The address field gives the offset in bytes from the base of
+        The <<address>> field gives the offset in bytes from the base of
         the section data which owns the relocation record to the first
         byte of relocatable information. The actual data relocated
-        will be relative to this point - for example, a relocation
+        will be relative to this point; for example, a relocation
         type which modifies the bottom two bytes of a four byte word
         would not touch the first byte pointed to in a big endian
         world.
        
-       o addend
+       o <<addend>>
 
-       The addend is a value provided by the back end to be added (!)
+       The <<addend>> is a value provided by the back end to be added (!)
        to the relocation offset. Its interpretation is dependent upon
        the howto. For example, on the 68k the code:
 
@@ -156,8 +156,8 @@ DESCRIPTION
 |        rts
 
 
-        This could create a reloc pointing to foo, but leave the
-        offset in the data (something like)
+        This could create a reloc pointing to <<foo>>, but leave the
+        offset in the data, something like:
 
 
 |RELOCATION RECORDS FOR [.text]:
@@ -181,7 +181,7 @@ DESCRIPTION
 |        jmp      r1
 
 
-        This should create two relocs, both pointing to _foo, and with
+        This should create two relocs, both pointing to <<_foo>>, and with
         0x12340000 in their addend field. The data would consist of:
 
 
@@ -196,17 +196,17 @@ DESCRIPTION
 
 
         The relocation routine digs out the value from the data, adds
-        it to the addend to get the original offset and then adds the
-        value of _foo. Note that all 32 bits have to be kept around
+        it to the addend to get the original offset, and then adds the
+        value of <<_foo>>. Note that all 32 bits have to be kept around
         somewhere, to cope with carry from bit 15 to bit 16.
 
         One further example is the sparc and the a.out format. The
         sparc has a similar problem to the 88k, in that some
         instructions don't have room for an entire offset, but on the
-        sparc the parts are created odd sized lumps. The designers of
-        the a.out format chose not to use the data within the section
+        sparc the parts are created in odd sized lumps. The designers of
+        the a.out format chose to not use the data within the section
         for storing part of the offset; all the offset is kept within
-        the reloc. Any thing in the data should be ignored. 
+        the reloc. Anything in the data should be ignored. 
 
 |        save %sp,-112,%sp
 |        sethi %hi(_foo+0x12345678),%g2
@@ -214,7 +214,7 @@ DESCRIPTION
 |        ret
 |        restore
 
-        Both relocs contains a pointer to foo, and the offsets would
+        Both relocs contains a pointer to <<foo>>, and the offsets
         contain junk.
 
 
@@ -230,11 +230,11 @@ DESCRIPTION
 |00000010 81e80000     ; restore
 
 
-        o howto 
+        o <<howto>>
 
-        The howto field can be imagined as a
-        relocation instruction. It is a pointer to a struct which
-        contains information on what to do with all the other
+        The <<howto>> field can be imagined as a
+        relocation instruction. It is a pointer to a structure which
+        contains information on what to do with all of the other
         information in the reloc record and data section. A back end
         would normally have a relocation instruction set and turn
         relocations into pointers to the correct structure on input -
@@ -242,24 +242,52 @@ DESCRIPTION
         
 */
 
+/*
+SUBSUBSECTION
+       <<enum complain_overflow>>
+
+       Indicates what sort of overflow checking should be done when
+       performing a relocation.
+
+CODE_FRAGMENT
+.
+.enum complain_overflow
+.{
+.      {* Do not complain on overflow. *}
+.  complain_overflow_dont,
+.
+.      {* Complain if the bitfield overflows, whether it is considered
+.         as signed or unsigned. *}
+.  complain_overflow_bitfield,
+.
+.      {* Complain if the value overflows when considered as signed
+.         number. *}
+.  complain_overflow_signed,
+.
+.      {* Complain if the value overflows when considered as an
+.         unsigned number. *}
+.  complain_overflow_unsigned
+.};
+
+*/
 
 /*
 SUBSUBSECTION 
         <<reloc_howto_type>>
 
         The <<reloc_howto_type>> is a structure which contains all the
-        information that BFD needs to know to tie up a back end's data.
+        information that libbfd needs to know to tie up a back end's data.
 
 CODE_FRAGMENT
 .struct symbol_cache_entry;            {* Forward declaration *}
 .
-.typedef CONST struct reloc_howto_struct 
+.typedef struct reloc_howto_struct 
 .{ 
 .       {*  The type field has mainly a documetary use - the back end can
-.           to what it wants with it, though the normally the back end's
-.           external idea of what a reloc number would be would be stored
-.           in this field. For example, the a PC relative word relocation
-.           in a coff environment would have the type 023 - because that's
+.           do what it wants with it, though normally the back end's
+.           external idea of what a reloc number is stored
+.           in this field. For example, a PC relative word relocation
+.           in a coff environment has the type 023 - because that's
 .           what the outside world calls a R_PCRWORD reloc. *}
 .  unsigned int type;
 .
@@ -267,12 +295,21 @@ CODE_FRAGMENT
 .           unwanted data from the relocation.  *}
 .  unsigned int rightshift;
 .
-.       {*  The size of the item to be relocated - 0, is one byte, 1 is 2
-.           bytes, 2 is four bytes.  A negative value indicates that the
-.          result is to be subtracted from the data.  *}
+.      {*  The size of the item to be relocated.  This is *not* a
+.          power-of-two measure.
+.               0 : one byte
+.               1 : two bytes
+.               2 : four bytes
+.               3 : nothing done (unless special_function is nonzero)
+.               4 : eight bytes
+.              -2 : two bytes, result should be subtracted from the
+.                   data instead of added
+.          There is currently no trivial way to extract a "number of
+.          bytes" from a howto pointer.  *}
 .  int size;
 .
-.       {*  Now obsolete?  But m68k-coff still uses it... *}
+.       {*  The number of bits in the item to be relocated.  This is used
+.          when doing overflow checking.  *}
 .  unsigned int bitsize;
 .
 .       {*  Notes that the relocation is relative to the location in the
@@ -281,14 +318,13 @@ CODE_FRAGMENT
 .           being relocated. *}
 .  boolean pc_relative;
 .
+.      {*  The bit position of the reloc value in the destination.
+.          The relocated value is left shifted by this amount. *}
 .  unsigned int bitpos;
 .
-.       {*  Now obsolete *}
-.  boolean absolute;
-.
-.       {* Causes the relocation routine to return an error if overflow
-.          is detected when relocating. *}
-.  boolean complain_on_overflow;
+.      {* What type of overflow error should be checked for when
+.         relocating. *}
+.  enum complain_overflow complain_on_overflow;
 .
 .       {* If this field is non null, then the supplied function is
 .          called rather than the normal function. This allows really
@@ -309,7 +345,7 @@ CODE_FRAGMENT
 .          relocations rather than the data - this flag signals this.*}
 .  boolean partial_inplace;
 .
-.       {* The src_mask is used to select what parts of the read in data
+.       {* The src_mask selects which parts of the read in data
 .          are to be used in the relocation sum.  E.g., if this was an 8 bit
 .          bit of data which we read and relocated, this would be
 .          0x000000ff. When we have relocs which have an addend, such as
@@ -318,7 +354,7 @@ CODE_FRAGMENT
 .          the mask would be 0x00000000. *}
 .  bfd_vma src_mask;
 .
-.       {* The dst_mask is what parts of the instruction are replaced
+.       {* The dst_mask selects which parts of the instruction are replaced
 .          into the instruction. In most cases src_mask == dst_mask,
 .          except in the above special case, where dst_mask would be
 .          0x000000ff, and src_mask would be 0x00000000.   *}
@@ -329,7 +365,7 @@ CODE_FRAGMENT
 .          slot of the instruction, so that a PC relative relocation can
 .          be made just by adding in an ordinary offset (e.g., sun3 a.out).
 .          Some formats leave the displacement part of an instruction
-.          empty (e.g., m88k bcs), this flag signals the fact.*}
+.          empty (e.g., m88k bcs); this flag signals the fact.*}
 .  boolean pcrel_offset;
 .
 .} reloc_howto_type;
@@ -338,21 +374,21 @@ CODE_FRAGMENT
 
 /*
 FUNCTION
-       the HOWTO macro
+       The HOWTO Macro
 
 DESCRIPTION
        The HOWTO define is horrible and will go away.
 
 
-.#define HOWTO(C, R,S,B, P, BI, ABS, O, SF, NAME, INPLACE, MASKSRC, MASKDST, PC) \
-.  {(unsigned)C,R,S,B, P, BI, ABS,O,SF,NAME,INPLACE,MASKSRC,MASKDST,PC}
+.#define HOWTO(C, R,S,B, P, BI, O, SF, NAME, INPLACE, MASKSRC, MASKDST, PC) \
+.  {(unsigned)C,R,S,B, P, BI, O,SF,NAME,INPLACE,MASKSRC,MASKDST,PC}
 
 DESCRIPTION
        And will be replaced with the totally magic way. But for the
-       moment, we are compatible, so do it this way..
+       moment, we are compatible, so do it this way.
 
 
-.#define NEWHOWTO( FUNCTION, NAME,SIZE,REL,IN) HOWTO(0,0,SIZE,0,REL,0,false,false,FUNCTION, NAME,false,0,0,IN)
+.#define NEWHOWTO( FUNCTION, NAME,SIZE,REL,IN) HOWTO(0,0,SIZE,0,REL,0,complain_overflow_dont,FUNCTION, NAME,false,0,0,IN)
 .
 DESCRIPTION
        Helper routine to turn a symbol into a relocation value.
@@ -373,11 +409,11 @@ DESCRIPTION
 
 /*
 TYPEDEF
-       reloc_chain
+       arelent_chain
 
 DESCRIPTION
 
-       How relocs are tied together
+       How relocs are tied together in an <<asection>>:
 
 .typedef unsigned char bfd_byte;
 .
@@ -397,18 +433,18 @@ FUNCTION
 SYNOPSIS
        bfd_reloc_status_type
                 bfd_perform_relocation
-                        (bfd * abfd,
+                        (bfd *abfd,
                         arelent *reloc_entry,
                         PTR data,
                         asection *input_section,
                         bfd *output_bfd);
 
 DESCRIPTION
-       If an output_bfd is supplied to this function the generated
-       image will be relocatable, the relocations are copied to the
+       If @var{output_bfd} is supplied to this function, the generated
+       image will be relocatable; the relocations are copied to the
        output file after they have been changed to reflect the new
        state of the world. There are two ways of reflecting the
-       results of partial linkage in an output file; by modifying the
+       results of partial linkage in an output file: by modifying the
        output data in place, and by modifying the relocation record.
        Some native formats (e.g., basic a.out and basic coff) have no
        way of specifying an addend in the relocation type, so the
@@ -449,7 +485,12 @@ DEFUN(bfd_perform_relocation,(abfd,
       return bfd_reloc_ok;
     }
 
-  if ((symbol->section == &bfd_und_section) && output_bfd == (bfd *)NULL)
+  /* If we are not producing relocateable output, return an error if
+     the symbol is not defined.  An undefined weak symbol is
+     considered to have a value of zero (SVR4 ABI, p. 4-27).  */
+  if (symbol->section == &bfd_und_section
+      && (symbol->flags & BSF_WEAK) == 0
+      && output_bfd == (bfd *) NULL)
     flag = bfd_reloc_undefined;
 
   /* If there is a function supplied to handle this relocation type,
@@ -491,16 +532,38 @@ DEFUN(bfd_perform_relocation,(abfd,
   /* Add in supplied addend.  */
   relocation += reloc_entry->addend;
 
+  /* Here the variable relocation holds the final address of the
+     symbol we are relocating against, plus any addend.  */
+
   if (howto->pc_relative == true)
     {
-      /* Anything which started out as pc relative should end up that
-        way too. 
-
-        There are two ways we can see a pcrel instruction. Sometimes
-        the pcrel displacement has been partially calculated, it
-        includes the distance from the start of the section to the
-        instruction in it (e.g., sun3), and sometimes the field is
-        totally blank - e.g., m88kbcs.  */
+      /* This is a PC relative relocation.  We want to set RELOCATION
+        to the distance between the address of the symbol and the
+        location.  RELOCATION is already the address of the symbol.
+
+        We start by subtracting the address of the section containing
+        the location.
+
+        If pcrel_offset is set, we must further subtract the position
+        of the location within the section.  Some targets arrange for
+        the addend to be the negative of the position of the location
+        within the section; for example, i386-aout does this.  For
+        i386-aout, pcrel_offset is false.  Some other targets do not
+        include the position of the location; for example, m88kbcs,
+        or ELF.  For those targets, pcrel_offset is true.
+
+        If we are producing relocateable output, then we must ensure
+        that this reloc will be correctly computed when the final
+        relocation is done.  If pcrel_offset is false we want to wind
+        up with the negative of the location within the section,
+        which means we must adjust the existing addend by the change
+        in the location within the section.  If pcrel_offset is true
+        we do not want to adjust the existing addend at all.
+
+        FIXME: This seems logical to me, but for the case of
+        producing relocateable output it is not what the code
+        actually does.  I don't want to change it, because it seems
+        far too likely that something will break.  */
 
       relocation -= 
        input_section->output_section->vma + input_section->output_offset;
@@ -533,7 +596,78 @@ DEFUN(bfd_perform_relocation,(abfd,
          /* WTF?? */
          if (abfd->xvec->flavour == bfd_target_coff_flavour) 
            {
+#if 1
+             /* For m68k-coff, the addend was being subtracted twice during
+                relocation with -r.  Removing the line below this comment
+                fixes that problem; see PR 2953.
+
+However, Ian wrote the following, regarding removing the line below,
+which explains why it is still enabled:  --djm
+
+If you put a patch like that into BFD you need to check all the COFF
+linkers.  I am fairly certain that patch will break coff-i386 (e.g.,
+SCO); see coff_i386_reloc in coff-i386.c where I worked around the
+problem in a different way.  There may very well be a reason that the
+code works as it does.
+
+Hmmm.  The first obvious point is that bfd_perform_relocation should
+not have any tests that depend upon the flavour.  It's seem like
+entirely the wrong place for such a thing.  The second obvious point
+is that the current code ignores the reloc addend when producing
+relocateable output for COFF.  That's peculiar.  In fact, I really
+have no idea what the point of the line you want to remove is.
+
+A typical COFF reloc subtracts the old value of the symbol and adds in
+the new value to the location in the object file (if it's a pc
+relative reloc it adds the difference between the symbol value and the
+location).  When relocating we need to preserve that property.
+
+BFD handles this by setting the addend to the negative of the old
+value of the symbol.  Unfortunately it handles common symbols in a
+non-standard way (it doesn't subtract the old value) but that's a
+different story (we can't change it without losing backward
+compatibility with old object files) (coff-i386 does subtract the old
+value, to be compatible with existing coff-i386 targets, like SCO).
+
+So everything works fine when not producing relocateable output.  When
+we are producing relocateable output, logically we should do exactly
+what we do when not producing relocateable output.  Therefore, your
+patch is correct.  In fact, it should probably always just set
+reloc_entry->addend to 0 for all cases, since it is, in fact, going to
+add the value into the object file.  This won't hurt the COFF code,
+which doesn't use the addend; I'm not sure what it will do to other
+formats (the thing to check for would be whether any formats both use
+the addend and set partial_inplace).
+
+When I wanted to make coff-i386 produce relocateable output, I ran
+into the problem that you are running into: I wanted to remove that
+line.  Rather than risk it, I made the coff-i386 relocs use a special
+function; it's coff_i386_reloc in coff-i386.c.  The function
+specifically adds the addend field into the object file, knowing that
+bfd_perform_relocation is not going to.  If you remove that line, then
+coff-i386.c will wind up adding the addend field in twice.  It's
+trivial to fix; it just needs to be done.
+
+The problem with removing the line is just that it may break some
+working code.  With BFD it's hard to be sure of anything.  The right
+way to deal with this is simply to build and test at least all the
+supported COFF targets.  It should be straightforward if time and disk
+space consuming.  For each target:
+    1) build the linker
+    2) generate some executable, and link it using -r (I would
+       probably use paranoia.o and link against newlib/libc.a, which
+       for all the supported targets would be available in
+       /usr/cygnus/progressive/H-host/target/lib/libc.a).
+    3) make the change to reloc.c
+    4) rebuild the linker
+    5) repeat step 2
+    6) if the resulting object files are the same, you have at least
+       made it no worse
+    7) if they are different you have to figure out which version is
+       right
+*/
              relocation -= reloc_entry->addend;
+#endif
              reloc_entry->addend = 0;
            }
          else
@@ -547,18 +681,84 @@ DEFUN(bfd_perform_relocation,(abfd,
       reloc_entry->addend = 0;
     }
 
-
-  if (howto->complain_on_overflow && howto->pc_relative)
+  /* FIXME: This overflow checking is incomplete, because the value
+     might have overflowed before we get here.  For a correct check we
+     need to compute the value in a size larger than bitsize, but we
+     can't reasonably do that for a reloc the same size as a host
+     machine word.
+     FIXME: We should also do overflow checking on the result after
+     adding in the value contained in the object file.  */
+  if (howto->complain_on_overflow != complain_overflow_dont)
     {
-      /* We can detect overflow safely here */
-
-      bfd_signed_vma reloc_max = (1 << (howto->bitsize - 1))-1;
-      bfd_signed_vma reloc_min = ~(reloc_max);
+      bfd_vma check;
 
-      if ((bfd_signed_vma) relocation > reloc_max
-         || (bfd_signed_vma) relocation < reloc_min)
+      /* Get the value that will be used for the relocation, but
+        starting at bit position zero.  */
+      if (howto->rightshift > howto->bitpos)
+       check = relocation >> (howto->rightshift - howto->bitpos);
+      else
+       check = relocation << (howto->bitpos - howto->rightshift);
+      switch (howto->complain_on_overflow)
        {
-         flag = bfd_reloc_overflow;
+       case complain_overflow_signed:
+         {
+           /* Assumes two's complement.  */
+           bfd_signed_vma reloc_signed_max = (1 << (howto->bitsize - 1)) - 1;
+           bfd_signed_vma reloc_signed_min = ~ reloc_signed_max;
+
+           /* The above right shift is incorrect for a signed value.
+              Fix it up by forcing on the upper bits.  */
+           if (howto->rightshift > howto->bitpos
+               && (bfd_signed_vma) relocation < 0)
+             check |= ((bfd_vma) -1
+                       &~ ((bfd_vma) -1
+                           >> (howto->rightshift - howto->bitpos)));
+           if ((bfd_signed_vma) check > reloc_signed_max
+               || (bfd_signed_vma) check < reloc_signed_min)
+             flag = bfd_reloc_overflow;
+         }
+         break;
+       case complain_overflow_unsigned:
+         {
+           /* Assumes two's complement.  This expression avoids
+              overflow if howto->bitsize is the number of bits in
+              bfd_vma.  */
+           bfd_vma reloc_unsigned_max =
+             (((1 << (howto->bitsize - 1)) - 1) << 1) | 1;
+
+           if ((bfd_vma) check > reloc_unsigned_max)
+             flag = bfd_reloc_overflow;
+         }
+         break;
+       case complain_overflow_bitfield:
+         {
+           /* Assumes two's complement.  This expression avoids
+              overflow if howto->bitsize is the number of bits in
+              bfd_vma.  */
+           bfd_vma reloc_bits = (((1 << (howto->bitsize - 1)) - 1) << 1) | 1;
+
+           if (((bfd_vma) check &~ reloc_bits) != 0
+               && ((bfd_vma) check &~ reloc_bits) != (-1 &~ reloc_bits))
+             {
+               /* The above right shift is incorrect for a signed
+                  value.  See if turning on the upper bits fixes the
+                  overflow.  */
+               if (howto->rightshift > howto->bitpos
+                   && (bfd_signed_vma) relocation < 0)
+                 {
+                   check |= ((bfd_vma) -1
+                             &~ ((bfd_vma) -1
+                                 >> (howto->rightshift - howto->bitpos)));
+                   if (((bfd_vma) check &~ reloc_bits) != (-1 &~ reloc_bits))
+                     flag = bfd_reloc_overflow;
+                 }
+               else
+                 flag = bfd_reloc_overflow;
+             }
+         }
+         break;
+       default:
+         abort ();
        }
     }
   
@@ -567,11 +767,35 @@ DEFUN(bfd_perform_relocation,(abfd,
     the relocation to the reloc entry (probably because there isn't
     any room in the output format to describe addends to relocs)
     */
-  relocation >>= howto->rightshift;
+
+  /* The cast to bfd_vma avoids a bug in the Alpha OSF/1 C compiler
+     (OSF version 1.3, compiler version 3.11).  It miscompiles the
+     following program:
+
+     struct str
+     {
+       unsigned int i0;
+     } s = { 0 };
+
+     int
+     main ()
+     {
+       unsigned long x;
+
+       x = 0x100000000;
+       x <<= (unsigned long) s.i0;
+       if (x == 0)
+        printf ("failed\n");
+       else
+        printf ("succeeded (%lx)\n", x);
+     }
+     */
+
+  relocation >>= (bfd_vma) howto->rightshift;
 
   /* Shift everything up to where it's going to be used */
    
-  relocation <<= howto->bitpos;
+  relocation <<= (bfd_vma) howto->bitpos;
 
   /* Wait for the day when all have the mask in them */
 
@@ -645,9 +869,21 @@ DEFUN(bfd_perform_relocation,(abfd,
        break;
 
      case 3:
-
        /* Do nothing */
        break;
+
+     case 4:
+#ifdef BFD64
+       if (relocation)
+        {
+          bfd_vma x = bfd_get_64 (abfd, (bfd_byte *) data + addr);
+          DOIT (x);
+          bfd_put_64 (abfd, x, (bfd_byte *) data + addr);
+        }
+#else
+       abort ();
+#endif
+       break;
      default:
        return bfd_reloc_other;
      }
@@ -676,54 +912,57 @@ TYPEDEF
        bfd_reloc_code_type
 
 DESCRIPTION
-       The insides of a reloc code
+       The insides of a reloc code.  The idea is that, eventually, there
+       will be one enumerator for every type of relocation we ever do.
+       Pass one of these values to <<bfd_reloc_type_lookup>>, and it'll
+       return a howto pointer.
+
+       This does mean that the application must determine the correct
+       enumerator value; you can't get a howto pointer from a random set
+       of attributes.
 
 CODE_FRAGMENT
 .
 .typedef enum bfd_reloc_code_real 
-.
 .{
-.      {* 64 bits wide, simple reloc *}
+.  {* Basic absolute relocations *}
 .  BFD_RELOC_64,
-.      {* 64 bits, PC-relative *}
-.  BFD_RELOC_64_PCREL,
-.
-.       {* 32 bits wide, simple reloc *}
 .  BFD_RELOC_32,
-.      {* 32 bits, PC-relative *}
-.  BFD_RELOC_32_PCREL,
-.
-.       {* 16 bits wide, simple reloc *}
 .  BFD_RELOC_16,        
-.      {* 16 bits, PC-relative *}
-.  BFD_RELOC_16_PCREL,
-.
-.       {* 8 bits wide, simple *}
 .  BFD_RELOC_8,
-.       {* 8 bits wide, pc relative *}
+.
+.  {* PC-relative relocations *}
+.  BFD_RELOC_64_PCREL,
+.  BFD_RELOC_32_PCREL,
+.  BFD_RELOC_24_PCREL,    {* used by i960 *}
+.  BFD_RELOC_16_PCREL,
 .  BFD_RELOC_8_PCREL,
-.       {* 8 bits wide, but used to form an address like 0xffnn *}
-.  BFD_RELOC_8_FFnn,
 .
-.       {* The type of reloc used to build a contructor table - at the
-.          moment probably a 32 bit wide abs address, but the cpu can
-.          choose. *}
+.  {* Linkage-table relative *}
+.  BFD_RELOC_32_BASEREL,
+.  BFD_RELOC_16_BASEREL,
+.  BFD_RELOC_8_BASEREL,
 .
+.  {* The type of reloc used to build a contructor table - at the moment
+.     probably a 32 bit wide abs address, but the cpu can choose. *}
 .  BFD_RELOC_CTOR,
 .
-.      {* High 22 bits of 32-bit value; simple reloc.  *}
+.  {* 8 bits wide, but used to form an address like 0xffnn *}
+.  BFD_RELOC_8_FFnn,
+.
+.  {* 32-bit pc-relative, shifted right 2 bits (i.e., 30-bit
+.     word displacement, e.g. for SPARC) *}
+.  BFD_RELOC_32_PCREL_S2,
+.
+.  {* High 22 bits of 32-bit value, placed into lower 22 bits of
+.     target word; simple reloc.  *}
 .  BFD_RELOC_HI22,
-.      {* Low 10 bits.  *}
+.  {* Low 10 bits.  *}
 .  BFD_RELOC_LO10,
 .
-.      {* Reloc types used for i960/b.out.  *}
-.  BFD_RELOC_24_PCREL,
+.  {* Reloc types used for i960/b.out.  *}
 .  BFD_RELOC_I960_CALLJ,
 .
-.      {* 32-bit pc-relative, shifted right 2 bits (i.e., 30-bit
-.         word displacement, e.g. for SPARC) *}
-.  BFD_RELOC_32_PCREL_S2,
-.
 .  {* now for the sparc/elf codes *}
 .  BFD_RELOC_NONE,             {* actually used *}
 .  BFD_RELOC_SPARC_WDISP22,
@@ -741,7 +980,7 @@ CODE_FRAGMENT
 .  BFD_RELOC_SPARC_RELATIVE,
 .  BFD_RELOC_SPARC_UA32,
 .
-.  {* this one is a.out specific? *}
+.  {* these are a.out specific? *}
 .  BFD_RELOC_SPARC_BASE13,
 .  BFD_RELOC_SPARC_BASE22,
 .
@@ -762,37 +1001,45 @@ CODE_FRAGMENT
 .  BFD_RELOC_SPARC_LO7,
 .  {* end-sanitize-v9 *}
 .
-.       {* Bits 27..2 of the relocation address shifted right 2 bits;
-.         simple reloc otherwise.  *}
+.  {* Bits 27..2 of the relocation address shifted right 2 bits;
+.     simple reloc otherwise.  *}
 .  BFD_RELOC_MIPS_JMP,
 .
-.       {* signed 16-bit pc-relative, shifted right 2 bits (e.g. for MIPS) *}
+.  {* signed 16-bit pc-relative, shifted right 2 bits (e.g. for MIPS) *}
 .  BFD_RELOC_16_PCREL_S2,
 .
-.       {* High 16 bits of 32-bit value; simple reloc.  *}
+.  {* High 16 bits of 32-bit value; simple reloc.  *}
 .  BFD_RELOC_HI16,
-.       {* High 16 bits of 32-bit value but the low 16 bits will be sign
-.          extended and added to form the final result.  If the low 16
-.          bits form a negative number, we need to add one to the high value
-.          to compensate for the borrow when the low bits are added.  *}
+.  {* High 16 bits of 32-bit value but the low 16 bits will be sign
+.     extended and added to form the final result.  If the low 16
+.     bits form a negative number, we need to add one to the high value
+.     to compensate for the borrow when the low bits are added.  *}
 .  BFD_RELOC_HI16_S,
-.       {* Low 16 bits.  *}
+.  {* Low 16 bits.  *}
 .  BFD_RELOC_LO16,
 .
-.      {* 16 bit relocation relative to the global pointer.  *}
+.  {* 16 bit relocation relative to the global pointer.  *}
 .  BFD_RELOC_MIPS_GPREL,
 .
-.       {* These are, so far, specific to HPPA processors.  I'm not sure that
-.         some don't duplicate other reloc types, such as BFD_RELOC_32 and
-.         _32_PCREL.  Also, many more were in the list I got that don't
-.         fit in well in the model BFD uses, so I've omitted them for now.
-.         If we do make this reloc type get used for code that really does
-.         implement the funky reloc types, they'll have to be added to this
-.         list.   *}
+.  {* Relocation against a MIPS literal section.  *}
+.  BFD_RELOC_MIPS_LITERAL,
+.
+.  {* MIPS ELF relocations.  *}
+.  BFD_RELOC_MIPS_GOT16,
+.  BFD_RELOC_MIPS_CALL16,
+.  BFD_RELOC_MIPS_GPREL32,
+.
+.  {* These are, so far, specific to HPPA processors.  I'm not sure that some
+.     don't duplicate other reloc types, such as BFD_RELOC_32 and _32_PCREL.
+.     Also, many more were in the list I got that don't fit in well in the
+.     model BFD uses, so I've omitted them for now.  If we do make this reloc
+.     type get used for code that really does implement the funky reloc types,
+.     they'll have to be added to this list.  *}
 .  BFD_RELOC_HPPA_32,
 .  BFD_RELOC_HPPA_11,
 .  BFD_RELOC_HPPA_14,
 .  BFD_RELOC_HPPA_17,
+.
 .  BFD_RELOC_HPPA_L21,
 .  BFD_RELOC_HPPA_R11,
 .  BFD_RELOC_HPPA_R14,
@@ -808,6 +1055,7 @@ CODE_FRAGMENT
 .  BFD_RELOC_HPPA_LR21,
 .  BFD_RELOC_HPPA_RR14,
 .  BFD_RELOC_HPPA_RR17,
+.
 .  BFD_RELOC_HPPA_GOTOFF_11,
 .  BFD_RELOC_HPPA_GOTOFF_14,
 .  BFD_RELOC_HPPA_GOTOFF_L21,
@@ -821,12 +1069,14 @@ CODE_FRAGMENT
 .  BFD_RELOC_HPPA_GOTOFF_RD14,
 .  BFD_RELOC_HPPA_GOTOFF_LR21,
 .  BFD_RELOC_HPPA_GOTOFF_RR14,
+.
 .  BFD_RELOC_HPPA_DLT_32,
 .  BFD_RELOC_HPPA_DLT_11,
 .  BFD_RELOC_HPPA_DLT_14,
 .  BFD_RELOC_HPPA_DLT_L21,
 .  BFD_RELOC_HPPA_DLT_R11,
 .  BFD_RELOC_HPPA_DLT_R14,
+.
 .  BFD_RELOC_HPPA_ABS_CALL_11,
 .  BFD_RELOC_HPPA_ABS_CALL_14,
 .  BFD_RELOC_HPPA_ABS_CALL_17,
@@ -845,6 +1095,7 @@ CODE_FRAGMENT
 .  BFD_RELOC_HPPA_ABS_CALL_LR21,
 .  BFD_RELOC_HPPA_ABS_CALL_RR14,
 .  BFD_RELOC_HPPA_ABS_CALL_RR17,
+.
 .  BFD_RELOC_HPPA_PCREL_CALL_11,
 .  BFD_RELOC_HPPA_PCREL_CALL_12,
 .  BFD_RELOC_HPPA_PCREL_CALL_14,
@@ -864,24 +1115,35 @@ CODE_FRAGMENT
 .  BFD_RELOC_HPPA_PCREL_CALL_LR21,
 .  BFD_RELOC_HPPA_PCREL_CALL_RR14,
 .  BFD_RELOC_HPPA_PCREL_CALL_RR17,
+.
 .  BFD_RELOC_HPPA_PLABEL_32,
 .  BFD_RELOC_HPPA_PLABEL_11,
 .  BFD_RELOC_HPPA_PLABEL_14,
 .  BFD_RELOC_HPPA_PLABEL_L21,
 .  BFD_RELOC_HPPA_PLABEL_R11,
 .  BFD_RELOC_HPPA_PLABEL_R14,
+.
 .  BFD_RELOC_HPPA_UNWIND_ENTRY,
 .  BFD_RELOC_HPPA_UNWIND_ENTRIES,
 .
+.  {* i386/elf relocations *}
+.  BFD_RELOC_386_GOT32,
+.  BFD_RELOC_386_PLT32,
+.  BFD_RELOC_386_COPY,
+.  BFD_RELOC_386_GLOB_DAT,
+.  BFD_RELOC_386_JUMP_SLOT,
+.  BFD_RELOC_386_RELATIVE,
+.  BFD_RELOC_386_GOTOFF,
+.  BFD_RELOC_386_GOTPC,
+.
 .  {* this must be the highest numeric value *}
 .  BFD_RELOC_UNUSED
 . } bfd_reloc_code_real_type;
 */
 
 
-
 /*
-SECTION
+FUNCTION
        bfd_reloc_type_lookup
 
 SYNOPSIS
@@ -889,8 +1151,8 @@ SYNOPSIS
        bfd_reloc_type_lookup (bfd *abfd, bfd_reloc_code_real_type code);
 
 DESCRIPTION
-       This routine returns a pointer to a howto struct which when
-       invoked, will perform the supplied relocation on data from the
+       Return a pointer to a howto struct which, when
+       invoked, will perform the relocation @var{code} on data from the
        architecture noted.
 
 */
@@ -905,7 +1167,7 @@ DEFUN(bfd_reloc_type_lookup,(abfd, code),
 }
 
 static reloc_howto_type bfd_howto_32 =
- HOWTO(0, 00,2,32,false,0,false,true,0,"VRT32", false,0xffffffff,0xffffffff,true);
+ HOWTO(0, 00,2,32,false,0,complain_overflow_bitfield,0,"VRT32", false,0xffffffff,0xffffffff,true);
 
 
 /*
@@ -932,7 +1194,7 @@ DEFUN(bfd_default_reloc_type_lookup, (abfd, code),
     {
     case BFD_RELOC_CTOR:
       /* The type of reloc used in a ctor, which will be as wide as the
-        address - so either a 64, 32, or 16 bitter.. */
+        address - so either a 64, 32, or 16 bitter.  */
       switch (bfd_get_arch_info (abfd)->bits_per_address) {
       case 64:
        BFD_FAIL();