Locale changes from Bruno Haible <haible@clisp.cons.org>.
[deliverable/binutils-gdb.git] / gas / config / tc-m32r.c
index 06146d43388085e9d1c75c3132c2543094f542b4..6ee3a2e8dbf658e83b3b558ad87b6e41ef02e3cc 100644 (file)
@@ -1,5 +1,6 @@
 /* tc-m32r.c -- Assembler for the Mitsubishi M32R.
-   Copyright (C) 1996, 1997, 1998 Free Software Foundation.
+   Copyright 1996, 1997, 1998, 1999, 2000, 2001
+   Free Software Foundation, Inc.
 
    This file is part of GAS, the GNU Assembler.
 
    Boston, MA 02111-1307, USA.  */
 
 #include <stdio.h>
-#include <ctype.h>
 #include "as.h"
-#include "subsegs.h"     
+#include "safe-ctype.h"
+#include "subsegs.h"
 #include "symcat.h"
-#include "cgen-opc.h"
+#include "opcodes/m32r-desc.h"
+#include "opcodes/m32r-opc.h"
 #include "cgen.h"
 
 /* Linked list of symbols that are debugging symbols to be defined as the
 typedef struct sym_link
 {
   struct sym_link *next;
-  symbolS        *symbol;
+  symbolS *symbol;
 } sym_linkS;
 
-static sym_linkS *debug_sym_link = (sym_linkS *)0;
-  
+static sym_linkS *debug_sym_link = (sym_linkS *) 0;
+
 /* Structure to hold all of the different components describing
    an individual instruction.  */
 typedef struct
 {
-  const CGEN_INSN *    insn;
-  const CGEN_INSN *    orig_insn;
-  CGEN_FIELDS          fields;
+  const CGEN_INSN *insn;
+  const CGEN_INSN *orig_insn;
+  CGEN_FIELDS fields;
 #if CGEN_INT_INSN_P
-  CGEN_INSN_INT         buffer [1];
+  CGEN_INSN_INT buffer[1];
 #define INSN_VALUE(buf) (*(buf))
 #else
-  unsigned char         buffer [CGEN_MAX_INSN_SIZE];
+  unsigned char buffer[CGEN_MAX_INSN_SIZE];
 #define INSN_VALUE(buf) (buf)
 #endif
-  char *               addr;
-  fragS *              frag;
-  int                   num_fixups;
-  fixS *                fixups [GAS_CGEN_MAX_FIXUPS];
-  int                   indices [MAX_OPERAND_INSTANCES];
-  sym_linkS            *debug_sym_link;
+  char *addr;
+  fragS *frag;
+  int num_fixups;
+  fixS *fixups[GAS_CGEN_MAX_FIXUPS];
+  int indices[MAX_OPERAND_INSTANCES];
+  sym_linkS *debug_sym_link;
 }
 m32r_insn;
 
 /* prev_insn.insn is non-null if last insn was a 16 bit insn on a 32 bit
    boundary (i.e. was the first of two 16 bit insns).  */
-static m32r_insn       prev_insn;
+static m32r_insn prev_insn;
 
 /* Non-zero if we've seen a relaxable insn since the last 32 bit
    alignment request.  */
@@ -73,10 +75,11 @@ static int seen_relaxable_p = 0;
    This flag does not apply to them.  */
 static int m32r_relax;
 
-#if 0 /* not supported yet */
+#if 0
+/* Not supported yet.  */
 /* If non-NULL, pointer to cpu description file to read.
    This allows runtime additions to the assembler.  */
-static const char * m32r_cpu_desc;
+static const char *m32r_cpu_desc;
 #endif
 
 /* Non-zero if warn when a high/shigh reloc has no matching low reloc.
@@ -90,7 +93,6 @@ static const char * m32r_cpu_desc;
    shouldn't assume or require it to).  */
 static int warn_unmatched_high = 0;
 
-/* start-sanitize-m32rx */
 /* Non-zero if -m32rx has been specified, in which case support for the
    extended M32RX instruction set should be enabled.  */
 static int enable_m32rx = 0;
@@ -105,9 +107,8 @@ static int warn_explicit_parallel_conflicts = 1;
 
 /* Non-zero if insns can be made parallel.  */
 static int optimize;
-/* end-sanitize-m32rx */
 
-/* stuff for .scomm symbols.  */
+/* Stuff for .scomm symbols.  */
 static segT     sbss_section;
 static asection scom_section;
 static asymbol  scom_symbol;
@@ -127,22 +128,24 @@ const char FLT_CHARS[]            = "dD";
 
    In order to implement this, we keep track of each unmatched HI
    relocation.  We then sort them so that they immediately precede the
-   corresponding LO relocation. */
+   corresponding LO relocation.  */
 
 struct m32r_hi_fixup
 {
-  struct m32r_hi_fixup * next;  /* Next HI fixup.  */
-  fixS *                 fixp;  /* This fixup.  */
-  segT                   seg;   /* The section this fixup is in.  */
+  /* Next HI fixup.  */
+  struct m32r_hi_fixup *next;
 
+  /* This fixup.  */
+  fixS *fixp;
+
+  /* The section this fixup is in.  */
+  segT seg;
 };
 
 /* The list of unmatched HI relocs.  */
 
-static struct m32r_hi_fixup * m32r_hi_fixup_list;
-
+static struct m32r_hi_fixup *m32r_hi_fixup_list;
 \f
-/* start-sanitize-m32rx */
 static void
 allow_m32rx (on)
      int on;
@@ -153,69 +156,68 @@ allow_m32rx (on)
     bfd_set_arch_mach (stdoutput, TARGET_ARCH,
                       enable_m32rx ? bfd_mach_m32rx : bfd_mach_m32r);
 }
-/* end-sanitize-m32rx */
 \f
-#define M32R_SHORTOPTS ""
-/* start-sanitize-m32rx */
-#undef M32R_SHORTOPTS
 #define M32R_SHORTOPTS "O"
-/* end-sanitize-m32rx */
-const char * md_shortopts = M32R_SHORTOPTS;
+
+const char *md_shortopts = M32R_SHORTOPTS;
 
 struct option md_longopts[] =
 {
-/* start-sanitize-m32rx */
-#define OPTION_M32RX   (OPTION_MD_BASE)
+#define OPTION_M32R            (OPTION_MD_BASE)
+#define OPTION_M32RX           (OPTION_M32R + 1)
+#define OPTION_WARN_PARALLEL   (OPTION_M32RX + 1)
+#define OPTION_NO_WARN_PARALLEL        (OPTION_WARN_PARALLEL + 1)
+#define OPTION_SPECIAL         (OPTION_NO_WARN_PARALLEL + 1)
+#define OPTION_WARN_UNMATCHED  (OPTION_SPECIAL + 1)
+#define OPTION_NO_WARN_UNMATCHED (OPTION_WARN_UNMATCHED + 1)
+  {"m32r",  no_argument, NULL, OPTION_M32R},
   {"m32rx", no_argument, NULL, OPTION_M32RX},
-#define OPTION_WARN_PARALLEL   (OPTION_MD_BASE + 1)
   {"warn-explicit-parallel-conflicts", no_argument, NULL, OPTION_WARN_PARALLEL},
   {"Wp", no_argument, NULL, OPTION_WARN_PARALLEL},
-#define OPTION_NO_WARN_PARALLEL        (OPTION_MD_BASE + 2)
   {"no-warn-explicit-parallel-conflicts", no_argument, NULL, OPTION_NO_WARN_PARALLEL},
   {"Wnp", no_argument, NULL, OPTION_NO_WARN_PARALLEL},
-#define OPTION_SPECIAL (OPTION_MD_BASE + 3)
   {"hidden", no_argument, NULL, OPTION_SPECIAL},
-/* end-sanitize-m32rx */
-
   /* Sigh.  I guess all warnings must now have both variants.  */
-#define OPTION_WARN_UNMATCHED (OPTION_MD_BASE + 4)
-  {"warn-unmatched-high", OPTION_WARN_UNMATCHED},
-  {"Wuh", OPTION_WARN_UNMATCHED},
-#define OPTION_NO_WARN_UNMATCHED (OPTION_MD_BASE + 5)
-  {"no-warn-unmatched-high", OPTION_WARN_UNMATCHED},
-  {"Wnuh", OPTION_WARN_UNMATCHED},
-
-#if 0 /* not supported yet */
-#define OPTION_RELAX  (OPTION_MD_BASE + 6)
+  {"warn-unmatched-high", no_argument, NULL, OPTION_WARN_UNMATCHED},
+  {"Wuh", no_argument, NULL, OPTION_WARN_UNMATCHED},
+  {"no-warn-unmatched-high", no_argument, NULL, OPTION_NO_WARN_UNMATCHED},
+  {"Wnuh", no_argument, NULL, OPTION_NO_WARN_UNMATCHED},
+
+#if 0
+  /* Not supported yet.  */
+#define OPTION_RELAX           (OPTION_NO_WARN_UNMATCHED + 1)
+#define OPTION_CPU_DESC                (OPTION_RELAX + 1)
   {"relax", no_argument, NULL, OPTION_RELAX},
-#define OPTION_CPU_DESC (OPTION_MD_BASE + 7)
   {"cpu-desc", required_argument, NULL, OPTION_CPU_DESC},
 #endif
-
   {NULL, no_argument, NULL, 0}
 };
+
 size_t md_longopts_size = sizeof (md_longopts);
 
 int
 md_parse_option (c, arg)
