* config/tc-hppa.c (pa_ip): Put check for missing label on .PROC
[deliverable/binutils-gdb.git] / gas / config / tc-hppa.c
index 095fec9ec50148d116bce13fcbe7bc6eef4b1149..5005bbd786ee902894a59365f97afd663473c2c9 100644 (file)
@@ -52,8 +52,10 @@ typedef elf_symbol_type obj_symbol_type;
 /* How to generate a relocation.  */
 #define hppa_gen_reloc_type hppa_elf_gen_reloc_type
 
-/* Who knows.  */
+/* ELF objects can have versions, but apparently do not have anywhere
+   to store a copyright string.  */
 #define obj_version obj_elf_version
+#define obj_copyright obj_elf_version
 
 /* Use space aliases.  */
 #define USE_ALIASES 1
@@ -73,8 +75,9 @@ static void hppa_tc_make_symextn_section PARAMS ((void));
 /* Object file formats specify relocation types.  */
 typedef int reloc_type;
 
-/* Who knows.  */
+/* SOM objects can have both a version string and a copyright string.  */
 #define obj_version obj_som_version
+#define obj_copyright obj_som_copyright
 
 /* Do not use space aliases.  */
 #define USE_ALIASES 0
@@ -135,12 +138,6 @@ struct unwind_table
 
 struct call_info
   {
-    /* Should sr3 be saved in the prologue?  */
-    int entry_sr;
-
-    /* Does this function make calls?  */
-    int makes_calls;
-
     /* The unwind descriptor being built.  */
     struct unwind_table ci_unwind;
 
@@ -150,23 +147,6 @@ struct call_info
     /* (temporary) symbol used to mark the end of this function.  */
     symbolS *end_symbol;
 
-    /* frags associated with start and end of this function.  */
-    fragS *start_frag;
-    fragS *end_frag;
-
-    /* frags for starting/ending offset of this descriptor.  */
-    fragS *start_offset_frag;
-    fragS *end_offset_frag;
-
-    /* The location within {start,end}_offset_frag to find the 
-       {start,end}_offset.  */
-    int start_frag_where;
-    int end_frag_where;
-
-    /* Fixups (relocations) for start_offset and end_offset.  */
-    fixS *start_fix;
-    fixS *end_fix;
-
     /* Next entry in the chain.  */
     struct call_info *ci_next;
   };
@@ -281,50 +261,9 @@ struct call_desc
 
 struct subspace_dictionary_chain
   {
-    /* Index of containing space.  */
-    unsigned long ssd_space_index;
-
     /* Nonzero if this space has been defined by the user code.  */
     unsigned int ssd_defined;
 
-    /* Which quadrant within the space this subspace should be loaded into.  */
-    unsigned char ssd_quadrant;
-
-    /* Alignment (in bytes) for this subspace.  */
-    unsigned long ssd_alignment;
-
-    /* Access control bits to determine read/write/execute permissions
-       as well as gateway privilege promotions.  */
-    unsigned char ssd_access_control_bits;
-
-    /* A sorting key so that it is possible to specify ordering of
-       subspaces within a space.  */
-    unsigned char ssd_sort_key;
-
-    /* Nonzero of this space should be zero filled.  */
-    unsigned long ssd_zero;
-
-    /* Nonzero if this is a common subspace.  */
-    unsigned char ssd_common;
-
-    /* Nonzero if this is a common subspace which allows symbols to be 
-       multiply defined.  */
-    unsigned char ssd_dup_common;
-
-    /* Nonzero if this subspace is loadable.  Note loadable subspaces
-       must be contained within loadable spaces; unloadable subspaces
-       must be contained in unloadable spaces.  */
-    unsigned char ssd_loadable;
-
-    /* Nonzero if this subspace contains only code.  */
-    unsigned char ssd_code_only;
-
-    /* Starting offset of this subspace.  */
-    unsigned long ssd_subspace_start;
-
-    /* Length of this subspace.  */
-    unsigned long ssd_subspace_length;
-
     /* Name of this subspace.  */
     char *ssd_name;
 
@@ -332,13 +271,6 @@ struct subspace_dictionary_chain
     asection *ssd_seg;
     int ssd_subseg;
 
-    /* Index of this subspace within the subspace dictionary of the object
-       file.  Not used until object file is written.  */
-    int object_file_index;
-
-    /* The size of the last alignment request for this subspace.  */
-    int ssd_last_align;
-
     /* Next space in the subspace dictionary chain.  */
     struct subspace_dictionary_chain *ssd_next;
   };
