Fix compile time warning messages.
[deliverable/binutils-gdb.git] / gas / cgen.c
index 7c4d12f3c1043756bcfd657d5b48f196dd3855ae..75138a45039f0733cca81e946ca118f8d99a46fe 100644 (file)
@@ -29,6 +29,8 @@ Foundation, 59 Temple Place - Suite 330, Boston, MA 02111-1307, USA.  */
 #include "cgen.h"
 #include "dwarf2dbg.h"
 
+static void queue_fixup PARAMS ((int, int, expressionS *));
+
 /* Opcode table descriptor, must be set by md_begin.  */
 
 CGEN_CPU_DESC gas_cgen_cpu_desc;
@@ -129,80 +131,84 @@ queue_fixup (opindex, opinfo, expP)
 */
 
 struct saved_fixups {
-     struct fixup fixup_chain[GAS_CGEN_MAX_FIXUPS];
-     int num_fixups_in_chain;
+  struct fixup fixup_chain[GAS_CGEN_MAX_FIXUPS];
+  int num_fixups_in_chain;
 };
 
 static struct saved_fixups stored_fixups[MAX_SAVED_FIXUP_CHAINS];
 
-void 
+void
 gas_cgen_initialize_saved_fixups_array ()
 {
-   int i = 0;
-   while (i < MAX_SAVED_FIXUP_CHAINS)
-      stored_fixups[i++].num_fixups_in_chain = 0;
+  int i = 0;
+
+  while (i < MAX_SAVED_FIXUP_CHAINS)
+    stored_fixups[i++].num_fixups_in_chain = 0;
 }
 
-void 
-gas_cgen_save_fixups (int i)
+void
+gas_cgen_save_fixups (i)
+     int i;
 {
-      if (i < 0 || i >= MAX_SAVED_FIXUP_CHAINS)
-      {
-          as_fatal("Index into stored_fixups[] out of bounds.");
-          return;
-      }
-      stored_fixups[i].num_fixups_in_chain = num_fixups;
-      memcpy(stored_fixups[i].fixup_chain, fixups,
-                    sizeof (fixups[0])*num_fixups);
-      num_fixups = 0;
+  if (i < 0 || i >= MAX_SAVED_FIXUP_CHAINS)
+    {
+      as_fatal ("index into stored_fixups[] out of bounds");
+      return;
+    }
+
+  stored_fixups[i].num_fixups_in_chain = num_fixups;
+  memcpy (stored_fixups[i].fixup_chain, fixups,
+         sizeof (fixups[0]) * num_fixups);
+  num_fixups = 0;
 }
 
-void 
-gas_cgen_restore_fixups (int i)
+void
+gas_cgen_restore_fixups (i)
+     int i;
 {
-      if (i < 0 || i >= MAX_SAVED_FIXUP_CHAINS)
-      {
-          as_fatal("Index into stored_fixups[] out of bounds.");
-          return;
-      }
-      num_fixups = stored_fixups[i].num_fixups_in_chain;
-      memcpy(fixups,stored_fixups[i].fixup_chain,
-                    (sizeof (stored_fixups[i].fixup_chain[0]))*num_fixups);
-      stored_fixups[i].num_fixups_in_chain = 0;
+  if (i < 0 || i >= MAX_SAVED_FIXUP_CHAINS)
+    {
+      as_fatal ("index into stored_fixups[] out of bounds");
+      return;
+    }
+
+  num_fixups = stored_fixups[i].num_fixups_in_chain;
+  memcpy (fixups,stored_fixups[i].fixup_chain,
+         (sizeof (stored_fixups[i].fixup_chain[0])) * num_fixups);
+  stored_fixups[i].num_fixups_in_chain = 0;
 }
 