-     int    c;
-     char * arg;
+     int c;
+     char *arg;
 {
   switch (c)
     {
-/* start-sanitize-m32rx */
     case 'O':
       optimize = 1;
       break;
 
+    case OPTION_M32R:
+      allow_m32rx (0);
+      break;
+
     case OPTION_M32RX:
       allow_m32rx (1);
       break;
-      
+
     case OPTION_WARN_PARALLEL:
       warn_explicit_parallel_conflicts = 1;
       break;
-      
+
     case OPTION_NO_WARN_PARALLEL:
       warn_explicit_parallel_conflicts = 0;
       break;
@@ -225,14 +227,11 @@ md_parse_option (c, arg)
        enable_special = 1;
       else
        {
-         extern char * myname;
-
          /* Pretend that we do not recognise this option.  */
-         fprintf (stderr, _("%s: unrecognised option: -hidden\n"), myname);
+         as_bad (_("Unrecognised option: -hidden"));
          return 0;
        }
       break;
-/* end-sanitize-m32rx */
 
     case OPTION_WARN_UNMATCHED:
       warn_unmatched_high = 1;
@@ -241,8 +240,9 @@ md_parse_option (c, arg)
     case OPTION_NO_WARN_UNMATCHED:
       warn_unmatched_high = 0;
       break;
-      
-#if 0 /* not supported yet */
+
+#if 0
+    /* Not supported yet.  */
     case OPTION_RELAX:
       m32r_relax = 1;
       break;
@@ -254,16 +254,18 @@ md_parse_option (c, arg)
     default:
       return 0;
     }
+
   return 1;
 }
 
 void
 md_show_usage (stream)
-  FILE * stream;
+     FILE *stream;
 {
   fprintf (stream, _(" M32R specific command line options:\n"));
 
-/* start-sanitize-m32rx */
+  fprintf (stream, _("\
+  -m32r                   disable support for the m32rx instruction set\n"));
   fprintf (stream, _("\
   -m32rx                  support the extended m32rx instruction set\n"));
   fprintf (stream, _("\
@@ -281,7 +283,6 @@ md_show_usage (stream)
   -Wp                     synonym for -warn-explicit-parallel-conflicts\n"));
   fprintf (stream, _("\
   -Wnp                    synonym for -no-warn-explicit-parallel-conflicts\n"));
-/* end-sanitize-m32rx */
 
   fprintf (stream, _("\
   -warn-unmatched-high    warn when an (s)high reloc has no matching low reloc\n"));
@@ -298,7 +299,7 @@ md_show_usage (stream)
   fprintf (stream, _("\
   -cpu-desc              provide runtime cpu description file\n"));
 #endif
-} 
+}
 
 static void fill_insn PARAMS ((int));
 static void m32r_scomm PARAMS ((int));
@@ -316,66 +317,55 @@ const pseudo_typeS md_pseudo_table[] =
   { "fillinsn", fill_insn,     0 },
   { "scomm",   m32r_scomm,     0 },
   { "debugsym",        debug_sym,      0 },
-/* start-sanitize-m32rx */
-  /* Not documented as so far there is no need for them.... */  
+  /* Not documented as so far there is no need for them....  */
   { "m32r",    allow_m32rx,    0 },
   { "m32rx",   allow_m32rx,    1 },
-/* end-sanitize-m32rx */
   { NULL, NULL, 0 }
 };
 
 /* FIXME: Should be machine generated.  */
 #define NOP_INSN 0x7000
-#define PAR_NOP_INSN 0xf000 /* can only be used in 2nd slot */
+#define PAR_NOP_INSN 0xf000 /* Can only be used in 2nd slot.  */
 
-/* When we align the .text section, insert the correct NOP pattern.
-   N is the power of 2 alignment.  LEN is the length of pattern FILL.
-   MAX is the maximum number of characters to skip when doing the alignment,
-   or 0 if there is no maximum.  */
+/* This is called from HANDLE_ALIGN in write.c.  Fill in the contents
+   of an rs_align_code fragment.  */
 
-int
-m32r_do_align (n, fill, len, max)
-     int          n;
-     const char * fill;
-     int          len;
-     int          max;
+void
+m32r_handle_align (fragp)
+     fragS *fragp;
 {
-  /* Only do this if the fill pattern wasn't specified.  */
-  if (fill == NULL
-      && (now_seg->flags & SEC_CODE) != 0
-      /* Only do this special handling if aligning to at least a
-        4 byte boundary.  */
-      && n > 1
-     /* Only do this special handling if we're allowed to emit at
-        least two bytes.  */
-      && (max == 0 || max > 1))
+  static const unsigned char nop_pattern[] = { 0xf0, 0x00 };
+  static const unsigned char multi_nop_pattern[] = { 0x70, 0x00, 0xf0, 0x00 };
+
+  int bytes, fix;
+  char *p;
+
+  if (fragp->fr_type != rs_align_code)
+    return;
+
+  bytes = fragp->fr_next->fr_address - fragp->fr_address - fragp->fr_fix;
+  p = fragp->fr_literal + fragp->fr_fix;
+  fix = 0;
+
+  if (bytes & 1)
     {
-      static const unsigned char nop_pattern[] = { 0xf0, 0x00 };
+      fix = 1;
+      *p++ = 0;
+      bytes--;
+    }
 
-#if 0
-      /* First align to a 2 byte boundary, in case there is an odd .byte.  */
-      /* FIXME: How much memory will cause gas to use when assembling a big
-        program?  Perhaps we can avoid the frag_align call?  */
-      frag_align (1, 0, 0);
-#endif
-      /* Next align to a 4 byte boundary (we know n >= 2) using a parallel
-        nop.  */
-      frag_align_pattern (2, nop_pattern, sizeof nop_pattern, 0);
-      /* If doing larger alignments use a repeating sequence of appropriate
-        nops.  */
-      if (n > 2)
-       {
-         static const unsigned char multi_nop_pattern[] =
-         { 0x70, 0x00, 0xf0, 0x00 };
-         frag_align_pattern (n, multi_nop_pattern, sizeof multi_nop_pattern,
-                             max ? max - 2 : 0);
-       }
-      
-      prev_insn.insn = NULL;
-      return 1;
+  if (bytes & 2)
+    {
+      memcpy (p, nop_pattern, 2);
+      p += 2;
+      bytes -= 2;
+      fix += 2;
     }
 
-  return 0;
+  memcpy (p, multi_nop_pattern, 4);
+
+  fragp->fr_fix += fix;
+  fragp->fr_var = 4;
 }
 
 /* If the last instruction was the first of 2 16 bit insns,
@@ -391,7 +381,7 @@ static void
 fill_insn (ignore)
      int ignore;
 {
-  (void) m32r_do_align (2, NULL, 0, 0);
+  frag_align_code (2, 0);
   prev_insn.insn = NULL;
   seen_relaxable_p = 0;
 }
@@ -415,7 +405,7 @@ debug_sym (ignore)
   name = input_line_pointer;
   delim = get_symbol_end ();
   end_name = input_line_pointer;
+
   if ((symbolP = symbol_find (name)) == NULL
       && (symbolP = md_undefined_symbol (name)) == NULL)
     {
@@ -433,7 +423,7 @@ debug_sym (ignore)
       link->symbol = symbolP;
       link->next = debug_sym_link;
       debug_sym_link = link;
-      symbolP->local = 1;
+      symbol_get_obj (symbolP)->local = 1;
     }
 
   *end_name = delim;
@@ -454,14 +444,14 @@ expand_debug_syms (syms, align)
   if (!syms)
     return;
 
-  (void) m32r_do_align (align, NULL, 0, 0);
-  for (; syms != (sym_linkS *)0; syms = next_syms)
+  (void) frag_align_code (align, 0);
+  for (; syms != (sym_linkS *) 0; syms = next_syms)
     {
       symbolS *symbolP = syms->symbol;
       next_syms = syms->next;
       input_line_pointer = ".\n";
       pseudo_set (symbolP);
-      free ((char *)syms);
+      free ((char *) syms);
     }
 
   input_line_pointer = save_input_line;
@@ -477,11 +467,11 @@ m32r_fill_insn (done)
 {
   if (prev_seg != NULL)
     {
-      segT    seg    = now_seg;
+      segT seg = now_seg;
       subsegT subseg = now_subseg;
 
       subseg_set (prev_seg, prev_subseg);
-      
+
       fill_insn (0);
 
       subseg_set (seg, subseg);
@@ -490,7 +480,7 @@ m32r_fill_insn (done)
   if (done && debug_sym_link)
     {
       expand_debug_syms (debug_sym_link, 1);
-      debug_sym_link = (sym_linkS *)0;
+      debug_sym_link = (sym_linkS *) 0;
     }
 
   return 1;
@@ -500,28 +490,34 @@ void
 md_begin ()
 {
   flagword applicable;
-  segT     seg;
-  subsegT  subseg;
+  segT seg;
+  subsegT subseg;
 
   /* Initialize the `cgen' interface.  */
-  
+
   /* Set the machine number and endian.  */
-  gas_cgen_opcode_desc = m32r_cgen_opcode_open (0 /* mach number */,
-                                               target_big_endian ?
-                                               CGEN_ENDIAN_BIG
-                                               : CGEN_ENDIAN_LITTLE);
-  m32r_cgen_init_asm (gas_cgen_opcode_desc);
+  gas_cgen_cpu_desc = m32r_cgen_cpu_open (CGEN_CPU_OPEN_MACHS, 0,
+                                         CGEN_CPU_OPEN_ENDIAN,
+                                         CGEN_ENDIAN_BIG,
+                                         CGEN_CPU_OPEN_END);
+  m32r_cgen_init_asm (gas_cgen_cpu_desc);
+
+  /* The operand instance table is used during optimization to determine
+     which insns can be executed in parallel.  It is also used to give
+     warnings regarding operand interference in parallel insns.  */
+  m32r_cgen_init_opinst_table (gas_cgen_cpu_desc);
 
   /* This is a callback from cgen to gas to parse operands.  */
-  cgen_set_parse_operand_fn (gas_cgen_opcode_desc, gas_cgen_parse_operand);
+  cgen_set_parse_operand_fn (gas_cgen_cpu_desc, gas_cgen_parse_operand);
 