@@ -350,14 +282,6 @@ typedef struct subspace_dictionary_chain ssd_chain_struct;
 
 struct space_dictionary_chain
   {
-
-    /* Holds the index into the string table of the name of this 
-       space.  */
-    unsigned int sd_name_index;
-
-    /* Nonzero if the space is loadable.  */
-    unsigned int sd_loadable;
-
     /* Nonzero if this space has been defined by the user code or 
        as a default space.  */
     unsigned int sd_defined;
@@ -365,16 +289,9 @@ struct space_dictionary_chain
     /* Nonzero if this spaces has been defined by the user code.  */
     unsigned int sd_user_defined;
 
-    /* Nonzero if this space is not sharable.  */
-    unsigned int sd_private;
-
     /* The space number (or index).  */
     unsigned int sd_spnum;
 
-    /* The sort key for this space.  May be used to determine how to lay
-       out the spaces within the object file.  */
-    unsigned char sd_sort_key;
-
     /* The name of this subspace.  */
     char *sd_name;
 
@@ -536,7 +453,6 @@ struct selector_entry
 static fp_operand_format pa_parse_fp_format PARAMS ((char **s));
 static void pa_cons PARAMS ((int));
 static void pa_data PARAMS ((int));
-static void pa_desc PARAMS ((int));
 static void pa_float_cons PARAMS ((int));
 static void pa_fill PARAMS ((int));
 static void pa_lcomm PARAMS ((int));
@@ -610,7 +526,6 @@ static void fix_new_hppa PARAMS ((fragS *, int, short int, symbolS *,
                                  bfd_reloc_code_real_type,
                                  enum hppa_reloc_field_selector_type,
                                  int, long, char *));
-static void md_apply_fix_1 PARAMS ((fixS *, long));
 static int is_end_of_statement PARAMS ((void));
 static int reg_name_search PARAMS ((char *));
 static int pa_chk_field_selector PARAMS ((char **));
@@ -687,8 +602,6 @@ const pseudo_typeS md_pseudo_table[] =
   {"COPYRIGHT", pa_copyright, 0},
   {"data", pa_data, 0},
   {"DATA", pa_data, 0},
-  {"desc", pa_desc, 0},
-  {"DESC", pa_desc, 0},
   {"double", pa_float_cons, 'd'},
   {"DOUBLE", pa_float_cons, 'd'},
   {"end", pa_end, 0},
@@ -810,6 +723,9 @@ static label_symbol_struct *label_symbols_rootp = NULL;
 /* Holds the last field selector.  */
 static int hppa_field_selector;
 
+/* A dummy bfd symbol so that all relocations have symbols of some kind.  */
+static asymbol *dummy_symbol;
+
 /* Nonzero if errors are to be printed.  */
 static int print_errors = 1;
 
@@ -874,101 +790,101 @@ static const struct pd_reg pre_defined_registers[] =
   {"%eiem", 15},
   {"%eirr", 23},
   {"%fr0", 0},
-  {"%fr0L", 0},
-  {"%fr0R", 0},
+  {"%fr0l", 0},
+  {"%fr0r", 0},
   {"%fr1", 1},
   {"%fr10", 10},
-  {"%fr10L", 10},
-  {"%fr10R", 10},
+  {"%fr10l", 10},
+  {"%fr10r", 10},
   {"%fr11", 11},
-  {"%fr11L", 11},
-  {"%fr11R", 11},
+  {"%fr11l", 11},
+  {"%fr11r", 11},
   {"%fr12", 12},
-  {"%fr12L", 12},
-  {"%fr12R", 12},
+  {"%fr12l", 12},
+  {"%fr12r", 12},
   {"%fr13", 13},
-  {"%fr13L", 13},
-  {"%fr13R", 13},
+  {"%fr13l", 13},
+  {"%fr13r", 13},
   {"%fr14", 14},
-  {"%fr14L", 14},
-  {"%fr14R", 14},
+  {"%fr14l", 14},
+  {"%fr14r", 14},
   {"%fr15", 15},
-  {"%fr15L", 15},
-  {"%fr15R", 15},
+  {"%fr15l", 15},
+  {"%fr15r", 15},
   {"%fr16", 16},
-  {"%fr16L", 16},
-  {"%fr16R", 16},
+  {"%fr16l", 16},
+  {"%fr16r", 16},
   {"%fr17", 17},
-  {"%fr17L", 17},
-  {"%fr17R", 17},
+  {"%fr17l", 17},
+  {"%fr17r", 17},
   {"%fr18", 18},
-  {"%fr18L", 18},
-  {"%fr18R", 18},
+  {"%fr18l", 18},
+  {"%fr18r", 18},
   {"%fr19", 19},
-  {"%fr19L", 19},
-  {"%fr19R", 19},
-  {"%fr1L", 1},
-  {"%fr1R", 1},
+  {"%fr19l", 19},
+  {"%fr19r", 19},
+  {"%fr1l", 1},
+  {"%fr1r", 1},
   {"%fr2", 2},
   {"%fr20", 20},
-  {"%fr20L", 20},
-  {"%fr20R", 20},
+  {"%fr20l", 20},
+  {"%fr20r", 20},
   {"%fr21", 21},
-  {"%fr21L", 21},
-  {"%fr21R", 21},
+  {"%fr21l", 21},
+  {"%fr21r", 21},
   {"%fr22", 22},
-  {"%fr22L", 22},
-  {"%fr22R", 22},
+  {"%fr22l", 22},
+  {"%fr22r", 22},
   {"%fr23", 23},
-  {"%fr23L", 23},
-  {"%fr23R", 23},
+  {"%fr23l", 23},
+  {"%fr23r", 23},
   {"%fr24", 24},
-  {"%fr24L", 24},
-  {"%fr24R", 24},
+  {"%fr24l", 24},
+  {"%fr24r", 24},
   {"%fr25", 25},
-  {"%fr25L", 25},
-  {"%fr25R", 25},
+  {"%fr25l", 25},
+  {"%fr25r", 25},
   {"%fr26", 26},
-  {"%fr26L", 26},
-  {"%fr26R", 26},
+  {"%fr26l", 26},
+  {"%fr26r", 26},
   {"%fr27", 27},
-  {"%fr27L", 27},
-  {"%fr27R", 27},
+  {"%fr27l", 27},
+  {"%fr27r", 27},
   {"%fr28", 28},
-  {"%fr28L", 28},
-  {"%fr28R", 28},
+  {"%fr28l", 28},
+  {"%fr28r", 28},
   {"%fr29", 29},
-  {"%fr29L", 29},
-  {"%fr29R", 29},
-  {"%fr2L", 2},
-  {"%fr2R", 2},
+  {"%fr29l", 29},
+  {"%fr29r", 29},
+  {"%fr2l", 2},
+  {"%fr2r", 2},
   {"%fr3", 3},
   {"%fr30", 30},
-  {"%fr30L", 30},
-  {"%fr30R", 30},
+  {"%fr30l", 30},
+  {"%fr30r", 30},
   {"%fr31", 31},
-  {"%fr31L", 31},
-  {"%fr31R", 31},
-  {"%fr3L", 3},
-  {"%fr3R", 3},
+  {"%fr31l", 31},
+  {"%fr31r", 31},
+  {"%fr3l", 3},
+  {"%fr3r", 3},
   {"%fr4", 4},
-  {"%fr4L", 4},
-  {"%fr4R", 4},
+  {"%fr4l", 4},
+  {"%fr4r", 4},
   {"%fr5", 5},
-  {"%fr5L", 5},
-  {"%fr5R", 5},
+  {"%fr5l", 5},
+  {"%fr5r", 5},
   {"%fr6", 6},
-  {"%fr6L", 6},
-  {"%fr6R", 6},
+  {"%fr6l", 6},
+  {"%fr6r", 6},
   {"%fr7", 7},
-  {"%fr7L", 7},
-  {"%fr7R", 7},
+  {"%fr7l", 7},
+  {"%fr7r", 7},
   {"%fr8", 8},
-  {"%fr8L", 8},
-  {"%fr8R", 8},
+  {"%fr8l", 8},
+  {"%fr8r", 8},
   {"%fr9", 9},
-  {"%fr9L", 9},
-  {"%fr9R", 9},
+  {"%fr9l", 9},
+  {"%fr9r", 9},
   {"%hta", 25},
   {"%iir", 19},
   {"%ior", 21},
@@ -1010,23 +926,11 @@ static const struct pd_reg pre_defined_registers[] =
   {"%r30", 30},
   {"%r31", 31},
   {"%r4", 4},
-  {"%r4L", 4},
-  {"%r4R", 4},
   {"%r5", 5},
-  {"%r5L", 5},
-  {"%r5R", 5},
   {"%r6", 6},
-  {"%r6L", 6},
-  {"%r6R", 6},
   {"%r7", 7},
-  {"%r7L", 7},
-  {"%r7R", 7},
   {"%r8", 8},
-  {"%r8L", 8},
-  {"%r8R", 8},
   {"%r9", 9},
-  {"%r9L", 9},
-  {"%r9R", 9},
   {"%rctr", 0},
   {"%ret0", 28},
   {"%ret1", 29},
@@ -1092,37 +996,21 @@ static const struct fp_cond_map fp_cond_map[] =
 
 static const struct selector_entry selector_table[] =
 {
-  {"F'", e_fsel},
-  {"F%", e_fsel},
-  {"LS'", e_lssel},
-  {"LS%", e_lssel},
-  {"RS'", e_rssel},
-  {"RS%", e_rssel},
-  {"L'", e_lsel},
-  {"L%", e_lsel},
-  {"R'", e_rsel},
-  {"R%", e_rsel},
-  {"LD'", e_ldsel},
-  {"LD%", e_ldsel},
-  {"RD'", e_rdsel},
-  {"RD%", e_rdsel},
-  {"LR'", e_lrsel},
-  {"LR%", e_lrsel},
-  {"RR'", e_rrsel},
-  {"RR%", e_rrsel},
-  {"P'", e_psel},
-  {"P%", e_psel},
-  {"RP'", e_rpsel},
-  {"RP%", e_rpsel},
-  {"LP'", e_lpsel},
-  {"LP%", e_lpsel},
-  {"T'", e_tsel},
-  {"T%", e_tsel},
-  {"RT'", e_rtsel},
-  {"RT%", e_rtsel},
-  {"LT'", e_ltsel},
-  {"LT%", e_ltsel},
-  {NULL, e_fsel}
+  {"f", e_fsel},
+  {"l", e_lsel},
+  {"ld", e_ldsel},
+  {"lp", e_lpsel},
+  {"lr", e_lrsel},
+  {"ls", e_lssel},
+  {"lt", e_ltsel},
+  {"p", e_psel},
+  {"r", e_rsel},
+  {"rd", e_rdsel},
+  {"rp", e_rpsel},
+  {"rr", e_rrsel},
+  {"rs", e_rssel},
+  {"rt", e_rtsel},
+  {"t", e_tsel},
 };
 
 /* default space and subspace dictionaries */
@@ -1168,26 +1056,10 @@ static struct default_space_dict pa_def_spaces[] =
 /* These macros are used to maintain spaces/subspaces.  */
 #define SPACE_DEFINED(space_chain)     (space_chain)->sd_defined
 #define SPACE_USER_DEFINED(space_chain) (space_chain)->sd_user_defined
-#define SPACE_PRIVATE(space_chain)     (space_chain)->sd_private
-#define SPACE_LOADABLE(space_chain)    (space_chain)->sd_loadable
 #define SPACE_SPNUM(space_chain)       (space_chain)->sd_spnum
-#define SPACE_SORT(space_chain)                (space_chain)->sd_sort_key
 #define SPACE_NAME(space_chain)                (space_chain)->sd_name
-#define SPACE_NAME_INDEX(space_chain)   (space_chain)->sd_name_index
 
-#define SUBSPACE_SPACE_INDEX(ss_chain)  (ss_chain)->ssd_space_index
 #define SUBSPACE_DEFINED(ss_chain)     (ss_chain)->ssd_defined
-#define SUBSPACE_QUADRANT(ss_chain)    (ss_chain)->ssd_quadrant
-#define SUBSPACE_ALIGN(ss_chain)       (ss_chain)->ssd_alignment
-#define SUBSPACE_ACCESS(ss_chain)      (ss_chain)->ssd_access_control_bits
-#define SUBSPACE_SORT(ss_chain)                (ss_chain)->ssd_sort_key
-#define SUBSPACE_COMMON(ss_chain)      (ss_chain)->ssd_common
-#define SUBSPACE_ZERO(ss_chain)                (ss_chain)->ssd_zero
-#define SUBSPACE_DUP_COMM(ss_chain)    (ss_chain)->ssd_dup_common
-#define SUBSPACE_CODE_ONLY(ss_chain)    (ss_chain)->ssd_code_only
-#define SUBSPACE_LOADABLE(ss_chain)    (ss_chain)->ssd_loadable
-#define SUBSPACE_SUBSPACE_START(ss_chain) (ss_chain)->ssd_subspace_start
-#define SUBSPACE_SUBSPACE_LENGTH(ss_chain) (ss_chain)->ssd_subspace_length
 #define SUBSPACE_NAME(ss_chain)                (ss_chain)->ssd_name
 
 /* Insert FIELD into OPCODE starting at bit START.  Continue pa_ip
@@ -1380,20 +1252,20 @@ cons_fix_new_hppa (frag, where, size, exp)
      int size;
      expressionS *exp;
 {
-  unsigned int reloc_type;
+  unsigned int rel_type;
 
   if (is_DP_relative (*exp))
-    reloc_type = R_HPPA_GOTOFF;
+    rel_type = R_HPPA_GOTOFF;
   else if (is_complex (*exp))
-    reloc_type = R_HPPA_COMPLEX;
+    rel_type = R_HPPA_COMPLEX;
   else
-    reloc_type = R_HPPA;
+    rel_type = R_HPPA;
 
   if (hppa_field_selector != e_psel && hppa_field_selector != e_fsel)
     as_warn ("Invalid field selector.  Assuming F%%.");
 
   fix_new_hppa (frag, where, size,
-               (symbolS *) NULL, (offsetT) 0, exp, 0, reloc_type,
+               (symbolS *) NULL, (offsetT) 0, exp, 0, rel_type,
                hppa_field_selector, 32, 0, (char *) 0);
 
   /* Reset field selector to its default state.  */
@@ -1425,8 +1297,6 @@ md_begin ()
   pa_spaces_begin ();
 
   op_hash = hash_new ();
-  if (op_hash == NULL)
-    as_fatal ("Virtual memory exhausted");
 
   while (i < NUMOPCODES)
     {
@@ -1457,15 +1327,9 @@ md_begin ()
   /* SOM will change text_section.  To make sure we never put
      anything into the old one switch to the new one now.  */
   subseg_set (text_section, 0);
-}
-
-/* Called at the end of assembling a source file.  Nothing to do
-   at this point on the PA.  */
 
-void
-md_end ()
-{
-  return;
+  dummy_symbol = symbol_find_or_make ("L$dummy");
+  dummy_symbol->section = text_section;
 }
 
 /* Assemble a single instruction storing it into a frag.  */
@@ -1478,6 +1342,27 @@ md_assemble (str)
   /* The had better be something to assemble.  */
   assert (str);
 
+  /* If we are within a procedure definition, make sure we've
+     defined a label for the procedure; handle case where the
+     label was defined after the .PROC directive.  */
+  if (within_procedure && last_call_info->start_symbol == NULL)
+    {
+      label_symbol_struct *label_symbol = pa_get_label ();
+
+      if (label_symbol)
+       {
+         if (label_symbol->lss_label)
+           {
+             last_call_info->start_symbol = label_symbol->lss_label;
+             label_symbol->lss_label->bsym->flags |= BSF_FUNCTION;
+           }
+         else
+           as_bad ("Missing function name for .PROC (corrupted label chain)");
+       }
+      else
+       as_bad ("Missing function name for .PROC");
+    }
+
   /* Assemble the instruction.  Results are saved into "the_insn".  */
   pa_ip (str);
 
@@ -1493,14 +1378,10 @@ md_assemble (str)
                  (offsetT) 0, &the_insn.exp, the_insn.pcrel,
                  the_insn.reloc, the_insn.field_selector,
                  the_insn.format, the_insn.arg_reloc, NULL);