-void 
-gas_cgen_swap_fixups (int i)
+void
+gas_cgen_swap_fixups (i)
+     int i;
 {
-     int tmp;
-     struct fixup tmp_fixup;
-
-     if (i < 0 || i >= MAX_SAVED_FIXUP_CHAINS)
-     {
-         as_fatal("Index into stored_fixups[] out of bounds.");
-         return;
-     }
-
-     if (num_fixups == 0)
-     {
-       gas_cgen_restore_fixups (i);
-     }
-     else if (stored_fixups[i].num_fixups_in_chain == 0)
-     {
-       gas_cgen_save_fixups (i);
-     }
-     else
-     {
-       tmp = stored_fixups[i].num_fixups_in_chain;
-       stored_fixups[i].num_fixups_in_chain = num_fixups;
-       num_fixups = tmp;
-
-       for (tmp = GAS_CGEN_MAX_FIXUPS; tmp--;)
-       {
-         tmp_fixup          = stored_fixups[i].fixup_chain [tmp];
-         stored_fixups[i].fixup_chain[tmp] = fixups [tmp];
-         fixups [tmp]       = tmp_fixup;
-       }
+  if (i < 0 || i >= MAX_SAVED_FIXUP_CHAINS)
+    {
+      as_fatal ("index into stored_fixups[] out of bounds");
+      return;
+    }
+
+  if (num_fixups == 0)
+    gas_cgen_restore_fixups (i);
+
+  else if (stored_fixups[i].num_fixups_in_chain == 0)
+    gas_cgen_save_fixups (i);
+
+  else
+    {
+      int tmp;
+      struct fixup tmp_fixup;
+
+      tmp = stored_fixups[i].num_fixups_in_chain;
+      stored_fixups[i].num_fixups_in_chain = num_fixups;
+      num_fixups = tmp;
+
+      for (tmp = GAS_CGEN_MAX_FIXUPS; tmp--;)
+       {
+         tmp_fixup = stored_fixups[i].fixup_chain [tmp];
+         stored_fixups[i].fixup_chain[tmp] = fixups [tmp];
+         fixups [tmp] = tmp_fixup;
+       }
     }
 }
 
@@ -217,7 +223,7 @@ gas_cgen_swap_fixups (int i)
    At this point we do not use a bfd_reloc_code_real_type for
    operands residing in the insn, but instead just use the
    operand index.  This lets us easily handle fixups for any
-   operand type.  We pick a BFD reloc type in md_apply_fix.  */
+   operand type.  We pick a BFD reloc type in md_apply_fix3.  */
 
 fixS *
 gas_cgen_record_fixup (frag, where, insn, length, operand, opinfo, symbol, offset)
@@ -257,7 +263,7 @@ gas_cgen_record_fixup (frag, where, insn, length, operand, opinfo, symbol, offse
    At this point we do not use a bfd_reloc_code_real_type for
    operands residing in the insn, but instead just use the
    operand index.  This lets us easily handle fixups for any
-   operand type.  We pick a BFD reloc type in md_apply_fix.  */
+   operand type.  We pick a BFD reloc type in md_apply_fix3.  */
 
 fixS *
 gas_cgen_record_fixup_exp (frag, where, insn, length, operand, opinfo, exp)
@@ -318,7 +324,7 @@ gas_cgen_parse_operand (cd, want, strP, opindex, opinfo, resultP, valueP)
   static char *hold;
   static enum cgen_parse_operand_result *resultP_1;
 #endif
-  const char *errmsg = NULL;
+  const char *errmsg;
   expressionS exp;
 
   if (want == CGEN_PARSE_OPERAND_INIT)
@@ -338,12 +344,13 @@ gas_cgen_parse_operand (cd, want, strP, opindex, opinfo, resultP, valueP)
       expr_jmp_buf_p = 0;
       input_line_pointer = (char *) hold;
       *resultP_1 = CGEN_PARSE_OPERAND_RESULT_ERROR;
-      return "illegal operand";
+      return _("illegal operand");
     }
 
   expr_jmp_buf_p = 1;
   expression (&exp);
   expr_jmp_buf_p = 0;
+  errmsg = NULL;
 
   *strP = input_line_pointer;
   input_line_pointer = hold;
@@ -562,14 +569,14 @@ gas_cgen_finish_insn (insn, buf, length, relax_p, result)
    handles the rest.  bfd_install_relocation (or some other bfd function)
    should handle them all.  */
 
-int
-gas_cgen_md_apply_fix3 (fixP, valueP, seg)
+void
+gas_cgen_md_apply_fix3 (fixP, valP, seg)
      fixS *   fixP;
-     valueT * valueP;
+     valueT * valP;
      segT     seg ATTRIBUTE_UNUSED;
 {
   char *where = fixP->fx_frag->fr_literal + fixP->fx_where;
-  valueT value;
+  valueT value = * valP;
   /* Canonical name, since used a lot.  */
   CGEN_CPU_DESC cd = gas_cgen_cpu_desc;
 
@@ -585,15 +592,15 @@ gas_cgen_md_apply_fix3 (fixP, valueP, seg)
      result of md_pcrel_from.  This is confusing.  */
 
   if (fixP->fx_addsy == (symbolS *) NULL)
-    {
-      value = *valueP;
-      fixP->fx_done = 1;
-    }
+    fixP->fx_done = 1;
+
   else if (fixP->fx_pcrel)
-    value = *valueP;
+    ;
+
   else
     {
       value = fixP->fx_offset;
+
       if (fixP->fx_subsy != (symbolS *) NULL)
        {
          if (S_GET_SEGMENT (fixP->fx_subsy) == absolute_section)
@@ -647,7 +654,7 @@ gas_cgen_md_apply_fix3 (fixP, valueP, seg)
        }
 
       if (fixP->fx_done)
-       return 1;
+       return;
 
       /* The operand isn't fully resolved.  Determine a BFD reloc value
         based on the operand information and leave it to
@@ -655,16 +662,15 @@ gas_cgen_md_apply_fix3 (fixP, valueP, seg)
         partial_inplace == false.  */
 
       reloc_type = md_cgen_lookup_reloc (insn, operand, fixP);
+
       if (reloc_type != BFD_RELOC_NONE)
-       {
-         fixP->fx_r_type = reloc_type;
-       }
+       fixP->fx_r_type = reloc_type;
       else
        {
          as_bad_where (fixP->fx_file, fixP->fx_line,
                        _("unresolved expression that must be resolved"));
          fixP->fx_done = 1;
-         return 1;
+         return;
        }
     }
   else if (fixP->fx_done)
@@ -692,17 +698,13 @@ gas_cgen_md_apply_fix3 (fixP, valueP, seg)
          break;
        }
     }
-  else
-    {
-      /* bfd_install_relocation will be called to finish things up.  */
-    }
+  /* else
+     bfd_install_relocation will be called to finish things up.  */
 
   /* Tuck `value' away for use by tc_gen_reloc.
      See the comment describing fx_addnumber in write.h.
      This field is misnamed (or misused :-).  */
   fixP->fx_addnumber = value;
-
-  return 1;
 }
 
 /* Translate internal representation of relocation info to BFD target format.
This page took 0.025776 seconds and 4 git commands to generate.