-#if 0 /* not supported yet */
+#if 0
+  /* Not supported yet.  */
   /* If a runtime cpu description file was provided, parse it.  */
   if (m32r_cpu_desc != NULL)
     {
-      const char * errmsg;
+      const char *errmsg;
 
-      errmsg = cgen_read_cpu_file (gas_cgen_opcode_desc, m32r_cpu_desc);
+      errmsg = cgen_read_cpu_file (gas_cgen_cpu_desc, m32r_cpu_desc);
       if (errmsg != NULL)
        as_bad ("%s: %s", m32r_cpu_desc, errmsg);
     }
@@ -534,12 +530,13 @@ md_begin ()
 
   /* The sbss section is for local .scomm symbols.  */
   sbss_section = subseg_new (".sbss", 0);
-  
+
   /* This is copied from perform_an_assembly_pass.  */
   applicable = bfd_applicable_section_flags (stdoutput);
   bfd_set_section_flags (stdoutput, sbss_section, applicable & SEC_ALLOC);
-  
-#if 0 /* What does this do? [see perform_an_assembly_pass]  */
+
+#if 0
+  /* What does this do? [see perform_an_assembly_pass]  */
   seg_info (bss_section)->bss = 1;
 #endif
 
@@ -549,71 +546,72 @@ md_begin ()
      but with the name .scommon.  */
   scom_section                = bfd_com_section;
   scom_section.name           = ".scommon";
-  scom_section.output_section = & scom_section;
-  scom_section.symbol         = & scom_symbol;
-  scom_section.symbol_ptr_ptr = & scom_section.symbol;
-  scom_symbol                 = * bfd_com_section.symbol;
+  scom_section.output_section = &scom_section;
+  scom_section.symbol         = &scom_symbol;
+  scom_section.symbol_ptr_ptr = &scom_section.symbol;
+  scom_symbol                 = *bfd_com_section.symbol;
   scom_symbol.name            = ".scommon";
-  scom_symbol.section         = & scom_section;
+  scom_symbol.section         = &scom_section;
 
-/* start-sanitize-m32rx */
   allow_m32rx (enable_m32rx);
-/* end-sanitize-m32rx */
+
+  gas_cgen_initialize_saved_fixups_array ();
 }
 
-/* start-sanitize-m32rx */
+#define OPERAND_IS_COND_BIT(operand, indices, index) \
+  ((operand)->hw_type == HW_H_COND                     \
+   || ((operand)->hw_type == HW_H_PSW)                 \
+   || ((operand)->hw_type == HW_H_CR                   \
+       && (indices [index] == 0 || indices [index] == 1)))
 
-#define OPERAND_IS_COND_BIT(operand, indices, index)                   \
-       (CGEN_OPERAND_INSTANCE_HW (operand)->type == HW_H_COND          \
-        || (CGEN_OPERAND_INSTANCE_HW (operand)->type == HW_H_CR        \
-            && (indices [index] == 0 || indices [index] == 1)))
-     
 /* Returns true if an output of instruction 'a' is referenced by an operand
    of instruction 'b'.  If 'check_outputs' is true then b's outputs are
    checked, otherwise its inputs are examined.  */
 
 static int
 first_writes_to_seconds_operands (a, b, check_outputs)
-     m32r_insn * a;
-     m32r_insn * b;
-     const int   check_outputs;
+     m32r_insn *a;
+     m32r_insn *b;
+     const int check_outputs;
 {
-  const CGEN_OPERAND_INSTANCE * a_operands = CGEN_INSN_OPERANDS (a->insn);
-  const CGEN_OPERAND_INSTANCE * b_ops = CGEN_INSN_OPERANDS (b->insn);
-  int                           a_index;
+  const CGEN_OPINST *a_operands = CGEN_INSN_OPERANDS (a->insn);
+  const CGEN_OPINST *b_ops = CGEN_INSN_OPERANDS (b->insn);
+  int a_index;
 
   /* If at least one of the instructions takes no operands, then there is
      nothing to check.  There really are instructions without operands,
      eg 'nop'.  */
   if (a_operands == NULL || b_ops == NULL)
     return 0;
-      
+
   /* Scan the operand list of 'a' looking for an output operand.  */
   for (a_index = 0;
-       CGEN_OPERAND_INSTANCE_TYPE (a_operands) != CGEN_OPERAND_INSTANCE_END;
+       a_operands->type != CGEN_OPINST_END;
        a_index ++, a_operands ++)
     {
-      if (CGEN_OPERAND_INSTANCE_TYPE (a_operands) == CGEN_OPERAND_INSTANCE_OUTPUT)
+      if (a_operands->type == CGEN_OPINST_OUTPUT)
        {
          int b_index;
-         const CGEN_OPERAND_INSTANCE * b_operands = b_ops;
+         const CGEN_OPINST *b_operands = b_ops;
 
          /* Special Case:
             The Condition bit 'C' is a shadow of the CBR register (control
             register 1) and also a shadow of bit 31 of the program status
             word (control register 0).  For now this is handled here, rather
-            than by cgen.... */
-         
+            than by cgen....  */
+
          if (OPERAND_IS_COND_BIT (a_operands, a->indices, a_index))
            {
              /* Scan operand list of 'b' looking for another reference to the
                 condition bit, which goes in the right direction.  */
              for (b_index = 0;
-                  CGEN_OPERAND_INSTANCE_TYPE (b_operands) != CGEN_OPERAND_INSTANCE_END;
-                  b_index ++, b_operands ++)
+                  b_operands->type != CGEN_OPINST_END;
+                  b_index++, b_operands++)
                {
-                 if ((CGEN_OPERAND_INSTANCE_TYPE (b_operands) ==
-                      (check_outputs ? CGEN_OPERAND_INSTANCE_OUTPUT : CGEN_OPERAND_INSTANCE_INPUT))
+                 if ((b_operands->type
+                      == (check_outputs
+                          ? CGEN_OPINST_OUTPUT
+                          : CGEN_OPINST_INPUT))
                      && OPERAND_IS_COND_BIT (b_operands, b->indices, b_index))
                    return 1;
                }
@@ -624,84 +622,89 @@ first_writes_to_seconds_operands (a, b, check_outputs)
                 references the same hardware element, and which goes in the
                 right direction.  */
              for (b_index = 0;
-                  CGEN_OPERAND_INSTANCE_TYPE (b_operands) != CGEN_OPERAND_INSTANCE_END;
-                  b_index ++, b_operands ++)
+                  b_operands->type != CGEN_OPINST_END;
+                  b_index++, b_operands++)
                {
-                 if ((CGEN_OPERAND_INSTANCE_TYPE (b_operands) ==
-                      (check_outputs ? CGEN_OPERAND_INSTANCE_OUTPUT : CGEN_OPERAND_INSTANCE_INPUT))
-                     && (CGEN_OPERAND_INSTANCE_HW (b_operands) == CGEN_OPERAND_INSTANCE_HW (a_operands))
-                     && (a->indices [a_index] == b->indices [b_index]))
+                 if ((b_operands->type
+                      == (check_outputs
+                          ? CGEN_OPINST_OUTPUT
+                          : CGEN_OPINST_INPUT))
+                     && (b_operands->hw_type == a_operands->hw_type)
+                     && (a->indices[a_index] == b->indices[b_index]))
                    return 1;
                }
            }
        }
     }
 
-    return 0;
+  return 0;
 }
 
 /* Returns true if the insn can (potentially) alter the program counter.  */
 
 static int
 writes_to_pc (a)
-     m32r_insn * a;
+     m32r_insn *a;
 {
-#if 0  /* Once PC operands are working.... */
-  const CGEN_OPERAND_INSTANCE * a_operands == CGEN_INSN_OPERANDS (a->insn);
+#if 0
+  /* Once PC operands are working....  */
+  const CGEN_OPINST *a_operands == CGEN_INSN_OPERANDS (gas_cgen_cpu_desc,
+                                                      a->insn);
 
   if (a_operands == NULL)
     return 0;
 
-  while (CGEN_OPERAND_INSTANCE_TYPE (a_operands) != CGEN_OPERAND_INSTANCE_END)
+  while (a_operands->type != CGEN_OPINST_END)
     {
-      if (CGEN_OPERAND_INSTANCE_OPERAND (a_operands) != NULL
-         && CGEN_OPERAND_INDEX (CGEN_OPERAND_INSTANCE_OPERAND (a_operands)) == M32R_OPERAND_PC)
+      if (a_operands->operand != NULL
+         && CGEN_OPERAND_INDEX (gas_cgen_cpu_desc,
+                                a_operands->operand) == M32R_OPERAND_PC)
        return 1;
-      
-      a_operands ++;
+
+      a_operands++;
     }
 #else
-  if (CGEN_INSN_ATTR (a->insn, CGEN_INSN_UNCOND_CTI)
-      || CGEN_INSN_ATTR (a->insn, CGEN_INSN_COND_CTI))
+  if (CGEN_INSN_ATTR_VALUE (a->insn, CGEN_INSN_UNCOND_CTI)
+      || CGEN_INSN_ATTR_VALUE (a->insn, CGEN_INSN_COND_CTI))
     return 1;
 #endif
   return 0;
 }
 
-/* Returns NULL if the two 16 bit insns can be executed in parallel,
-   otherwise it returns a pointer to an error message explaining why not.  */
+/* Return NULL if the two 16 bit insns can be executed in parallel.
+   Otherwise return a pointer to an error message explaining why not.  */
 
 static const char *
 can_make_parallel (a, b)