-
 }
 
 /* Do the real work for assembling a single instruction.  Store results
-   into the global "the_insn" variable.
-
-   FIXME:  Should define and use some functions/macros to handle
-   various common insertions of information into the opcode.  */
+   into the global "the_insn" variable.  */
 
 static void
 pa_ip (str)
@@ -1865,17 +1746,17 @@ pa_ip (str)
                      cmpltr = 0;
                      flag = 1;
                    }
-                 else if (strcasecmp (name, "<>") == 0)
+                 else if (strcmp (name, "<>") == 0)
                    {
                      cmpltr = 1;
                      flag = 1;
                    }
-                 else if (strcasecmp (name, ">=") == 0)
+                 else if (strcmp (name, ">=") == 0)
                    {
                      cmpltr = 2;
                      flag = 1;
                    }
-                 else if (strcasecmp (name, ">") == 0)
+                 else if (strcmp (name, ">") == 0)
                    {
                      cmpltr = 3;
                      flag = 1;
@@ -2542,7 +2423,6 @@ pa_ip (str)
     }
 
   the_insn.opcode = opcode;
-  return;
 }
 
 /* Turn a string in input_line_pointer into a floating point constant of type
@@ -2613,21 +2493,7 @@ md_number_to_chars (buf, val, n)
      valueT val;
      int n;
 {
-
-  switch (n)
-    {
-    case 4:
-      *buf++ = val >> 24;
-      *buf++ = val >> 16;
-    case 2:
-      *buf++ = val >> 8;
-    case 1:
-      *buf = val;
-      break;
-    default:
-      abort ();
-    }
-  return;
+  number_to_chars_bigendian (buf, val, n);
 }
 
 /* Translate internal representation of relocation info to BFD target
@@ -2786,9 +2652,8 @@ tc_gen_reloc (section, fixp)
             relocation should be either 0 (no static link) or 2
             (static link required).
 
-            FIXME: assume that fx_addnumber contains this
-            information */
-         reloc->addend = fixp->fx_addnumber;
+            FIXME: We always assume no static link!  */
+         reloc->addend = 0;
          break;
 
        case R_HPPA_ABS_CALL_11:
@@ -2839,31 +2704,20 @@ tc_gen_reloc (section, fixp)
     }
 #else /* OBJ_SOM */
 
