]> git.ipfire.org Git - thirdparty/binutils-gdb.git/blobdiff - gas/config/tc-ns32k.c
Update year range in copyright notice of binutils files
[thirdparty/binutils-gdb.git] / gas / config / tc-ns32k.c
index a0db7690ace41741e97ed1cb25d0c4e07999f054..82e27861f9bf3adbaed86a2f1629461db0cfaccb 100644 (file)
@@ -1,5 +1,5 @@
 /* ns32k.c  -- Assemble on the National Semiconductor 32k series
-   Copyright (C) 1987-2016 Free Software Foundation, Inc.
+   Copyright (C) 1987-2021 Free Software Foundation, Inc.
 
    This file is part of GAS, the GNU Assembler.
 
@@ -84,7 +84,7 @@ struct addr_mode
 typedef struct addr_mode addr_modeS;
 
 char *freeptr, *freeptr_static;        /* Points at some number of free bytes.  */
-struct hash_control *inst_hash_handle;
+htab_t inst_hash_handle;
 
 struct ns32k_opcode *desc;     /* Pointer at description of instruction.  */
 addr_modeS addr_modeP;
@@ -148,8 +148,8 @@ expressionS exprP;
    10  implied1
    11  implied2
 
-   For every entry there is a datalength in bytes. This is stored in size[n].
-        0,     the objectlength is not explicitly given by the instruction
+   For every entry there is a data length in bytes. This is stored in size[n].
+        0,     the object length is not explicitly given by the instruction
                and the operand is undefined. This is a case for relaxation.
                Reserve 4 bytes for the final object.
 
@@ -173,9 +173,9 @@ expressionS exprP;
 
    The low-order-byte corresponds to low physical memory.
    Obviously a FRAGment must be created for each valid disp in PART whose
-   datalength is undefined (to bad) .
+   data length is undefined (to bad) .
    The case where just the expression is undefined is less severe and is
-   handled by fix. Here the number of bytes in the objectfile is known.
+   handled by fix. Here the number of bytes in the object file is known.
    With this representation we simplify the assembly and separates the
    machine dependent/independent parts in a more clean way (said OE).  */
 \f
@@ -333,7 +333,7 @@ const pseudo_typeS md_pseudo_table[] =
    displacement base-adjust as there are other routines that must
    consider this. Also, as we have two various offset-adjusts in the
    ns32k (acb versus br/brs/jsr/bcond), two set of limits would have