-     m32r_insn * a;
-     m32r_insn * b;
+     m32r_insn *a;
+     m32r_insn *b;
 {
   PIPE_ATTR a_pipe;
   PIPE_ATTR b_pipe;
-  
+
   /* Make sure the instructions are the right length.  */
-  if (   CGEN_FIELDS_BITSIZE (& a->fields) != 16
-      || CGEN_FIELDS_BITSIZE (& b->fields) != 16)
-    abort();
+  if (CGEN_FIELDS_BITSIZE (&a->fields) != 16
+      || CGEN_FIELDS_BITSIZE (&b->fields) != 16)
+    abort ();
 
   if (first_writes_to_seconds_operands (a, b, true))
     return _("Instructions write to the same destination register.");
-  
-  a_pipe = CGEN_INSN_ATTR (a->insn, CGEN_INSN_PIPE);
-  b_pipe = CGEN_INSN_ATTR (b->insn, CGEN_INSN_PIPE);
+
+  a_pipe = CGEN_INSN_ATTR_VALUE (a->insn, CGEN_INSN_PIPE);
+  b_pipe = CGEN_INSN_ATTR_VALUE (b->insn, CGEN_INSN_PIPE);
 
   /* Make sure that the instructions use the correct execution pipelines.  */
-  if (   a_pipe == PIPE_NONE
+  if (a_pipe == PIPE_NONE
       || b_pipe == PIPE_NONE)
     return _("Instructions do not use parallel execution pipelines.");
 
   /* Leave this test for last, since it is the only test that can
      go away if the instructions are swapped, and we want to make
      sure that any other errors are detected before this happens.  */
-  if (   a_pipe == PIPE_S
+  if (a_pipe == PIPE_S
       || b_pipe == PIPE_O)
     return _("Instructions share the same execution pipeline");
-  
+
   return NULL;
 }
 
@@ -714,7 +717,7 @@ make_parallel (buffer)
 #if CGEN_INT_INSN_P
   *buffer |= 0x8000;
 #else
-  buffer [CGEN_OPCODE_ENDIAN (gas_cgen_opcode_desc) == CGEN_ENDIAN_BIG ? 0 : 1]
+  buffer[CGEN_CPU_ENDIAN (gas_cgen_cpu_desc) == CGEN_ENDIAN_BIG ? 0 : 1]
     |= 0x80;
 #endif
 }
@@ -725,90 +728,112 @@ static void
 target_make_parallel (buffer)
      char *buffer;
 {
-  buffer [CGEN_OPCODE_ENDIAN (gas_cgen_opcode_desc) == CGEN_ENDIAN_BIG ? 0 : 1]
+  buffer[CGEN_CPU_ENDIAN (gas_cgen_cpu_desc) == CGEN_ENDIAN_BIG ? 0 : 1]
     |= 0x80;
 }
 
 /* Assemble two instructions with an explicit parallel operation (||) or
    sequential operation (->).  */
+
 static void
 assemble_two_insns (str, str2, parallel_p)
-     char * str;
-     char * str2;
-     int    parallel_p;
+     char *str;
+     char *str2;
+     int parallel_p;
 {
-  char *    str3;
+  char *str3;
   m32r_insn first;
   m32r_insn second;
-  char *    errmsg;
-  char      save_str2 = *str2;
+  char *errmsg;
+  char save_str2 = *str2;
 
-  * str2 = 0; /* Seperate the two instructions.  */
+  /* Seperate the two instructions.  */
+  *str2 = 0;
 
-  /* If there was a previous 16 bit insn, then fill the following 16 bit slot,
-     so that the parallel instruction will start on a 32 bit boundary.  */
-  if (prev_insn.insn)
-    fill_insn (0);
+  /* Make sure the two insns begin on a 32 bit boundary.
+     This is also done for the serial case (foo -> bar), relaxing doesn't
+     affect insns written like this.
+     Note that we must always do this as we can't assume anything about
+     whether we're currently on a 32 bit boundary or not.  Relaxing may
+     change this.  */
+  fill_insn (0);
 
   first.debug_sym_link = debug_sym_link;
-  debug_sym_link = (sym_linkS *)0;
+  debug_sym_link = (sym_linkS *) 0;
 
   /* Parse the first instruction.  */
   if (! (first.insn = m32r_cgen_assemble_insn
-        (gas_cgen_opcode_desc, str, & first.fields, first.buffer, & errmsg)))
+        (gas_cgen_cpu_desc, str, & first.fields, first.buffer, & errmsg)))
     {
       as_bad (errmsg);
       return;
     }
 
-  if (! enable_special
-      && CGEN_INSN_ATTR (first.insn, CGEN_INSN_SPECIAL))
+  /* Check it.  */
+  if (CGEN_FIELDS_BITSIZE (&first.fields) != 16)
     {
-      /* xgettext:c-format */
+      /* xgettext:c-format  */
+      as_bad (_("not a 16 bit instruction '%s'"), str);
+      return;
+    }
+  else if (! enable_special
+      && CGEN_INSN_ATTR_VALUE (first.insn, CGEN_INSN_SPECIAL))
+    {
+      /* xgettext:c-format  */
       as_bad (_("unknown instruction '%s'"), str);
       return;
     }
   else if (! enable_m32rx
-      /* FIXME: Need standard macro to perform this test.  */
-      && CGEN_INSN_ATTR (first.insn, CGEN_INSN_MACH) == (1 << MACH_M32RX))
+          /* FIXME: Need standard macro to perform this test.  */
+          && (CGEN_INSN_ATTR_VALUE (first.insn, CGEN_INSN_MACH)
+              == (1 << MACH_M32RX)))
     {
-      /* xgettext:c-format */
+      /* xgettext:c-format  */
       as_bad (_("instruction '%s' is for the M32RX only"), str);
       return;
     }
-    
+
   /* Check to see if this is an allowable parallel insn.  */
-  if (parallel_p && CGEN_INSN_ATTR (first.insn, CGEN_INSN_PIPE) == PIPE_NONE)
+  if (parallel_p
+      && CGEN_INSN_ATTR_VALUE (first.insn, CGEN_INSN_PIPE) == PIPE_NONE)
     {
-      /* xgettext:c-format */
+      /* xgettext:c-format  */
       as_bad (_("instruction '%s' cannot be executed in parallel."), str);
       return;
     }
-  
-  *str2 = save_str2; /* Restore the original assembly text, just in case it is needed.  */
-  str3  = str;       /* Save the original string pointer.  */
-  str   = str2 + 2;  /* Advanced past the parsed string.  */
-  str2  = str3;      /* Remember the entire string in case it is needed for error messages.  */
+
+  /* Restore the original assembly text, just in case it is needed.  */
+  *str2 = save_str2;
+
+  /* Save the original string pointer.  */
+  str3 = str;
+
+  /* Advanced past the parsed string.  */
+  str = str2 + 2;
+
+  /* Remember the entire string in case it is needed for error
+     messages.  */
+  str2 = str3;
 
   /* Convert the opcode to lower case.  */
   {
     char *s2 = str;
-    
-    while (isspace (*s2 ++))
+
+    while (ISSPACE (*s2++))
       continue;
 
     --s2;
 
-    while (isalnum (*s2))
+    while (ISALNUM (*s2))
       {
-       if (isupper ((unsigned char) *s2))
-         *s2 = tolower (*s2);
-       s2 ++;
+       *s2 = TOLOWER (*s2);
+       s2++;
       }
   }
-  
-  /* Preserve any fixups that have been generated and reset the list to empty.  */
-  gas_cgen_save_fixups ();
+
+  /* Preserve any fixups that have been generated and reset the list
+     to empty.  */
+  gas_cgen_save_fixups (0);
 
   /* Get the indices of the operands of the instruction.  */
   /* FIXME: CGEN_FIELDS is already recorded, but relying on that fact
@@ -820,10 +845,13 @@ assemble_two_insns (str, str2, parallel_p)
      version (eg relaxability).  When aliases behave differently this
      may have to change.  */
   first.orig_insn = first.insn;
-  first.insn = m32r_cgen_lookup_get_insn_operands
-    (gas_cgen_opcode_desc, NULL, INSN_VALUE (first.buffer), 16,
-     first.indices);
-  
+  {
+    CGEN_FIELDS tmp_fields;
+    first.insn = cgen_lookup_get_insn_operands
+      (gas_cgen_cpu_desc, NULL, INSN_VALUE (first.buffer), NULL, 16,
+       first.indices, &tmp_fields);
+  }
+
   if (first.insn == NULL)
     as_fatal (_("internal error: lookup/get operands failed"));
 
@@ -831,42 +859,49 @@ assemble_two_insns (str, str2, parallel_p)
 
   /* Parse the second instruction.  */
   if (! (second.insn = m32r_cgen_assemble_insn
-        (gas_cgen_opcode_desc, str, & second.fields, second.buffer, & errmsg)))
+        (gas_cgen_cpu_desc, str, & second.fields, second.buffer, & errmsg)))
     {
       as_bad (errmsg);
       return;
     }
 
   /* Check it.  */
-  if (! enable_special
-      && CGEN_INSN_ATTR (second.insn, CGEN_INSN_SPECIAL))
+  if (CGEN_FIELDS_BITSIZE (&second.fields) != 16)
+    {
+      /* xgettext:c-format  */
+      as_bad (_("not a 16 bit instruction '%s'"), str);
+      return;
+    }
+  else if (! enable_special
+      && CGEN_INSN_ATTR_VALUE (second.insn, CGEN_INSN_SPECIAL))
     {
-      /* xgettext:c-format */
+      /* xgettext:c-format  */
       as_bad (_("unknown instruction '%s'"), str);
       return;
     }
   else if (! enable_m32rx
-      && CGEN_INSN_ATTR (second.insn, CGEN_INSN_MACH) == (1 << MACH_M32RX))
+      && CGEN_INSN_ATTR_VALUE (second.insn, CGEN_INSN_MACH) == (1 << MACH_M32RX))
     {
-      /* xgettext:c-format */
+      /* xgettext:c-format  */
       as_bad (_("instruction '%s' is for the M32RX only"), str);
       return;
     }
 
   /* Check to see if this is an allowable parallel insn.  */
-  if (parallel_p && CGEN_INSN_ATTR (second.insn, CGEN_INSN_PIPE) == PIPE_NONE)
+  if (parallel_p
+      && CGEN_INSN_ATTR_VALUE (second.insn, CGEN_INSN_PIPE) == PIPE_NONE)
     {
-      /* xgettext:c-format */
+      /* xgettext:c-format  */
       as_bad (_("instruction '%s' cannot be executed in parallel."), str);
       return;
     }