-  /* Preliminary relocation handling for SOM.  Needs to handle
-     COMPLEX relocations (yes, I've seen them occur) and it will
-     need to handle R_ENTRY/R_EXIT relocations in the very near future
-     (for generating unwinds).  */
-  switch (fixp->fx_r_type)
+  /* Walk over reach relocation returned by the BFD backend.  */
+  for (i = 0; i < n_relocs; i++)
     {
-    case R_HPPA_COMPLEX:
-    case R_HPPA_COMPLEX_PCREL_CALL:
-    case R_HPPA_COMPLEX_ABS_CALL:
-      abort ();
-      break;
-    default:
-      assert (n_relocs == 1);
+      code = *codes[i];
 
-      code = *codes[0];
-
-      reloc->sym_ptr_ptr = &fixp->fx_addsy->bsym;
-      reloc->howto = bfd_reloc_type_lookup (stdoutput, code);
-      reloc->address = fixp->fx_frag->fr_address + fixp->fx_where;
+      relocs[i]->sym_ptr_ptr = &fixp->fx_addsy->bsym;
+      relocs[i]->howto = bfd_reloc_type_lookup (stdoutput, code);
+      relocs[i]->address = fixp->fx_frag->fr_address + fixp->fx_where;
 
       switch (code)
        {
        case R_PCREL_CALL:
        case R_ABS_CALL:
-         reloc->addend = HPPA_R_ADDEND (hppa_fixp->fx_arg_reloc, 0);
+         relocs[i]->addend = HPPA_R_ADDEND (hppa_fixp->fx_arg_reloc, 0);
          break;
 
        case R_DATA_PLABEL:
@@ -2873,14 +2727,26 @@ tc_gen_reloc (section, fixp)
             (static link required).
 
             FIXME: We always assume no static link!  */
-         reloc->addend = 0;
+         relocs[i]->addend = 0;
+         break;
+
+       case R_N_MODE:
+       case R_S_MODE:
+       case R_D_MODE:
+       case R_R_MODE:
+       case R_EXIT:
+       case R_FSEL:
+       case R_LSEL:
+       case R_RSEL:
+         /* There is no symbol or addend associated with these fixups.  */
+         relocs[i]->sym_ptr_ptr = dummy_symbol;
+         relocs[i]->addend = 0;
          break;
 
        default:
-         reloc->addend = fixp->fx_addnumber;
+         relocs[i]->addend = fixp->fx_addnumber;
          break;
        }
-      break;
     }
 #endif
 
@@ -2931,7 +2797,6 @@ md_section_align (segment, size)
   int align2 = (1 << align) - 1;
 
   return (size + align2) & ~align2;
-
 }
 
 /* Create a short jump from FROM_ADDR to TO_ADDR.  Not used on the PA.  */
@@ -3003,28 +2868,25 @@ md_operand (expressionP)
 {
 }
 
-/* Helper function for md_apply_fix.  Actually determine if the fix
-   can be applied, and if so, apply it.
-
-   If a fix is applied, then set fx_addsy to NULL which indicates
-   the fix was applied and need not be emitted into the object file.  */
+/* Apply a fixup to an instruction.  */
 
-static void
-md_apply_fix_1 (fixP, val)
+int
+md_apply_fix (fixP, valp)
      fixS *fixP;
-     long val;
+     valueT *valp;
 {
   char *buf = fixP->fx_where + fixP->fx_frag->fr_literal;
   struct hppa_fix_struct *hppa_fixP = fixP->tc_fix_data;
   long new_val, result;
   unsigned int w1, w2, w;
+  valueT val = *valp;
 
   /* SOM uses R_HPPA_ENTRY and R_HPPA_EXIT relocations which can 
-     never be "applied".  They must always be emitted.  */
+     never be "applied" (they are just markers).  */
 #ifdef OBJ_SOM
   if (fixP->fx_r_type == R_HPPA_ENTRY
       || fixP->fx_r_type == R_HPPA_EXIT)
-    return;
+    return 1;
 #endif
 
   /* There should have been an HPPA specific fixup associated
@@ -3046,11 +2908,15 @@ md_apply_fix_1 (fixP, val)
       if ((fixP->fx_addsy && fixP->fx_addsy->bsym->section == &bfd_und_section)
          || (fixP->fx_subsy
              && fixP->fx_subsy->bsym->section == &bfd_und_section))
-       return;
+       return 1;
 
+      /* PLABEL field selectors should not be passed to hppa_field_adjust.  */
       if (fmt != 0 && hppa_fixP->fx_r_field != R_HPPA_PSEL
          && hppa_fixP->fx_r_field != R_HPPA_LPSEL
-         && hppa_fixP->fx_r_field != R_HPPA_RPSEL)
+         && hppa_fixP->fx_r_field != R_HPPA_RPSEL
+         && hppa_fixP->fx_r_field != R_HPPA_TSEL
+         && hppa_fixP->fx_r_field != R_HPPA_LTSEL
+         && hppa_fixP->fx_r_field != R_HPPA_RTSEL)
        new_val = hppa_field_adjust (val, 0, hppa_fixP->fx_r_field);
       else
        new_val = 0;
@@ -3104,21 +2970,22 @@ md_apply_fix_1 (fixP, val)
          result = ((w1 << 2) | w);
          break;
 
+       /* Handle some of the opcodes with the 'W' operand type.  */
+       case 17:
+
 #define stub_needed(CALLER, CALLEE) \
   ((CALLEE) && (CALLER) && ((CALLEE) != (CALLER)))
+       /* It is necessary to force PC-relative calls/jumps to have a
+          relocation entry if they're going to need either a argument
+          relocation or long call stub.  FIXME.  Can't we need the same
+          for absolute calls?  */
+       if (fixP->fx_addsy
+           && (stub_needed (((obj_symbol_type *)
+                             fixP->fx_addsy->bsym)->tc_data.hppa_arg_reloc,
+                            hppa_fixP->fx_arg_reloc)))
+         return 1;
+#undef stub_needed
 
-       /* Handle some of the opcodes with the 'W' operand type.  */
-       case 17:
-         /* If a long-call stub or argument relocation stub is
-            needed, then we can not apply this relocation, instead
-            the linker must handle it.  */
-         if (new_val > 262143 || new_val < -262144
-             || stub_needed (((obj_symbol_type *)
-                              fixP->fx_addsy->bsym)->tc_data.hppa_arg_reloc,
-                             hppa_fixP->fx_arg_reloc))
-           return;
-
-         /* No stubs were needed, we can perform this relocation.  */
          CHECK_FIELD (new_val, 262143, -262144, 0);
 
          /* Mask off 17 bits to be changed.  */
@@ -3130,9 +2997,6 @@ md_apply_fix_1 (fixP, val)
          result = ((w2 << 2) | (w1 << 16) | w);
          break;
 
-#undef too_far
-#undef stub_needed
-
        case 32:
 #ifdef OBJ_ELF
          /* These are ELF specific relocations.  ELF unfortunately
@@ -3145,36 +3009,36 @@ md_apply_fix_1 (fixP, val)
            {
              result = 0;
              fixP->fx_addnumber = fixP->fx_offset;
-             bfd_put_32 (stdoutput, 0, buf);
-             return;
+             /* If we have a real relocation, then we want zero to
+                be stored in the object file.  If no relocation is going
+                to be emitted, then we need to store new_val into the
+                object file.  */
+             if (fixP->fx_addsy)
+               bfd_put_32 (stdoutput, 0, buf);
+             else
+               bfd_put_32 (stdoutput, new_val, buf);
+             return 1;
            }
          break;
 
        case 0:
-         return;
+         return 1;
 
        default:
          as_bad ("Unknown relocation encountered in md_apply_fix.");
-         return;
+         return 1;
        }
 
       /* Insert the relocation.  */
       bfd_put_32 (stdoutput, bfd_get_32 (stdoutput, buf) | result, buf);
+      return 1;
     }
   else