-   had to be used.  Now we dont have to think about that.  */
+   had to be used.  Now we don't have to think about that.  */
 
 const relax_typeS md_relax_table[] =
 {
@@ -362,7 +362,7 @@ char disp_test[] =
 char disp_size[] =
 {4, 1, 2, 0, 4};
 \f
-/* Parse a general operand into an addressingmode struct
+/* Parse a general operand into an addressing mode struct
 
    In:  pointer at operand in ascii form
         pointer at addr_mode struct for result
@@ -432,6 +432,7 @@ addr_mode (char *operand,
                  addrmodeP->disp[0] = str + 2;
                  return -1;
                }
+             /* Fall through.  */
            default:
              as_bad (_("Invalid syntax in PC-relative addressing mode"));
              return 0;
@@ -481,7 +482,7 @@ addr_mode (char *operand,
        {
        case 'f':
          addrmodeP->float_flag = 1;
-         /* Drop through.  */
+         /* Fall through.  */
        case 'r':
          if (str[1] >= '0' && str[1] < '8')
            {
@@ -492,7 +493,7 @@ addr_mode (char *operand,
        default:
          break;
        }
-      /* Drop through.  */
+      /* Fall through.  */
 
     case 3:
       if (!strncmp (str, "tos", 3))
@@ -564,7 +565,7 @@ addr_mode (char *operand,
                  str[strl - 4] = 0;
                  return -1;            /* reg rel */
                }
-             /* Drop through.  */
+             /* Fall through.  */
 
            default:
              if (!strncmp (&str[strl - 4], "(fp", 3))
@@ -621,7 +622,7 @@ addr_mode (char *operand,
 
          addrmodeP->am_size += 1;      /* scaled index byte.  */
          j = str[strl - 4] - '0';      /* store temporary.  */
-         str[strl - 6] = '\000';       /* nullterminate for recursive call.  */
+         str[strl - 6] = '\000';       /* null terminate for recursive call.  */
          i = addr_mode (str, addrmodeP, 1);
 
          if (!i || addrmodeP->mode == 20)
@@ -703,7 +704,7 @@ get_addr_mode (char *ptr, addr_modeS *addrmodeP)
   if ((tmp = addrmodeP->scaled_reg))
     {                          /* Build indexbyte.  */
       tmp--;                   /* Remember regnumber comes incremented for
-                                  flagpurpose.  */
+                                  flag purpose.  */
       tmp |= addrmodeP->scaled_mode << 3;
       addrmodeP->index_byte = (char) tmp;
       addrmodeP->am_size += 1;
@@ -794,7 +795,7 @@ get_addr_mode (char *ptr, addr_modeS *addrmodeP)
   return addrmodeP->mode;
 }
 
-/* Read an optionlist.  */
+/* Read an option list.  */
 
 static void
 optlist (char *str,                    /* The string to extract options from.  */
@@ -923,8 +924,9 @@ encode_operand (int argc,
       switch ((d = operandsP[(loop << 1) + 1]))
        {
        case 'f':               /* Operand of sfsr turns out to be a nasty
-                                  specialcase.  */
+                                  special-case.  */
          opcode_bit_ptr -= 5;
+         /* Fall through.  */
        case 'Z':               /* Float not immediate.  */
        case 'F':               /* 32 bit float general form.  */
        case 'L':               /* 64 bit float.  */
@@ -983,10 +985,10 @@ encode_operand (int argc,
          argv[i] = freeptr;
          pcrel -= 1;           /* Make pcrel 0 in spite of what case 'p':
                                   wants.  */
-         /* fall thru */
+         /* fallthru */
        case 'p':               /* Displacement - pc relative addressing.  */
          pcrel += 1;
-         /* fall thru */
+         /* fallthru */
        case 'd':               /* Displacement.  */
          iif.instr_size += suffixP[i] ? suffixP[i] : 4;
          IIF (12, 2, suffixP[i], (unsigned long) argv[i], 0,
@@ -1009,12 +1011,12 @@ encode_operand (int argc,
          opcode_bit_ptr -= 3;
          iif.iifP[1].object |= tmp << opcode_bit_ptr;
          break;
-       case 'O':               /* Setcfg instruction optionslist.  */
+       case 'O':               /* Setcfg instruction options list.  */
          optlist (argv[i], opt3, &tmp);
          opcode_bit_ptr -= 4;
          iif.iifP[1].object |= tmp << 15;
          break;
-       case 'C':               /* Cinv instruction optionslist.  */
+       case 'C':               /* Cinv instruction options list.  */
          optlist (argv[i], opt4, &tmp);
          opcode_bit_ptr -= 4;
          iif.iifP[1].object |= tmp << 15; /* Insert the regtype in opcode.  */
@@ -1101,7 +1103,8 @@ parse (const char *line, int recursive_level)
       c = *lineptr;
       *(char *) lineptr = '\0';
 
-      if (!(desc = (struct ns32k_opcode *) hash_find (inst_hash_handle, line)))
+      desc = (struct ns32k_opcode *) str_hash_find (inst_hash_handle, line);
+      if (!desc)
        as_fatal (_("No such opcode"));
 
       *(char *) lineptr = c;
@@ -1329,7 +1332,7 @@ fix_new_ns32k (fragS *frag,               /* Which frag? */
   fix_bsr (fixP) = bsr;
   fix_bit_fixP (fixP) = bit_fixP;
   /* We have a MD overflow check for displacements.  */
-  fixP->fx_no_overflow = (im_disp != 0);
+  fixP->fx_no_overflow = im_disp != 0 || bit_fixP != NULL;
 }
 
 static void
@@ -1357,7 +1360,7 @@ fix_new_ns32k_exp (fragS *frag,           /* Which frag? */
   fix_bsr (fixP) = bsr;
   fix_bit_fixP (fixP) = bit_fixP;
   /* We have a MD overflow check for displacements.  */
-  fixP->fx_no_overflow = (im_disp != 0);
+  fixP->fx_no_overflow = im_disp != 0 || bit_fixP != NULL;
 }
 
 /* Convert number to chars in correct order.  */
@@ -1368,7 +1371,7 @@ md_number_to_chars (char *buf, valueT value, int nbytes)
   number_to_chars_littleendian (buf, value, nbytes);
 }
 
-/* This is a variant of md_numbers_to_chars. The reason for its'
+/* This is a variant of md_numbers_to_chars. The reason for its
    existence is the fact that ns32k uses Huffman coded
    displacements. This implies that the bit order is reversed in
    displacements and that they are prefixed with a size-tag.
@@ -1433,7 +1436,7 @@ md_number_to_disp (char *buf, long val, int n)
       break;
 
     default:
-      as_fatal (_("Internal logic error.  line %d, file \"%s\""),
+      as_fatal (_("Internal logic error.  Line %d, file: \"%s\""),
                __LINE__, __FILE__);
     }
 }
@@ -1816,7 +1819,7 @@ convert_iif (void)
                  {
                    /* Frag it.  */
                    if (exprP.X_op_symbol)
-                     /* We cant relax this case.  */
+                     /* We can't relax this case.  */
                      as_fatal (_("Can't relax difference"));
                    else
                      {
@@ -1893,18 +1896,14 @@ md_begin (void)
 {
   /* Build a hashtable of the instructions.  */
   const struct ns32k_opcode *ptr;
-  const char *status;
   const struct ns32k_opcode *endop;
 
-  inst_hash_handle = hash_new ();
+  inst_hash_handle = str_htab_create ();
 
   endop = ns32k_opcodes + sizeof (ns32k_opcodes) / sizeof (ns32k_opcodes[0]);
   for (ptr = ns32k_opcodes; ptr < endop; ptr++)
-    {
-      if ((status = hash_insert (inst_hash_handle, ptr->name, (char *) ptr)))
-       /* Fatal.  */
-       as_fatal (_("Can't hash %s: %s"), ptr->name, status);
-    }
+    if (str_hash_insert (inst_hash_handle, ptr->name, ptr, 0) != NULL)
+      as_fatal (_("duplicate %s"), ptr->name);
 
   /* Some private space please!  */
   freeptr_static = XNEWVEC (char, PRIVATE_SIZE);