-  
+
   if (parallel_p && ! enable_m32rx)
     {
       if (CGEN_INSN_NUM (first.insn) != M32R_INSN_NOP
          && CGEN_INSN_NUM (second.insn) != M32R_INSN_NOP)
        {
-         /* xgettext:c-format */
+         /* xgettext:c-format  */
          as_bad (_("'%s': only the NOP instruction can be issued in parallel on the m32r"), str2);
          return;
        }
@@ -874,10 +909,13 @@ assemble_two_insns (str, str2, parallel_p)
 
   /* Get the indices of the operands of the instruction.  */
   second.orig_insn = second.insn;
-  second.insn = m32r_cgen_lookup_get_insn_operands
-    (gas_cgen_opcode_desc, NULL, INSN_VALUE (second.buffer), 16,
-     second.indices);
-  
+  {
+    CGEN_FIELDS tmp_fields;
+    second.insn = cgen_lookup_get_insn_operands
+      (gas_cgen_cpu_desc, NULL, INSN_VALUE (second.buffer), NULL, 16,
+       second.indices, &tmp_fields);
+  }
+
   if (second.insn == NULL)
     as_fatal (_("internal error: lookup/get operands failed"));
 
@@ -888,85 +926,82 @@ assemble_two_insns (str, str2, parallel_p)
      variable warn_explicit_parallel_conflicts is true then we do generate
      a warning message.  Similarly we assume that parallel branch and jump
      instructions are deliberate and should not produce errors.  */
-  
+
   if (parallel_p && warn_explicit_parallel_conflicts)
     {
-      if (first_writes_to_seconds_operands (& first, & second, false))
-       /* xgettext:c-format */
+      if (first_writes_to_seconds_operands (&first, &second, false))
+       /* xgettext:c-format  */
        as_warn (_("%s: output of 1st instruction is the same as an input to 2nd instruction - is this intentional ?"), str2);
-      
-      if (first_writes_to_seconds_operands (& second, & first, false))
-       /* xgettext:c-format */
+
+      if (first_writes_to_seconds_operands (&second, &first, false))
+       /* xgettext:c-format  */
        as_warn (_("%s: output of 2nd instruction is the same as an input to 1st instruction - is this intentional ?"), str2);
     }
-      
-  if (!parallel_p || (errmsg = (char *) can_make_parallel (& first, & second)) == NULL)
+
+  if (!parallel_p
+      || (errmsg = (char *) can_make_parallel (&first, &second)) == NULL)
     {
       /* Get the fixups for the first instruction.  */
-      gas_cgen_swap_fixups ();
+      gas_cgen_swap_fixups (0);
 
       /* Write it out.  */
       expand_debug_syms (first.debug_sym_link, 1);
       gas_cgen_finish_insn (first.orig_insn, first.buffer,
-                           CGEN_FIELDS_BITSIZE (& first.fields), 0, NULL);
-      
+                           CGEN_FIELDS_BITSIZE (&first.fields), 0, NULL);
+
       /* Force the top bit of the second insn to be set.  */
       if (parallel_p)
        make_parallel (second.buffer);
 
       /* Get its fixups.  */
-      gas_cgen_restore_fixups ();
+      gas_cgen_restore_fixups (0);
 
       /* Write it out.  */
       expand_debug_syms (second.debug_sym_link, 1);
       gas_cgen_finish_insn (second.orig_insn, second.buffer,
-                           CGEN_FIELDS_BITSIZE (& second.fields), 0, NULL);
+                           CGEN_FIELDS_BITSIZE (&second.fields), 0, NULL);
     }
   /* Try swapping the instructions to see if they work that way.  */
-  else if (can_make_parallel (& second, & first) == NULL)
+  else if (can_make_parallel (&second, &first) == NULL)
     {
       /* Write out the second instruction first.  */
       expand_debug_syms (second.debug_sym_link, 1);
       gas_cgen_finish_insn (second.orig_insn, second.buffer,
-                           CGEN_FIELDS_BITSIZE (& second.fields), 0, NULL);
-      
+                           CGEN_FIELDS_BITSIZE (&second.fields), 0, NULL);
+
       /* Force the top bit of the first instruction to be set.  */
       make_parallel (first.buffer);
 
       /* Get the fixups for the first instruction.  */
-      gas_cgen_restore_fixups ();
+      gas_cgen_restore_fixups (0);
 
       /* Write out the first instruction.  */
       expand_debug_syms (first.debug_sym_link, 1);
       gas_cgen_finish_insn (first.orig_insn, first.buffer,
-                           CGEN_FIELDS_BITSIZE (& first.fields), 0, NULL);
+                           CGEN_FIELDS_BITSIZE (&first.fields), 0, NULL);
     }
   else
     {
       as_bad ("'%s': %s", str2, errmsg);
       return;
     }
-      
+
   /* Set these so m32r_fill_insn can use them.  */
   prev_seg    = now_seg;
   prev_subseg = now_subseg;
 }
 
-/* end-sanitize-m32rx */
-
-
 void
 md_assemble (str)
-     char * str;
+     char *str;
 {
   m32r_insn insn;
-  char *    errmsg;
-  char *    str2 = NULL;
+  char *errmsg;
+  char *str2 = NULL;
 
   /* Initialize GAS's cgen interface for a new instruction.  */
   gas_cgen_init_parse ();
 
-/* start-sanitize-m32rx */
   /* Look for a parallel instruction seperator.  */
   if ((str2 = strstr (str, "||")) != NULL)
     {
@@ -980,37 +1015,34 @@ md_assemble (str)
       assemble_two_insns (str, str2, 0);
       return;
     }
-/* end-sanitize-m32rx */
-  
+
   insn.debug_sym_link = debug_sym_link;
-  debug_sym_link = (sym_linkS *)0;
+  debug_sym_link = (sym_linkS *) 0;
 
   insn.insn = m32r_cgen_assemble_insn
-    (gas_cgen_opcode_desc, str, & insn.fields, insn.buffer, & errmsg);
-  
+    (gas_cgen_cpu_desc, str, &insn.fields, insn.buffer, & errmsg);
+
   if (!insn.insn)
     {
       as_bad (errmsg);
       return;
     }
 
-/* start-sanitize-m32rx */
   if (! enable_special
-      && CGEN_INSN_ATTR (insn.insn, CGEN_INSN_SPECIAL))
+      && CGEN_INSN_ATTR_VALUE (insn.insn, CGEN_INSN_SPECIAL))
     {
-      /* xgettext:c-format */
+      /* xgettext:c-format  */
       as_bad (_("unknown instruction '%s'"), str);
       return;
     }
   else if (! enable_m32rx
-          && CGEN_INSN_ATTR (insn.insn, CGEN_INSN_MACH) == (1 << MACH_M32RX))
+          && CGEN_INSN_ATTR_VALUE (insn.insn, CGEN_INSN_MACH) == (1 << MACH_M32RX))
     {
-      /* xgettext:c-format */
+      /* xgettext:c-format  */
       as_bad (_("instruction '%s' is for the M32RX only"), str);
       return;
     }
-/* end-sanitize-m32rx */
-  
+
   if (CGEN_INSN_BITSIZE (insn.insn) == 32)
     {
       /* 32 bit insns must live on 32 bit boundaries.  */
@@ -1026,38 +1058,44 @@ md_assemble (str)
 
       /* Doesn't really matter what we pass for RELAX_P here.  */
       gas_cgen_finish_insn (insn.insn, insn.buffer,
-                           CGEN_FIELDS_BITSIZE (& insn.fields), 1, NULL);
+                           CGEN_FIELDS_BITSIZE (&insn.fields), 1, NULL);
     }
   else
     {
       int on_32bit_boundary_p;
-/* start-sanitize-m32rx */
       int swap = false;
-/* end-sanitize-m32rx */
 
       if (CGEN_INSN_BITSIZE (insn.insn) != 16)
-       abort();
+       abort ();
 
       insn.orig_insn = insn.insn;
-/* start-sanitize-m32rx */
+
+      /* If the previous insn was relaxable, then it may be expanded
+        to fill the current 16 bit slot.  Emit a NOP here to occupy
+        this slot, so that we can start at optimizing at a 32 bit
+        boundary.  */
+      if (prev_insn.insn && seen_relaxable_p && optimize)
+       fill_insn (0);
+
       if (enable_m32rx)
        {
          /* Get the indices of the operands of the instruction.
             FIXME: See assemble_parallel for notes on orig_insn.  */
-         insn.insn = m32r_cgen_lookup_get_insn_operands
-           (gas_cgen_opcode_desc, NULL, INSN_VALUE (insn.buffer),
-            16, insn.indices);
-         
+         {
+           CGEN_FIELDS tmp_fields;
+           insn.insn = cgen_lookup_get_insn_operands
+             (gas_cgen_cpu_desc, NULL, INSN_VALUE (insn.buffer), NULL,
+              16, insn.indices, &tmp_fields);
+         }
+
          if (insn.insn == NULL)
            as_fatal (_("internal error: lookup/get operands failed"));
        }
-/* end-sanitize-m32rx */
 
       /* Compute whether we're on a 32 bit boundary or not.
         prev_insn.insn is NULL when we're on a 32 bit boundary.  */
       on_32bit_boundary_p = prev_insn.insn == NULL;
 
-/* start-sanitize-m32rx */
       /* Look to see if this instruction can be combined with the
         previous instruction to make one, parallel, 32 bit instruction.
         If the previous instruction (potentially) changed the flow of
@@ -1068,20 +1106,18 @@ md_assemble (str)
         input to the current instruction then it cannot be combined.
         Otherwise call can_make_parallel() with both orderings of the
         instructions to see if they can be combined.  */
-      if (     ! on_32bit_boundary_p
-         &&   enable_m32rx
-         &&   optimize
-         &&   CGEN_INSN_ATTR (insn.orig_insn, CGEN_INSN_RELAXABLE) == 0
-         && ! writes_to_pc (& prev_insn)
-         && ! first_writes_to_seconds_operands (& prev_insn, &insn, false)
-         )
+      if (! on_32bit_boundary_p
+         && enable_m32rx
+         && optimize
+         && CGEN_INSN_ATTR_VALUE (insn.orig_insn, CGEN_INSN_RELAXABLE) == 0
+         && ! writes_to_pc (&prev_insn)
+         && ! first_writes_to_seconds_operands (&prev_insn, &insn, false))
        {
-         if (can_make_parallel (& prev_insn, & insn) == NULL)
+         if (can_make_parallel (&prev_insn, &insn) == NULL)
            make_parallel (insn.buffer);
-         else if (can_make_parallel (& insn, & prev_insn) == NULL)
+         else if (can_make_parallel (&insn, &prev_insn) == NULL)
            swap = true;
        }
-/* end-sanitize-m32rx */
 
       expand_debug_syms (insn.debug_sym_link, 1);
 
@@ -1093,8 +1129,8 @@ md_assemble (str)
        frag_grow (4);
 
        gas_cgen_finish_insn (insn.orig_insn, insn.buffer,
-                             CGEN_FIELDS_BITSIZE (& insn.fields),
-                             1 /*relax_p*/, &fi);
+                             CGEN_FIELDS_BITSIZE (&insn.fields),
+                             1 /* relax_p  */, &fi);
        insn.addr = fi.addr;
        insn.frag = fi.frag;
        insn.num_fixups = fi.num_fixups;