-    printf ("no hppa_fixup entry for this fixup (fixP = 0x%x, type = 0x%x)\n",
-           (unsigned int) fixP, fixP->fx_r_type);
-}
-
-/* Apply a fix into a frag's data (if possible).  */
-
-int
-md_apply_fix (fixP, valp)
-     fixS *fixP;
-     valueT *valp;
-{
-  md_apply_fix_1 (fixP, (long) *valp);
-  return 1;
+    {
+      printf ("no hppa_fixup entry for this fixup (fixP = 0x%x, type = 0x%x)\n",
+             (unsigned int) fixP, fixP->fx_r_type);
+      return 0;
+    }
 }
 
 /* Exactly what point is a PC-relative offset relative TO?
@@ -3424,6 +3288,7 @@ reg_name_search (name)
      char *name;
 {
   int middle, low, high;
+  int cmp;
 
   low = 0;
   high = REG_NAME_CNT - 1;
@@ -3431,18 +3296,17 @@ reg_name_search (name)
   do
     {
       middle = (low + high) / 2;
-      if (strcasecmp (name, pre_defined_registers[middle].name) < 0)
+      cmp = strcasecmp (name, pre_defined_registers[middle].name);
+      if (cmp < 0)
        high = middle - 1;
-      else
+      else if (cmp > 0)
        low = middle + 1;
+      else
+       return pre_defined_registers[middle].value;
     }
-  while (!((strcasecmp (name, pre_defined_registers[middle].name) == 0) ||
-          (low > high)));
+  while (low <= high);
 
-  if (strcasecmp (name, pre_defined_registers[middle].name) == 0)
-    return (pre_defined_registers[middle].value);
-  else
-    return (-1);
+  return -1;
 }
 
 
@@ -3532,28 +3396,45 @@ static int
 pa_chk_field_selector (str)
      char **str;
 {
-  int selector;
-  const struct selector_entry *tablep;
-
-  selector = e_fsel;
+  int middle, low, high;
+  int cmp;
+  char name[3];
 
   /* Read past any whitespace.  */
+  /* FIXME: should we read past newlines and formfeeds??? */
   while (**str == ' ' || **str == '\t' || **str == '\n' || **str == '\f')
     *str = *str + 1;
 
-  /* Yuk.  Looks like a linear search through the table.  With the
-     frequence of some selectors it might make sense to sort the
-     table by usage.  */
-  for (tablep = selector_table; tablep->prefix; tablep++)
+  if ((*str)[1] == '\'' || (*str)[1] == '%')
+    name[0] = tolower ((*str)[0]),
+    name[1] = 0;
+  else if ((*str)[2] == '\'' || (*str)[2] == '%')
+    name[0] = tolower ((*str)[0]),
+    name[1] = tolower ((*str)[1]),
+    name[2] = 0;
+  else
+    return e_fsel;
+
+  low = 0;
+  high = sizeof (selector_table) / sizeof (struct selector_entry) - 1;
+
+  do
     {
-      if (strncasecmp (tablep->prefix, *str, strlen (tablep->prefix)) == 0)
+      middle = (low + high) / 2;
+      cmp = strcmp (name, selector_table[middle].prefix);
+      if (cmp < 0)
+       high = middle - 1;
+      else if (cmp > 0)
+       low = middle + 1;
+      else
        {
-         *str += strlen (tablep->prefix);
-         selector = tablep->field_selector;
-         break;
+         *str += strlen (name) + 1;
+         return selector_table[middle].field_selector;
        }
     }
-  return selector;
+  while (low <= high);
+
+  return e_fsel;
 }
 
 /* Mark (via expr_end) the end of an expression (I think).  FIXME.  */
@@ -3653,8 +3534,6 @@ evaluate_absolute (insn)
     /* Add 0x800 and arithmetic shift right 11 bits.  */
     case e_ldsel:
       value += 0x800;
-
-
       value = (value & 0xfffff800) >> 11;
       break;
 
@@ -3663,11 +3542,15 @@ evaluate_absolute (insn)
       value |= 0xfffff800;
       break;
 
-    /* This had better get fixed.  It looks like we're quickly moving
-       to LR/RR.  FIXME.  */
+#define RSEL_ROUND(c)  (((c) + 0x1000) & ~0x1fff)
     case e_rrsel:
+      value = (RSEL_ROUND (value) & 0x7ff) + (value - RSEL_ROUND (value));
+      break;
+
     case e_lrsel:
-      abort ();
+      value = (RSEL_ROUND (value) >> 11) & 0x1fffff;
+      break;
+#undef RSEL_ROUND
 
     default:
       BAD_CASE (field_selector);
@@ -4020,11 +3903,11 @@ pa_parse_neg_add_cmpltr (s, isbranch)
        {
          cmpltr = 3;
        }
-      else if (strcmp (name, "uv") == 0)
+      else if (strcasecmp (name, "uv") == 0)
        {
          cmpltr = 4;
        }
-      else if (strcmp (name, "vnz") == 0)
+      else if (strcasecmp (name, "vnz") == 0)
        {
          cmpltr = 5;
        }
@@ -4087,7 +3970,6 @@ pa_block (z)
 
   pa_undefine_label ();
   demand_empty_rest_of_line ();
-  return;
 }
 
 /* Handle a .CALL pseudo-op.  This involves storing away information
@@ -4100,7 +3982,6 @@ pa_call (unused)
 {
   pa_call_args (&last_call_desc);
   demand_empty_rest_of_line ();
-  return;
 }
 
 /* Do the dirty work of building a call descriptor which describes
@@ -4208,8 +4089,6 @@ pa_build_unwind_subspace (call_info)
   /* Get some space to hold relocation information for the unwind
      descriptor.  */
   p = frag_more (4);
-  call_info->start_offset_frag = frag_now;
-  call_info->start_frag_where = p - frag_now->fr_literal;
 
   /* Relocation info. for start offset of the function.  */
   fix_new_hppa (frag_now, p - frag_now->fr_literal, 4,
@@ -4217,26 +4096,7 @@ pa_build_unwind_subspace (call_info)
                (expressionS *) NULL, 0, R_HPPA_UNWIND, e_fsel, 32, 0,
                (char *) 0);
 
-  /* We need to search for the first relocation involving the start_symbol of
-     this call_info descriptor.  */
-  {
-    fixS *fixP;
-
-    call_info->start_fix = seg_info (now_seg)->fix_root;
-    for (fixP = call_info->start_fix; fixP; fixP = fixP->fx_next)
-      {
-       if (fixP->fx_addsy == call_info->start_symbol
-           || fixP->fx_subsy == call_info->start_symbol)
-         {
-           call_info->start_fix = fixP;
-           break;
-         }
-      }
-  }
-
   p = frag_more (4);
-  call_info->end_offset_frag = frag_now;
-  call_info->end_frag_where = p - frag_now->fr_literal;
 
   /* Relocation info. for end offset of the function.  */
   fix_new_hppa (frag_now, p - frag_now->fr_literal, 4,
@@ -4244,23 +4104,6 @@ pa_build_unwind_subspace (call_info)
                (expressionS *) NULL, 0, R_HPPA_UNWIND, e_fsel, 32, 0,
                (char *) 0);
 
-  /* We need to search for the first relocation involving the end_symbol of
-     this call_info descriptor.  */
-  {
-    fixS *fixP;
-
-    call_info->end_fix = seg_info (now_seg)->fix_root; /* the default */
-    for (fixP = call_info->end_fix; fixP; fixP = fixP->fx_next)
-      {
-       if (fixP->fx_addsy == call_info->end_symbol
-           || fixP->fx_subsy == call_info->end_symbol)
-         {
-           call_info->end_fix = fixP;
-           break;
-         }
-      }
-  }
-
   /* Dump it. */
   unwind = (char *) &call_info->ci_unwind;
   for (i = 8; i < sizeof (struct unwind_table); i++)
@@ -4351,7 +4194,6 @@ pa_callinfo (unused)
          temp = get_absolute_expression ();
          if (temp != 3)
            as_bad ("Value for ENTRY_SR must be 3\n");