@@ -1102,16 +1138,15 @@ md_assemble (str)
          insn.fixups[i] = fi.fixups[i];
       }
 
-/* start-sanitize-m32rx */
       if (swap)
        {
-         int i,tmp;
+         int i, tmp;
 
 #define SWAP_BYTES(a,b) tmp = a; a = b; b = tmp
 
          /* Swap the two insns */
-         SWAP_BYTES (prev_insn.addr [0], insn.addr [0]);
-         SWAP_BYTES (prev_insn.addr [1], insn.addr [1]);
+         SWAP_BYTES (prev_insn.addr[0], insn.addr[0]);
+         SWAP_BYTES (prev_insn.addr[1], insn.addr[1]);
 
          target_make_parallel (insn.addr);
 
@@ -1130,7 +1165,6 @@ md_assemble (str)
          for (i = 0; i < insn.num_fixups; ++i)
            insn.fixups[i]->fx_where -= 2;
        }
-/* end-sanitize-m32rx */
 
       /* Keep track of whether we've seen a pair of 16 bit insns.
         prev_insn.insn is NULL when we're on a 32 bit boundary.  */
@@ -1138,17 +1172,17 @@ md_assemble (str)
        prev_insn = insn;
       else
        prev_insn.insn = NULL;
-      
+
       /* If the insn needs the following one to be on a 32 bit boundary
         (e.g. subroutine calls), fill this insn's slot.  */
       if (on_32bit_boundary_p
-         && CGEN_INSN_ATTR (insn.orig_insn, CGEN_INSN_FILL_SLOT) != 0)
+         && CGEN_INSN_ATTR_VALUE (insn.orig_insn, CGEN_INSN_FILL_SLOT) != 0)
        fill_insn (0);
 
       /* If this is a relaxable insn (can be replaced with a larger version)
         mark the fact so that we can emit an alignment directive for a
         following 32 bit insn if we see one.   */
-      if (CGEN_INSN_ATTR (insn.orig_insn, CGEN_INSN_RELAXABLE) != 0)
+      if (CGEN_INSN_ATTR_VALUE (insn.orig_insn, CGEN_INSN_RELAXABLE) != 0)
        seen_relaxable_p = 1;
     }
 
@@ -1160,20 +1194,20 @@ md_assemble (str)
 /* The syntax in the manual says constants begin with '#'.
    We just ignore it.  */
 
-void 
+void
 md_operand (expressionP)
-     expressionS * expressionP;
+     expressionS *expressionP;
 {
-  if (* input_line_pointer == '#')
+  if (*input_line_pointer == '#')
     {
-      input_line_pointer ++;
+      input_line_pointer++;
       expression (expressionP);
     }
 }
 
 valueT
 md_section_align (segment, size)