-         last_call_info->entry_sr = temp - 2;
        }
       /* Note whether or not this function performs any calls.  */
       else if ((strncasecmp (name, "calls", 5) == 0) ||
@@ -4359,13 +4201,11 @@ pa_callinfo (unused)
        {
          p = input_line_pointer;
          *p = c;
-         last_call_info->makes_calls = 1;
        }
       else if ((strncasecmp (name, "no_calls", 8) == 0))
        {
          p = input_line_pointer;
          *p = c;
-         last_call_info->makes_calls = 0;
        }
       /* Should RP be saved into the stack.  */
       else if ((strncasecmp (name, "save_rp", 7) == 0))
@@ -4406,7 +4246,6 @@ pa_callinfo (unused)
     }
 
   demand_empty_rest_of_line ();
-  return;
 }
 
 /* Switch into the code subspace.  */
@@ -4433,7 +4272,6 @@ pa_code (unused)
   SPACE_DEFINED (sdchain) = 1;
   subseg_set (text_section, SUBSEG_CODE);
   demand_empty_rest_of_line ();
-  return;
 }
 
 /* This is different than the standard GAS s_comm(). On HP9000/800 machines,
@@ -4487,39 +4325,6 @@ pa_comm (unused)
   demand_empty_rest_of_line ();
 }
 
-/* Process a .COPYRIGHT pseudo-op.  */
-
-static void
-pa_copyright (unused)
-     int unused;
-{
-  char *name;
-  char c;
-
-  SKIP_WHITESPACE ();
-  if (*input_line_pointer == '\"')
-    {
-      ++input_line_pointer;
-      name = input_line_pointer;
-      while ((c = next_char_of_string ()) >= 0)
-       ;
-      c = *input_line_pointer;
-      *input_line_pointer = '\0';
-      *(input_line_pointer - 1) = '\0';
-      {
-       /* FIXME.  Not supported */
-       abort ();
-      }
-      *input_line_pointer = c;
-    }
-  else
-    {
-      as_bad ("Expected \"-ed string");
-    }
-  pa_undefine_label ();
-  demand_empty_rest_of_line ();
-}
-
 /* Process a .END pseudo-op.  */
 
 static void
@@ -4527,7 +4332,6 @@ pa_end (unused)
      int unused;
 {
   demand_empty_rest_of_line ();
-  return;
 }
 
 /* Process a .ENTER pseudo-op.  This is not supported.  */
@@ -4536,7 +4340,6 @@ pa_enter (unused)
      int unused;
 {
   abort ();
-  return;
 }
 
 /* Process a .ENTRY pseudo-op.  .ENTRY marks the beginning of the
@@ -4551,8 +4354,6 @@ pa_entry (unused)
     {
       if (!callinfo_found)
        as_bad ("Missing .callinfo.");
-
-      last_call_info->start_frag = frag_now;
     }
   demand_empty_rest_of_line ();
   within_entry_exit = TRUE;
@@ -4580,8 +4381,6 @@ pa_entry (unused)
                  (char *) &last_call_info->ci_unwind.descriptor);
   }
 #endif
-
-  return;
 }
 
 /* Handle a .EQU pseudo-op.  */
@@ -4609,7 +4408,6 @@ pa_equ (reg)
 
   pa_undefine_label ();
   demand_empty_rest_of_line ();
-  return;
 }
 
 /* Helper function.  Does processing for the end of a function.  This
@@ -4628,7 +4426,6 @@ process_exit ()
      for the end of the function, and finally call pa_build_unwind_subspace
      to add an entry in the unwind table.  */
   hppa_elf_mark_end_of_function ();
-  last_call_info->end_frag = frag_now;
   pa_build_unwind_subspace (last_call_info);
 #else
   /* SOM defers building of unwind descriptors until the link phase.
@@ -4644,7 +4441,6 @@ process_exit ()
                last_call_info->start_symbol, (offsetT) 0,
                NULL, 0, R_HPPA_EXIT, e_fsel, 0, 0, NULL);
 #endif
-
 }
 
 /* Process a .EXIT pseudo-op.  */
@@ -4671,7 +4467,6 @@ pa_exit (unused)
        }
     }
   demand_empty_rest_of_line ();
-  return;
 }
 
 /* Process a .EXPORT directive.  This makes functions external
@@ -4712,7 +4507,6 @@ pa_export (unused)
     }
 
   demand_empty_rest_of_line ();
-  return;
 }
 
 /* Helper function to process arguments to a .EXPORT pseudo-op.  */
@@ -4894,7 +4688,6 @@ pa_import (unused)
     }
 
   demand_empty_rest_of_line ();
-  return;
 }
 
 /* Handle a .LABEL pseudo-op.  */
@@ -4925,7 +4718,6 @@ pa_label (unused)
       ignore_rest_of_line ();
     }
   demand_empty_rest_of_line ();
-  return;
 }
 
 /* Handle a .LEAVE pseudo-op.  This is not supported yet.  */
@@ -4945,7 +4737,6 @@ pa_origin (unused)
 {
   s_org (0);
   pa_undefine_label ();
-  return;
 }
 
 /* Handle a .PARAM pseudo-op.  This is much like a .EXPORT, except it
@@ -4981,7 +4772,6 @@ pa_param (unused)
     }
 
   demand_empty_rest_of_line ();
-  return;
 }
 
 /* Handle a .PROC pseudo-op.  It is used to mark the beginning
@@ -5026,8 +4816,6 @@ pa_proc (unused)
   call_info->ci_unwind.descriptor.cannot_unwind = 0;
   call_info->ci_unwind.descriptor.region_desc = 1;
   call_info->ci_unwind.descriptor.hpux_interrupt_marker = 0;
-  call_info->entry_sr = ~0;
-  call_info->makes_calls = 1;
 
   /* If we got a .PROC pseudo-op, we know that the function is defined
      locally.  Make sure it gets into the symbol table.  */
@@ -5042,14 +4830,13 @@ pa_proc (unused)
            label_symbol->lss_label->bsym->flags |= BSF_FUNCTION;
          }
        else
-         as_bad ("Missing function name for .PROC (corrupted label)");
+         as_bad ("Missing function name for .PROC (corrupted label chain)");
       }
     else
-      as_bad ("Missing function name for .PROC");
+      last_call_info->start_symbol = NULL;
   }
 
   demand_empty_rest_of_line ();
-  return;
 }
 
 /* Process the syntatical end of a procedure.  Make sure all the 
@@ -5077,7 +4864,6 @@ pa_procend (unused)
 
   within_procedure = FALSE;
   demand_empty_rest_of_line ();
-  return;
 }
 
 /* Parse the parameters to a .SPACE directive; if CREATE_FLAG is nonzero,
@@ -5101,12 +4887,12 @@ pa_parse_space_stmt (space_name, create_flag)
   loadable = TRUE;
   defined = TRUE;
   private = FALSE;
-  if (strcasecmp (space_name, "$TEXT$") == 0)
+  if (strcmp (space_name, "$TEXT$") == 0)
     {
       seg = pa_def_spaces[0].segment;
       sort = pa_def_spaces[0].sort;
     }
-  else if (strcasecmp (space_name, "$PRIVATE$") == 0)
+  else if (strcmp (space_name, "$PRIVATE$") == 0)
     {
       seg = pa_def_spaces[1].segment;
       sort = pa_def_spaces[1].sort;
@@ -5128,29 +4914,29 @@ pa_parse_space_stmt (space_name, create_flag)
              input_line_pointer++;
              name = input_line_pointer;
              c = get_symbol_end ();
-             if ((strncasecmp (name, "SPNUM", 5) == 0))
+             if ((strncasecmp (name, "spnum", 5) == 0))
                {
                  *input_line_pointer = c;
                  input_line_pointer++;
                  spnum = get_absolute_expression ();
                }
-             else if ((strncasecmp (name, "SORT", 4) == 0))
+             else if ((strncasecmp (name, "sort", 4) == 0))
                {
                  *input_line_pointer = c;
                  input_line_pointer++;
                  sort = get_absolute_expression ();
                }
-             else if ((strncasecmp (name, "UNLOADABLE", 10) == 0))
+             else if ((strncasecmp (name, "unloadable", 10) == 0))
                {
                  *input_line_pointer = c;
                  loadable = FALSE;
                }
-             else if ((strncasecmp (name, "NOTDEFINED", 10) == 0))
+             else if ((strncasecmp (name, "notdefined", 10) == 0))
                {
                  *input_line_pointer = c;
                  defined = FALSE;
                }
-             else if ((strncasecmp (name, "PRIVATE", 7) == 0))
+             else if ((strncasecmp (name, "private", 7) == 0))
                {
                  *input_line_pointer = c;
                  private = TRUE;
@@ -5181,11 +4967,8 @@ pa_parse_space_stmt (space_name, create_flag)
     {
       space = is_defined_space (space_name);
       SPACE_SPNUM (space) = spnum;
-      SPACE_LOADABLE (space) = loadable & 1;
       SPACE_DEFINED (space) = defined & 1;
       SPACE_USER_DEFINED (space) = 1;
-      SPACE_PRIVATE (space) = private & 1;
-      SPACE_SORT (space) = sort & 0xff;
       space->sd_seg = seg;
     }
 
@@ -5217,7 +5000,9 @@ pa_space (unused)
       /* Check for some of the predefined spaces.   FIXME: most of the code
          below is repeated several times, can we extract the common parts
          and place them into a subroutine or something similar?  */
-      if (strncasecmp (input_line_pointer, "$text$", 6) == 0)
+      /* FIXME Is this (and the next IF stmt) really right?
+        What if INPUT_LINE_POINTER points to "$TEXT$FOO"?  */
+      if (strncmp (input_line_pointer, "$TEXT$", 6) == 0)
        {
          input_line_pointer += 6;
          sd_chain = is_defined_space ("$TEXT$");
@@ -5234,7 +5019,7 @@ pa_space (unused)
          demand_empty_rest_of_line ();
          return;
        }
-      if (strncasecmp (input_line_pointer, "$private$", 9) == 0)
+      if (strncmp (input_line_pointer, "$PRIVATE$", 9) == 0)
        {
          input_line_pointer += 9;
          sd_chain = is_defined_space ("$PRIVATE$");
@@ -5312,7 +5097,6 @@ pa_space (unused)
                                                  sd_chain->sd_last_subseg);
       demand_empty_rest_of_line ();
     }
-  return;
 }
 
 /* Switch to a new space.  (I think).  FIXME.  */
@@ -5339,7 +5123,6 @@ pa_spnum (unused)
 
   *input_line_pointer = c;
   demand_empty_rest_of_line ();
-  return;
 }
 
 /* If VALUE is an exact power of two between zero and 2^31, then 
@@ -5451,13 +5234,13 @@ pa_subspace (unused)
            {
              name = input_line_pointer;
              c = get_symbol_end ();
-             if ((strncasecmp (name, "QUAD", 4) == 0))
+             if ((strncasecmp (name, "quad", 4) == 0))
                {
                  *input_line_pointer = c;
                  input_line_pointer++;
                  quadrant = get_absolute_expression ();
                }
-             else if ((strncasecmp (name, "ALIGN", 5) == 0))
+             else if ((strncasecmp (name, "align", 5) == 0))
                {
                  *input_line_pointer = c;
                  input_line_pointer++;
@@ -5468,44 +5251,44 @@ pa_subspace (unused)
                      alignment = 1;
                    }
                }
-             else if ((strncasecmp (name, "ACCESS", 6) == 0))
+             else if ((strncasecmp (name, "access", 6) == 0))
                {
                  *input_line_pointer = c;
                  input_line_pointer++;
                  access = get_absolute_expression ();
                }
-             else if ((strncasecmp (name, "SORT", 4) == 0))
+             else if ((strncasecmp (name, "sort", 4) == 0))
                {
                  *input_line_pointer = c;
                  input_line_pointer++;
                  sort = get_absolute_expression ();
                }
-             else if ((strncasecmp (name, "CODE_ONLY", 9) == 0))
+             else if ((strncasecmp (name, "code_only", 9) == 0))
                {
                  *input_line_pointer = c;
                  code_only = 1;
                }
-             else if ((strncasecmp (name, "UNLOADABLE", 10) == 0))
+             else if ((strncasecmp (name, "unloadable", 10) == 0))
                {
                  *input_line_pointer = c;
                  loadable = 0;
                }
-             else if ((strncasecmp (name, "COMMON", 6) == 0))
+             else if ((strncasecmp (name, "common", 6) == 0))
                {
                  *input_line_pointer = c;
                  common = 1;
                }
-             else if ((strncasecmp (name, "DUP_COMM", 8) == 0))
+             else if ((strncasecmp (name, "dup_comm", 8) == 0))
                {
                  *input_line_pointer = c;
                  dup_common = 1;
                }
-             else if ((strncasecmp (name, "ZERO", 4) == 0))
+             else if ((strncasecmp (name, "zero", 4) == 0))
                {
                  *input_line_pointer = c;
                  zero = 1;
                }
-             else if ((strncasecmp (name, "FIRST", 5) == 0))
+             else if ((strncasecmp (name, "first", 5) == 0))
                as_bad ("FIRST not supported as a .SUBSPACE argument");
              else
                as_bad ("Invalid .SUBSPACE argument");
@@ -5583,7 +5366,6 @@ pa_subspace (unused)
       subseg_set (current_subspace->ssd_seg, current_subspace->ssd_subseg);
     }
   SUBSPACE_DEFINED (current_subspace) = 1;
-  return;
 }
 
 
@@ -5700,10 +5482,7 @@ pa_spaces_begin ()
 
 
 /* Create a new space NAME, with the appropriate flags as defined
-   by the given parameters.
-
-   Add the new space to the space dictionary chain in numerical
-   order as defined by the SORT entries.  */
+   by the given parameters.  */
 
 static sd_chain_struct *
 create_new_space (name, spnum, loadable, defined, private,
@@ -5726,13 +5505,9 @@ create_new_space (name, spnum, loadable, defined, private,
 
   SPACE_NAME (chain_entry) = (char *) xmalloc (strlen (name) + 1);
   strcpy (SPACE_NAME (chain_entry), name);
-  SPACE_NAME_INDEX (chain_entry) = 0;
-  SPACE_LOADABLE (chain_entry) = loadable;
   SPACE_DEFINED (chain_entry) = defined;
   SPACE_USER_DEFINED (chain_entry) = user_defined;
-  SPACE_PRIVATE (chain_entry) = private;
   SPACE_SPNUM (chain_entry) = spnum;
-  SPACE_SORT (chain_entry) = sort;
 
   chain_entry->sd_seg = seg;
   chain_entry->sd_last_subseg = -1;
@@ -5754,13 +5529,8 @@ create_new_space (name, spnum, loadable, defined, private,
 
       while (chain_pointer)
        {
-         if (SPACE_SORT (chain_pointer) <= SPACE_SORT (chain_entry))
-           {
-             prev_chain_pointer = chain_pointer;
-             chain_pointer = chain_pointer->sd_next;
-           }
-         else
-           break;
+         prev_chain_pointer = chain_pointer;
+         chain_pointer = chain_pointer->sd_next;
        }
 
       /* At this point we've found the correct place to add the new
@@ -5820,25 +5590,12 @@ create_new_subspace (space, name, loadable, code_only, common,
   SUBSPACE_NAME (chain_entry) = (char *) xmalloc (strlen (name) + 1);
   strcpy (SUBSPACE_NAME (chain_entry), name);
 
-  SUBSPACE_ACCESS (chain_entry) = access;
-  SUBSPACE_LOADABLE (chain_entry) = loadable;
-  SUBSPACE_COMMON (chain_entry) = common;
-  SUBSPACE_DUP_COMM (chain_entry) = dup_common;
-  SUBSPACE_SORT (chain_entry) = sort;
-  SUBSPACE_CODE_ONLY (chain_entry) = code_only;
-  SUBSPACE_ALIGN (chain_entry) = alignment;
-  SUBSPACE_QUADRANT (chain_entry) = quadrant;
-  SUBSPACE_SUBSPACE_START (chain_entry) = pa_subspace_start (space, quadrant);
-  SUBSPACE_SPACE_INDEX (chain_entry) = space_index;
-  SUBSPACE_ZERO (chain_entry) = is_zero;
-
   /* Initialize subspace_defined.  When we hit a .subspace directive
      we'll set it to 1 which "locks-in" the subspace attributes.  */
   SUBSPACE_DEFINED (chain_entry) = 0;
 
   chain_entry->ssd_subseg = USE_ALIASES ? pa_next_subseg (space) : 0;
   chain_entry->ssd_seg = seg;
-  chain_entry->ssd_last_align = 1;
   chain_entry->ssd_next = NULL;
 
   /* Find spot for the new subspace based on its sort key.  */
@@ -5854,14 +5611,8 @@ create_new_subspace (space, name, loadable, code_only, common,
 
       while (chain_pointer)
        {
-         if (SUBSPACE_SORT (chain_pointer) <= SUBSPACE_SORT (chain_entry))
-           {
-             prev_chain_pointer = chain_pointer;
-             chain_pointer = chain_pointer->ssd_next;
-           }
-         else
-           break;
-
+         prev_chain_pointer = chain_pointer;
+         chain_pointer = chain_pointer->ssd_next;
        }
 
       /* Now we have somewhere to put the new entry.  Insert it and update
@@ -5884,7 +5635,6 @@ create_new_subspace (space, name, loadable, code_only, common,
 #endif
 
   return chain_entry;
-
 }
 
 /* Update the information for the given subspace based upon the
@@ -5909,21 +5659,7 @@ update_subspace (space, name, loadable, code_only, common, dup_common, sort,
 {
   ssd_chain_struct *chain_entry;
 
-  if ((chain_entry = is_defined_subspace (name)))
-    {
-      SUBSPACE_ACCESS (chain_entry) = access;
-      SUBSPACE_LOADABLE (chain_entry) = loadable;
-      SUBSPACE_COMMON (chain_entry) = common;
-      SUBSPACE_DUP_COMM (chain_entry) = dup_common;
-      SUBSPACE_CODE_ONLY (chain_entry) = 1;
-      SUBSPACE_SORT (chain_entry) = sort;
-      SUBSPACE_ALIGN (chain_entry) = alignment;
-      SUBSPACE_QUADRANT (chain_entry) = quadrant;
-      SUBSPACE_SPACE_INDEX (chain_entry) = space_index;
-      SUBSPACE_ZERO (chain_entry) = zero;
-    }
-  else
-    chain_entry = NULL;
+  chain_entry = is_defined_subspace (name);
 
 #ifdef obj_set_subsection_attributes
   obj_set_subsection_attributes (section, space->sd_seg, access,
@@ -5931,7 +5667,6 @@ update_subspace (space, name, loadable, code_only, common, dup_common, sort,
 #endif
 
   return chain_entry;
-
 }
 
 /* Return the space chain entry for the space with the name NAME or
@@ -6080,7 +5815,7 @@ pa_subspace_start (space, quadrant)
 {
   /* FIXME.  Assumes everyone puts read/write data at 0x4000000, this
      is not correct for the PA OSF1 port.  */
-  if ((strcasecmp (SPACE_NAME (space), "$PRIVATE$") == 0) && quadrant == 1)
+  if ((strcmp (SPACE_NAME (space), "$PRIVATE$") == 0) && quadrant == 1)
     return 0x40000000;
   else if (space->sd_seg == data_section && quadrant == 1)
     return 0x40000000;
@@ -6204,6 +5939,16 @@ pa_version (unused)
   pa_undefine_label ();
 }
 
+/* Handle a .COPYRIGHT pseudo-op.  */
+
+static void
+pa_copyright (unused)
+     int unused;
+{
+  obj_copyright (0);
+  pa_undefine_label ();
+}
+
 /* Just like a normal cons, but when finished we have to undefine
    the latest space label.  */
 
@@ -6225,15 +5970,6 @@ pa_data (unused)
   pa_undefine_label ();
 }
 
-/* FIXME.  What's the purpose of this pseudo-op?  */
-
-static void
-pa_desc (unused)
-     int unused;
-{
-  pa_undefine_label ();
-}
-
 /* Like float_cons, but we need to undefine our label.  */
 
 static void
@@ -6333,7 +6069,7 @@ hppa_force_relocation (fixp)
   /* It is necessary to force PC-relative calls/jumps to have a relocation
      entry if they're going to need either a argument relocation or long
      call stub.  FIXME.  Can't we need the same for absolute calls?  */
-  if (fixp->fx_pcrel
+  if (fixp->fx_pcrel && fixp->fx_addsy
       && (stub_needed (((obj_symbol_type *)
                        fixp->fx_addsy->bsym)->tc_data.hppa_arg_reloc,
                       hppa_fixp->fx_arg_reloc)))
@@ -6404,10 +6140,6 @@ hppa_elf_mark_end_of_function ()
   else
     as_bad ("No memory for symbol name.");
   
-  /* Stuff away the location of the frag for the end of the function,
-     and call pa_build_unwind_subspace to add an entry in the unwind
-     table.  */
-  last_call_info->end_frag = frag_now;
 }
 
 /* Do any symbol processing requested by the target-cpu or target-format.  */
@@ -6458,8 +6190,6 @@ hppa_tc_make_sections (abfd)
      bfd *abfd;
 {
   symext_chainS *symextP;
-  int size, n;
-  asection *symextn_sec;
   segT save_seg = now_seg;
   subsegT save_subseg = now_subseg;
 
@@ -6475,14 +6205,8 @@ hppa_tc_make_sections (abfd)
   if (symext_rootP == NULL)
     return;
 
-  /* Count the number of symbols for the symbol extension section.  */
-  for (n = 0, symextP = symext_rootP; symextP; symextP = symextP->next, ++n)
-    ;
-
-  size = sizeof (symext_entryS) * n;
-
   /* Switch to the symbol extension section.  */
-  symextn_sec = subseg_new (SYMEXTN_SECTION_NAME, 0);
+  subseg_new (SYMEXTN_SECTION_NAME, 0);
 
   frag_wane (frag_now);
   frag_new (0);
@@ -6512,8 +6236,6 @@ hppa_tc_make_sections (abfd)
 
   /* Switch back to the original segment.  */
   subseg_set (save_seg, save_subseg);
-
-  return;
 }
 
 /* Make the symbol extension section.  */
@@ -6563,24 +6285,12 @@ pa_build_symextn_section ()
                         | SEC_ALLOC | SEC_LOAD);
 
   subseg_set (save_seg, save_subseg);
-
 }
 
 /* For ELF, this function serves one purpose:  to setup the st_size
    field of STT_FUNC symbols.  To do this, we need to scan the
-   call_info structure list, determining st_size in one of two possible
-   ways:
-
-   1. call_info->start_frag->fr_fix has the size of the fragment.
-   This approach assumes that the function was built into a
-   single fragment.  This works for most cases, but might fail.
-   For example, if there was a segment change in the middle of
-   the function.
-
-   2. The st_size field is the difference in the addresses of the
-   call_info->start_frag->fr_address field and the fr_address
-   field of the next fragment with fr_type == rs_fill and
-   fr_fix != 0.  */
+   call_info structure list, determining st_size in by taking the
+   difference in the address of the beginning/end marker symbols.  */
 
 void
 elf_hppa_final_processing ()
This page took 0.045543 seconds and 4 git commands to generate.