-     segT   segment;
+     segT segment;
      valueT size;
 {
   int align = bfd_get_section_alignment (stdoutput, segment);
@@ -1182,7 +1216,7 @@ md_section_align (segment, size)
 
 symbolS *
 md_undefined_symbol (name)
-  char * name;
+     char *name;
 {
   return 0;
 }
@@ -1190,51 +1224,52 @@ md_undefined_symbol (name)
 /* .scomm pseudo-op handler.
 
    This is a new pseudo-op to handle putting objects in .scommon.
-   By doing this the linker won't need to do any work and more importantly
-   it removes the implicit -G arg necessary to correctly link the object file.
-*/
+   By doing this the linker won't need to do any work,
+   and more importantly it removes the implicit -G arg necessary to
+   correctly link the object file.  */
 
 static void
 m32r_scomm (ignore)
      int ignore;
 {
-  register char *    name;
-  register char      c;
-  register char *    p;
-  offsetT            size;
-  register symbolS * symbolP;
-  offsetT            align;
-  int                align2;
+  register char *name;
+  register char c;
+  register char *p;
+  offsetT size;
+  register symbolS *symbolP;
+  offsetT align;
+  int align2;
 
   name = input_line_pointer;
   c = get_symbol_end ();
 
-  /* just after name is now '\0' */
+  /* Just after name is now '\0'.  */
   p = input_line_pointer;
-  * p = c;
+  *p = c;
   SKIP_WHITESPACE ();
-  if (* input_line_pointer != ',')
+  if (*input_line_pointer != ',')
     {
       as_bad (_("Expected comma after symbol-name: rest of line ignored."));
       ignore_rest_of_line ();
       return;
     }
 
-  input_line_pointer ++;               /* skip ',' */
+  /* Skip ','.  */
+  input_line_pointer++;
   if ((size = get_absolute_expression ()) < 0)
     {
-      /* xgettext:c-format */
+      /* xgettext:c-format  */
       as_warn (_(".SCOMMon length (%ld.) <0! Ignored."), (long) size);
       ignore_rest_of_line ();
       return;
     }
 
   /* The third argument to .scomm is the alignment.  */
-  if (* input_line_pointer != ',')
+  if (*input_line_pointer != ',')
     align = 8;
   else
     {
-      ++ input_line_pointer;
+      ++input_line_pointer;
       align = get_absolute_expression ();
       if (align <= 0)
        {
@@ -1242,10 +1277,11 @@ m32r_scomm (ignore)
          align = 8;
        }
     }
+
   /* Convert to a power of 2 alignment.  */
   if (align)
     {
-      for (align2 = 0; (align & 1) == 0; align >>= 1, ++ align2)
+      for (align2 = 0; (align & 1) == 0; align >>= 1, ++align2)
        continue;
       if (align != 1)
        {
@@ -1257,13 +1293,13 @@ m32r_scomm (ignore)
   else
     align2 = 0;
 
-  * p = 0;
+  *p = 0;
   symbolP = symbol_find_or_make (name);
-  * p = c;
+  *p = c;
 
   if (S_IS_DEFINED (symbolP))
     {
-      /* xgettext:c-format */
+      /* xgettext:c-format  */
       as_bad (_("Ignoring attempt to re-define symbol `%s'."),
              S_GET_NAME (symbolP));
       ignore_rest_of_line ();
@@ -1272,7 +1308,7 @@ m32r_scomm (ignore)
 
   if (S_GET_VALUE (symbolP) && S_GET_VALUE (symbolP) != (valueT) size)
     {
-      /* xgettext:c-format */
+      /* xgettext:c-format  */
       as_bad (_("Length of .scomm \"%s\" is already %ld. Not changed to %ld."),
              S_GET_NAME (symbolP),
              (long) S_GET_VALUE (symbolP),
@@ -1282,26 +1318,26 @@ m32r_scomm (ignore)
       return;
     }
 
-  if (symbolP->local)
+  if (symbol_get_obj (symbolP)->local)
     {
-      segT   old_sec    = now_seg;
-      int    old_subsec = now_subseg;
-      char * pfrag;
+      segT old_sec = now_seg;
+      int old_subsec = now_subseg;
+      char *pfrag;
 
       record_alignment (sbss_section, align2);
       subseg_set (sbss_section, 0);
-      
+
       if (align2)
        frag_align (align2, 0, 0);
-      
+
       if (S_GET_SEGMENT (symbolP) == sbss_section)
-       symbolP->sy_frag->fr_symbol = 0;
-      
-      symbolP->sy_frag = frag_now;
-      
+       symbol_get_frag (symbolP)->fr_symbol = 0;
+
+      symbol_set_frag (symbolP, frag_now);
+
       pfrag = frag_var (rs_org, 1, 1, (relax_substateT) 0, symbolP, size,
                        (char *) 0);
-      * pfrag = 0;
+      *pfrag = 0;
       S_SET_SIZE (symbolP, size);
       S_SET_SEGMENT (symbolP, sbss_section);
       S_CLEAR_EXTERNAL (symbolP);
@@ -1312,7 +1348,7 @@ m32r_scomm (ignore)
       S_SET_VALUE (symbolP, (valueT) size);
       S_SET_ALIGN (symbolP, align2);
       S_SET_EXTERNAL (symbolP);
-      S_SET_SEGMENT (symbolP, & scom_section);
+      S_SET_SEGMENT (symbolP, &scom_section);
     }
 
   demand_empty_rest_of_line ();
@@ -1349,9 +1385,10 @@ const relax_typeS md_relax_table[] =
 };
 
 long
-m32r_relax_frag (fragP, stretch)
-     fragS * fragP;
-     long    stretch;
+m32r_relax_frag (segment, fragP, stretch)
+     segT segment;
+     fragS *fragP;
+     long stretch;
 {
   /* Address of branch insn.  */
   long address = fragP->fr_address + fragP->fr_fix - 2;
@@ -1376,7 +1413,7 @@ m32r_relax_frag (fragP, stretch)
     }
   else
     {
-      growth = relax_frag (fragP, stretch);
+      growth = relax_frag (segment, fragP, stretch);
 
       /* Long jump on odd halfword boundary?  */
       if (fragP->fr_subtype == 2 && (address & 3) != 0)
@@ -1397,16 +1434,14 @@ m32r_relax_frag (fragP, stretch)
    Any symbol that is now undefined will not become defined.
    The guess for fr_var is ACTUALLY the growth beyond fr_fix.
    Whatever we do to grow fr_fix or fr_var contributes to our returned value.
-   Although it may not be explicit in the frag, pretend fr_var starts with a
-   0 value.  */
+   Although it may not be explicit in the frag, pretend fr_var starts
+   with a 0 value.  */
 
 int
 md_estimate_size_before_relax (fragP, segment)
-     fragS * fragP;
-     segT    segment;
+     fragS *fragP;
+     segT segment;
 {
-  int    old_fr_fix = fragP->fr_fix;
-
   /* The only thing we have to handle here are symbols outside of the
      current segment.  They may be undefined or in a different segment in
      which case linker scripts may place them anywhere.
@@ -1415,12 +1450,15 @@ md_estimate_size_before_relax (fragP, segment)
 
   if (S_GET_SEGMENT (fragP->fr_symbol) != segment)
     {
+      int old_fr_fix = fragP->fr_fix;
+
       /* The symbol is undefined in this segment.
         Change the relaxation subtype to the max allowable and leave
         all further handling to md_convert_frag.  */
       fragP->fr_subtype = 2;
 
-#if 0 /* Can't use this, but leave in for illustration.  */     
+#if 0
+      /* Can't use this, but leave in for illustration.  */
       /* Change 16 bit insn to 32 bit insn.  */
       fragP->fr_opcode[0] |= 0x80;
 
@@ -1430,17 +1468,18 @@ md_estimate_size_before_relax (fragP, segment)
       /* Create a relocation for it.  */
       fix_new (fragP, old_fr_fix, 4,
               fragP->fr_symbol,
-              fragP->fr_offset, 1 /* pcrel */,
+              fragP->fr_offset, 1 /* pcrel  */,
               /* FIXME: Can't use a real BFD reloc here.
                  gas_cgen_md_apply_fix3 can't handle it.  */
               BFD_RELOC_M32R_26_PCREL);
 
       /* Mark this fragment as finished.  */
       frag_wane (fragP);
+      return fragP->fr_fix - old_fr_fix;
 #else
       {
-       const CGEN_INSN * insn;
-       int               i;
+       const CGEN_INSN *insn;
+       int i;
 
        /* Update the recorded insn.
           Fortunately we don't have to look very far.
@@ -1451,7 +1490,7 @@ md_estimate_size_before_relax (fragP, segment)
            if ((strcmp (CGEN_INSN_MNEMONIC (insn),
                         CGEN_INSN_MNEMONIC (fragP->fr_cgen.insn))
                 == 0)
-               && CGEN_INSN_ATTR (insn, CGEN_INSN_RELAX))
+               && CGEN_INSN_ATTR_VALUE (insn, CGEN_INSN_RELAX))
              break;
          }
        if (i == 4)
@@ -1463,10 +1502,10 @@ md_estimate_size_before_relax (fragP, segment)
 #endif
     }
 
-  return (fragP->fr_var + fragP->fr_fix - old_fr_fix);
-} 
+  return md_relax_table[fragP->fr_subtype].rlx_length;
+}
 
-/* *fragP has been relaxed to its final size, and now needs to have
+/* *FRAGP has been relaxed to its final size, and now needs to have
    the bytes inside it modified to conform to the new size.
 
    Called after relaxation is finished.
@@ -1475,16 +1514,16 @@ md_estimate_size_before_relax (fragP, segment)
 
 void
 md_convert_frag (abfd, sec, fragP)
-  bfd *   abfd;
-  segT    sec;
-  fragS * fragP;
+     bfd *abfd;
+     segT sec;
+     fragS *fragP;
 {
-  char * opcode;
-  char * displacement;
-  int    target_address;
-  int    opcode_address;
-  int    extension;
-  int    addend;
+  char *opcode;
+  char *displacement;
+  int target_address;
+  int opcode_address;
+  int extension;
+  int addend;
 
   opcode = fragP->fr_opcode;
 
@@ -1493,29 +1532,29 @@ md_convert_frag (abfd, sec, fragP)
 
   switch (fragP->fr_subtype)
     {
-    case 1 :
+    case 1:
       extension = 0;
-      displacement = & opcode[1];
+      displacement = &opcode[1];
       break;
-    case 2 :
+    case 2:
       opcode[0] |= 0x80;
       extension = 2;
-      displacement = & opcode[1];
+      displacement = &opcode[1];
       break;
-    case 3 :
+    case 3:
       opcode[2] = opcode[0] | 0x80;
       md_number_to_chars (opcode, PAR_NOP_INSN, 2);
       opcode_address += 2;
       extension = 4;
-      displacement = & opcode[3];
+      displacement = &opcode[3];
       break;
-    default :
+    default:
       abort ();
     }
 
   if (S_GET_SEGMENT (fragP->fr_symbol) != sec)
     {
-      /* symbol must be resolved by linker */
+      /* Symbol must be resolved by linker.  */
       if (fragP->fr_offset & 3)
        as_warn (_("Addend to unresolved symbol not on word boundary."));
       addend = fragP->fr_offset >> 2;
@@ -1524,7 +1563,6 @@ md_convert_frag (abfd, sec, fragP)
     {
       /* Address we want to reach in file space.  */
       target_address = S_GET_VALUE (fragP->fr_symbol) + fragP->fr_offset;
-      target_address += fragP->fr_symbol->sy_frag->fr_address;
       addend = (target_address - (opcode_address & -4)) >> 2;
     }
 
@@ -1539,12 +1577,14 @@ md_convert_frag (abfd, sec, fragP)
                             /* Offset of branch insn in frag.  */
                             fragP->fr_fix + extension - 4,
                             fragP->fr_cgen.insn,
-                            4 /*length*/,
-                            /* FIXME: quick hack */
+                            4 /* Length.  */,
+                            /* FIXME: quick hack */
 #if 0
-                            CGEN_OPERAND_ENTRY (fragP->fr_cgen.opindex),
+                            cgen_operand_lookup_by_num (gas_cgen_cpu_desc,
+                                                        fragP->fr_cgen.opindex),
 #else
-                            CGEN_OPERAND_ENTRY (M32R_OPERAND_DISP24),
+                            cgen_operand_lookup_by_num (gas_cgen_cpu_desc,
+                                                        M32R_OPERAND_DISP24),
 #endif
                             fragP->fr_cgen.opinfo,
                             fragP->fr_symbol, fragP->fr_offset);
@@ -1565,8 +1605,8 @@ md_convert_frag (abfd, sec, fragP)
 
 long
 md_pcrel_from_section (fixP, sec)
-     fixS * fixP;
-     segT   sec;
+     fixS *fixP;
+     segT sec;
 {
   if (fixP->fx_addsy != (symbolS *) NULL
       && (! S_IS_DEFINED (fixP->fx_addsy)
@@ -1586,24 +1626,25 @@ md_pcrel_from_section (fixP, sec)
 
 bfd_reloc_code_real_type
 md_cgen_lookup_reloc (insn, operand, fixP)
-     const CGEN_INSN *    insn;
-     const CGEN_OPERAND * operand;
-     fixS *               fixP;
+     const CGEN_INSN *insn;
+     const CGEN_OPERAND *operand;
+     fixS *fixP;
 {
-  switch (CGEN_OPERAND_TYPE (operand))
+  switch (operand->type)
     {
-    case M32R_OPERAND_DISP8 : return  BFD_RELOC_M32R_10_PCREL;
-    case M32R_OPERAND_DISP16 : return BFD_RELOC_M32R_18_PCREL;
-    case M32R_OPERAND_DISP24 : return BFD_RELOC_M32R_26_PCREL;
-    case M32R_OPERAND_UIMM24 : return BFD_RELOC_M32R_24;
-    case M32R_OPERAND_HI16 :
-    case M32R_OPERAND_SLO16 :
-    case M32R_OPERAND_ULO16 :
+    case M32R_OPERAND_DISP8:  return BFD_RELOC_M32R_10_PCREL;
+    case M32R_OPERAND_DISP16: return BFD_RELOC_M32R_18_PCREL;
+    case M32R_OPERAND_DISP24: return BFD_RELOC_M32R_26_PCREL;
+    case M32R_OPERAND_UIMM24: return BFD_RELOC_M32R_24;
+    case M32R_OPERAND_HI16:
+    case M32R_OPERAND_SLO16:
+    case M32R_OPERAND_ULO16:
       /* If low/high/shigh/sda was used, it is recorded in `opinfo'.  */
-      if (fixP->tc_fix_data.opinfo != 0)
-       return fixP->tc_fix_data.opinfo;
+      if (fixP->fx_cgen.opinfo != 0)
+       return fixP->fx_cgen.opinfo;
       break;
-    default : /* avoid -Wall warning */
+    default:
+      /* Avoid -Wall warning.  */
       break;
     }
   return BFD_RELOC_NONE;
@@ -1613,11 +1654,11 @@ md_cgen_lookup_reloc (insn, operand, fixP)
 
 static void
 m32r_record_hi16 (reloc_type, fixP, seg)
-     int    reloc_type;
-     fixS * fixP;
-     segT   seg;
+     int reloc_type;
+     fixS *fixP;
+     segT seg;
 {
-  struct m32r_hi_fixup * hi_fixup;
+  struct m32r_hi_fixup *hi_fixup;
 
   assert (reloc_type == BFD_RELOC_M32R_HI16_SLO
          || reloc_type == BFD_RELOC_M32R_HI16_ULO);
@@ -1627,7 +1668,7 @@ m32r_record_hi16 (reloc_type, fixP, seg)
   hi_fixup->fixp = fixP;
   hi_fixup->seg  = now_seg;
   hi_fixup->next = m32r_hi_fixup_list;
-  
+
   m32r_hi_fixup_list = hi_fixup;
 }
 
@@ -1636,26 +1677,27 @@ m32r_record_hi16 (reloc_type, fixP, seg)
 
 fixS *
 m32r_cgen_record_fixup_exp (frag, where, insn, length, operand, opinfo, exp)
-     fragS *              frag;
-     int                  where;
-     const CGEN_INSN *    insn;
-     int                  length;
-     const CGEN_OPERAND * operand;
-     int                  opinfo;
-     expressionS *        exp;
+     fragS *frag;
+     int where;
+     const CGEN_INSN *insn;
+     int length;
+     const CGEN_OPERAND *operand;
+     int opinfo;
+     expressionS *exp;
 {
-  fixS * fixP = gas_cgen_record_fixup_exp (frag, where, insn, length,
-                                          operand, opinfo, exp);
+  fixS *fixP = gas_cgen_record_fixup_exp (frag, where, insn, length,
+                                         operand, opinfo, exp);
 
-  switch (CGEN_OPERAND_TYPE (operand))
+  switch (operand->type)
     {
-    case M32R_OPERAND_HI16 :
+    case M32R_OPERAND_HI16:
       /* If low/high/shigh/sda was used, it is recorded in `opinfo'.  */
-      if (fixP->tc_fix_data.opinfo == BFD_RELOC_M32R_HI16_SLO
-         || fixP->tc_fix_data.opinfo == BFD_RELOC_M32R_HI16_ULO)
-       m32r_record_hi16 (fixP->tc_fix_data.opinfo, fixP, now_seg);
+      if (fixP->fx_cgen.opinfo == BFD_RELOC_M32R_HI16_SLO
+         || fixP->fx_cgen.opinfo == BFD_RELOC_M32R_HI16_ULO)
+       m32r_record_hi16 (fixP->fx_cgen.opinfo, fixP, now_seg);
       break;
-    default : /* avoid -Wall warning */
+    default:
+      /* Avoid -Wall warning */
       break;
     }
 
@@ -1665,7 +1707,7 @@ m32r_cgen_record_fixup_exp (frag, where, insn, length, operand, opinfo, exp)
 /* Return BFD reloc type from opinfo field in a fixS.
    It's tricky using fx_r_type in m32r_frob_file because the values
    are BFD_RELOC_UNUSED + operand number.  */
-#define FX_OPINFO_R_TYPE(f) ((f)->tc_fix_data.opinfo)
+#define FX_OPINFO_R_TYPE(f) ((f)->fx_cgen.opinfo)
 
 /* Sort any unmatched HI16 relocs so that they immediately precede
    the corresponding LO16 reloc.  This is called before md_apply_fix and
@@ -1674,12 +1716,12 @@ m32r_cgen_record_fixup_exp (frag, where, insn, length, operand, opinfo, exp)
 void
 m32r_frob_file ()
 {
-  struct m32r_hi_fixup * l;
+  struct m32r_hi_fixup *l;
 
   for (l = m32r_hi_fixup_list; l != NULL; l = l->next)
     {
-      segment_info_type * seginfo;
-      int                 pass;
+      segment_info_type *seginfo;
+      int pass;
 
       assert (FX_OPINFO_R_TYPE (l->fixp) == BFD_RELOC_M32R_HI16_SLO
              || FX_OPINFO_R_TYPE (l->fixp) == BFD_RELOC_M32R_HI16_ULO);
@@ -1699,8 +1741,8 @@ m32r_frob_file ()
       seginfo = seg_info (l->seg);
       for (pass = 0; pass < 2; pass++)
        {
-         fixS * f;
-         fixS * prev;
+         fixS *f;
+         fixS *prev;
 
          prev = NULL;
          for (f = seginfo->fix_root; f != NULL; f = f->fx_next)
@@ -1714,17 +1756,17 @@ m32r_frob_file ()
                      || (FX_OPINFO_R_TYPE (prev) != BFD_RELOC_M32R_HI16_SLO
                          && FX_OPINFO_R_TYPE (prev) != BFD_RELOC_M32R_HI16_ULO)
                      || prev->fx_addsy != f->fx_addsy
-                     || prev->fx_offset !=  f->fx_offset))
+                     || prev->fx_offset != f->fx_offset))
                {
-                 fixS ** pf;
+                 fixS **pf;
 
                  /* Move l->fixp before f.  */
                  for (pf = &seginfo->fix_root;
-                      * pf != l->fixp;
-                      pf = & (* pf)->fx_next)
-                   assert (* pf != NULL);
+                      *pf != l->fixp;
+                      pf = & (*pf)->fx_next)
+                   assert (*pf != NULL);
 
-                 * pf = l->fixp->fx_next;
+                 *pf = l->fixp->fx_next;
 
                  l->fixp->fx_next = f;
                  if (prev == NULL)
@@ -1755,7 +1797,7 @@ m32r_frob_file ()
 
 int
 m32r_force_relocation (fix)
-     fixS * fix;
+     fixS *fix;
 {
   if (fix->fx_r_type == BFD_RELOC_VTABLE_INHERIT
       || fix->fx_r_type == BFD_RELOC_VTABLE_ENTRY)
@@ -1764,17 +1806,16 @@ m32r_force_relocation (fix)
   if (! m32r_relax)
     return 0;
 
-  return (fix->fx_pcrel
-         || 0 /* ??? */);
+  return fix->fx_pcrel;
 }
 \f
 /* Write a value out to the object file, using the appropriate endianness.  */
 
 void
 md_number_to_chars (buf, val, n)
-     char * buf;
+     char *buf;
      valueT val;
-     int    n;
+     int n;
 {
   if (target_big_endian)
     number_to_chars_bigendian (buf, val, n);
@@ -1782,12 +1823,12 @@ md_number_to_chars (buf, val, n)
     number_to_chars_littleendian (buf, val, n);
 }
 
-/* Turn a string in input_line_pointer into a floating point constant of type
-   type, and store the appropriate bytes in *litP.  The number of LITTLENUMS
-   emitted is stored in *sizeP .  An error message is returned, or NULL on OK.
-*/
+/* Turn a string in input_line_pointer into a floating point constant
+   of type TYPE, and store the appropriate bytes in *LITP.  The number
+   of LITTLENUMS emitted is stored in *SIZEP.  An error message is
+   returned, or NULL on OK.  */
 
-/* Equal to MAX_PRECISION in atof-ieee.c */
+/* Equal to MAX_PRECISION in atof-ieee.c */
 #define MAX_LITTLENUMS 6
 
 char *
@@ -1796,11 +1837,11 @@ md_atof (type, litP, sizeP)
      char *litP;
      int *sizeP;
 {
-  int              i;
-  int              prec;
-  LITTLENUM_TYPE   words [MAX_LITTLENUMS];
-  char *           t;
-  char *           atof_ieee ();
+  int i;
+  int prec;
+  LITTLENUM_TYPE words[MAX_LITTLENUMS];
+  char *t;
+  char *atof_ieee ();
 
   switch (type)
     {
@@ -1818,17 +1859,18 @@ md_atof (type, litP, sizeP)
       prec = 4;
       break;
 
-   /* FIXME: Some targets allow other format chars for bigger sizes here.  */
+      /* FIXME: Some targets allow other format chars for bigger sizes
+         here.  */
 
     default:
-      * sizeP = 0;
+      *sizeP = 0;
       return _("Bad call to md_atof()");
     }
 
   t = atof_ieee (input_line_pointer, type, words);
   if (t)
     input_line_pointer = t;
-  * sizeP = prec * sizeof (LITTLENUM_TYPE);
+  *sizeP = prec * sizeof (LITTLENUM_TYPE);
 
   if (target_big_endian)
     {
@@ -1848,7 +1890,7 @@ md_atof (type, litP, sizeP)
          litP += sizeof (LITTLENUM_TYPE);
        }
     }
-     
+
   return 0;
 }
 
@@ -1858,28 +1900,44 @@ m32r_elf_section_change_hook ()
   /* If we have reached the end of a section and we have just emitted a
      16 bit insn, then emit a nop to make sure that the section ends on
      a 32 bit boundary.  */
-  
+
   if (prev_insn.insn || seen_relaxable_p)
     (void) m32r_fill_insn (0);
 }
 
+/* Return true if can adjust the reloc to be relative to its section
+   (such as .data) instead of relative to some symbol.  */
+
 boolean
 m32r_fix_adjustable (fixP)
    fixS *fixP;
 {
 
+  bfd_reloc_code_real_type reloc_type;
+
+  if ((int) fixP->fx_r_type >= (int) BFD_RELOC_UNUSED)
+    {
+      const CGEN_INSN *insn = NULL;
+      int opindex = (int) fixP->fx_r_type - (int) BFD_RELOC_UNUSED;
+      const CGEN_OPERAND *operand =
+       cgen_operand_lookup_by_num(gas_cgen_cpu_desc, opindex);
+      reloc_type = md_cgen_lookup_reloc (insn, operand, fixP);
+    }
+  else
+    reloc_type = fixP->fx_r_type;
+
   if (fixP->fx_addsy == NULL)
     return 1;
-  
-  /* Prevent all adjustments to global symbols. */
+
+  /* Prevent all adjustments to global symbols.  */
   if (S_IS_EXTERN (fixP->fx_addsy))
     return 0;
   if (S_IS_WEAK (fixP->fx_addsy))
     return 0;
-  
-  /* We need the symbol name for the VTABLE entries */
-  if (fixP->fx_r_type == BFD_RELOC_VTABLE_INHERIT
-      || fixP->fx_r_type == BFD_RELOC_VTABLE_ENTRY)
+
+  /* We need the symbol name for the VTABLE entries */
+  if (reloc_type == BFD_RELOC_VTABLE_INHERIT
+      || reloc_type == BFD_RELOC_VTABLE_ENTRY)
     return 0;
 
   return 1;
This page took 0.052445 seconds and 4 git commands to generate.