Locale changes from Bruno Haible <haible@clisp.cons.org>.
[deliverable/binutils-gdb.git] / gas / config / tc-m32r.c
index c2e61546df1a0e905cc55791972ef4303a1aba6b..6ee3a2e8dbf658e83b3b558ad87b6e41ef02e3cc 100644 (file)
@@ -1,5 +1,6 @@
-/* tc-m32r.c -- Assembler for the Mitsubishi M32R/X.
-   Copyright (C) 1996, 1997, 1998 Free Software Foundation.
+/* tc-m32r.c -- Assembler for the Mitsubishi M32R.
+   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 "cgen-opc.h"
+#include "safe-ctype.h"
+#include "subsegs.h"
+#include "symcat.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
+   beginning of the current instruction.  */
+typedef struct sym_link
+{
+  struct sym_link *next;
+  symbolS *symbol;
+} sym_linkS;
+
+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;
+#if CGEN_INT_INSN_P
+  CGEN_INSN_INT buffer[1];
+#define INSN_VALUE(buf) (*(buf))
+#else
+  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;
+}
+m32r_insn;
 
-/* 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 const CGEN_INSN * prev_insn = NULL;
-static CGEN_FIELDS       prev_fields;
+/* 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;
 
 /* Non-zero if we've seen a relaxable insn since the last 32 bit
    alignment request.  */
@@ -39,17 +75,40 @@ static int seen_relaxable_p = 0;
    This flag does not apply to them.  */
 static int m32r_relax;
 
+#if 0
+/* Not supported yet.  */
 /* If non-NULL, pointer to cpu description file to read.
    This allows runtime additions to the assembler.  */
-static 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.
+   Each high/shigh reloc must be paired with it's low cousin in order to
+   properly calculate the addend in a relocatable link (since there is a
+   potential carry from the low to the high/shigh).
+   This option is off by default though for user-written assembler code it
+   might make sense to make the default be on (i.e. have gcc pass a flag
+   to turn it off).  This warning must not be on for GCC created code as
+   optimization may delete the low but not the high/shigh (at least we
+   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;
-/* end-sanitize-m32rx */
 
-/* stuff for .scomm symbols.  */
+/* Non-zero if -m32rx -hidden has been specified, in which case support for
+   the special M32RX instruction set should be enabled.  */
+static int enable_special = 0;
+
+/* Non-zero if the programmer should be warned when an explicit parallel
+   instruction might have constraint violations.  */
+static int warn_explicit_parallel_conflicts = 1;
+
+/* Non-zero if insns can be made parallel.  */
+static int optimize;
+
+/* Stuff for .scomm symbols.  */
 static segT     sbss_section;
 static asection scom_section;
 static asymbol  scom_symbol;
@@ -69,26 +128,27 @@ 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 void m32r_record_hi16 PARAMS ((int, fixS *, segT seg));
-
+static struct m32r_hi_fixup *m32r_hi_fixup_list;
 \f
-/* start-sanitize-m32rx */
 static void
-allow_m32rx (int on)
+allow_m32rx (on)
+     int on;
 {
   enable_m32rx = on;
 
@@ -96,42 +156,93 @@ allow_m32rx (int on)
     bfd_set_arch_mach (stdoutput, TARGET_ARCH,
                       enable_m32rx ? bfd_mach_m32rx : bfd_mach_m32r);
 }
-/* end-sanitize-m32rx */
 \f
-const char * md_shortopts = "";
+#define M32R_SHORTOPTS "O"
+
+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},
-/* end-sanitize-m32rx */
+  {"warn-explicit-parallel-conflicts", no_argument, NULL, OPTION_WARN_PARALLEL},
+  {"Wp", no_argument, NULL, OPTION_WARN_PARALLEL},
+  {"no-warn-explicit-parallel-conflicts", no_argument, NULL, OPTION_NO_WARN_PARALLEL},
+  {"Wnp", no_argument, NULL, OPTION_NO_WARN_PARALLEL},
+  {"hidden", no_argument, NULL, OPTION_SPECIAL},
+  /* Sigh.  I guess all warnings must now have both variants.  */
+  {"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_MD_BASE + 1)
+#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 + 2)
   {"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;
-/* end-sanitize-m32rx */
-      
-#if 0 /* not supported yet */
+
+    case OPTION_WARN_PARALLEL:
+      warn_explicit_parallel_conflicts = 1;
+      break;
+
+    case OPTION_NO_WARN_PARALLEL:
+      warn_explicit_parallel_conflicts = 0;
+      break;
+
+    case OPTION_SPECIAL:
+      if (enable_m32rx)
+       enable_special = 1;
+      else
+       {
+         /* Pretend that we do not recognise this option.  */
+         as_bad (_("Unrecognised option: -hidden"));
+         return 0;
+       }
+      break;
+
+    case OPTION_WARN_UNMATCHED:
+      warn_unmatched_high = 1;
+      break;
+
+    case OPTION_NO_WARN_UNMATCHED:
+      warn_unmatched_high = 0;
+      break;
+
+#if 0
+    /* Not supported yet.  */
     case OPTION_RELAX:
       m32r_relax = 1;
       break;
@@ -139,32 +250,61 @@ md_parse_option (c, arg)
       m32r_cpu_desc = arg;
       break;
 #endif
+
     default:
       return 0;
     }
+
   return 1;
 }
 
 void
 md_show_usage (stream)
-  FILE * stream;
+     FILE *stream;
 {
-  fprintf (stream, "M32R/X options:\n");
-/* start-sanitize-m32rx */
-  fprintf (stream, "\
---m32rx                        support the extended m32rx instruction set\n");
-/* end-sanitize-m32rx */
+  fprintf (stream, _(" M32R specific command line options:\n"));
+
+  fprintf (stream, _("\
+  -m32r                   disable support for the m32rx instruction set\n"));
+  fprintf (stream, _("\
+  -m32rx                  support the extended m32rx instruction set\n"));
+  fprintf (stream, _("\
+  -O                      try to combine instructions in parallel\n"));
+
+  fprintf (stream, _("\
+  -warn-explicit-parallel-conflicts     warn when parallel instructions\n"));
+  fprintf (stream, _("\
+                                         violate contraints\n"));
+  fprintf (stream, _("\
+  -no-warn-explicit-parallel-conflicts  do not warn when parallel\n"));
+  fprintf (stream, _("\
+                                         instructions violate contraints\n"));
+  fprintf (stream, _("\
+  -Wp                     synonym for -warn-explicit-parallel-conflicts\n"));
+  fprintf (stream, _("\
+  -Wnp                    synonym for -no-warn-explicit-parallel-conflicts\n"));
+
+  fprintf (stream, _("\
+  -warn-unmatched-high    warn when an (s)high reloc has no matching low reloc\n"));
+  fprintf (stream, _("\
+  -no-warn-unmatched-high do not warn about missing low relocs\n"));
+  fprintf (stream, _("\
+  -Wuh                    synonym for -warn-unmatched-high\n"));
+  fprintf (stream, _("\
+  -Wnuh                   synonym for -no-warn-unmatched-high\n"));
 
 #if 0
-  fprintf (stream, "\
---relax                        create linker relaxable code\n");
-  fprintf (stream, "\
---cpu-desc             provide runtime cpu description file\n");
+  fprintf (stream, _("\
+  -relax                 create linker relaxable code\n"));
+  fprintf (stream, _("\
+  -cpu-desc              provide runtime cpu description file\n"));
 #endif
-} 
+}
 
 static void fill_insn PARAMS ((int));
 static void m32r_scomm PARAMS ((int));
+static void debug_sym PARAMS ((int));
+static void expand_debug_syms PARAMS ((sym_linkS *, int));
 
 /* Set by md_assemble for use by m32r_fill_insn.  */
 static subsegT prev_subseg;
@@ -173,73 +313,59 @@ static segT prev_seg;
 /* The target specific pseudo-ops which we support.  */
 const pseudo_typeS md_pseudo_table[] =
 {
-  { "word", cons, 4 },
-  { "fillinsn", fill_insn, 0 },
-  { "scomm", m32r_scomm, 0 },
-/* start-sanitize-m32rx */
-  { "m32r",  allow_m32rx, 0},
-  { "m32rx", allow_m32rx, 1},
-/* end-sanitize-m32rx */
+  { "word",    cons,           4 },
+  { "fillinsn", fill_insn,     0 },
+  { "scomm",   m32r_scomm,     0 },
+  { "debugsym",        debug_sym,      0 },
+  /* Not documented as so far there is no need for them....  */
+  { "m32r",    allow_m32rx,    0 },
+  { "m32rx",   allow_m32rx,    1 },
   { 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;
 {
-  if ((fill == NULL || (* fill == 0 && len == 1))
-      && (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);
-       }
-      return 1;
+  if (bytes & 2)
+    {
+      memcpy (p, nop_pattern, 2);
+      p += 2;
+      bytes -= 2;
+      fix += 2;
     }
 
-  return 0;
-}
+  memcpy (p, multi_nop_pattern, 4);
 
-static void
-assemble_nop (opcode)
-     int opcode;
-{
-  char * f = frag_more (2);
-  md_number_to_chars (f, opcode, 2);
+  fragp->fr_fix += fix;
+  fragp->fr_var = 4;
 }
 
 /* If the last instruction was the first of 2 16 bit insns,
@@ -255,13 +381,83 @@ static void
 fill_insn (ignore)
      int ignore;
 {
-  (void) m32r_do_align (2, NULL, 0, 0);
-  prev_insn = NULL;
+  frag_align_code (2, 0);
+  prev_insn.insn = NULL;
   seen_relaxable_p = 0;
 }
 
-/* Cover function to fill_insn called after a label and at end of assembly.
+/* Record the symbol so that when we output the insn, we can create
+   a symbol that is at the start of the instruction.  This is used
+   to emit the label for the start of a breakpoint without causing
+   the assembler to emit a NOP if the previous instruction was a
+   16 bit instruction.  */
 
+static void
+debug_sym (ignore)
+     int ignore;
+{
+  register char *name;
+  register char delim;
+  register char *end_name;
+  register symbolS *symbolP;
+  register sym_linkS *link;
+
+  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)
+    {
+      symbolP = symbol_new (name, undefined_section, 0, &zero_address_frag);
+    }
+
+  symbol_table_insert (symbolP);
+  if (S_IS_DEFINED (symbolP) && S_GET_SEGMENT (symbolP) != reg_section)
+    /* xgettext:c-format */
+    as_bad (_("symbol `%s' already defined"), S_GET_NAME (symbolP));
+
+  else
+    {
+      link = (sym_linkS *) xmalloc (sizeof (sym_linkS));
+      link->symbol = symbolP;
+      link->next = debug_sym_link;
+      debug_sym_link = link;
+      symbol_get_obj (symbolP)->local = 1;
+    }
+
+  *end_name = delim;
+  demand_empty_rest_of_line ();
+}
+
+/* Second pass to expanding the debug symbols, go through linked
+   list of symbols and reassign the address.  */
+
+static void
+expand_debug_syms (syms, align)
+     sym_linkS *syms;
+     int align;
+{
+  char *save_input_line = input_line_pointer;
+  sym_linkS *next_syms;
+
+  if (!syms)
+    return;
+
+  (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);
+    }
+
+  input_line_pointer = save_input_line;
+}
+
+/* Cover function to fill_insn called after a label and at end of assembly.
    The result is always 1: we're called in a conditional to see if the
    current line is a label.  */
 
@@ -269,21 +465,24 @@ int
 m32r_fill_insn (done)
      int done;
 {
-  segT    seg;
-  subsegT subseg;
-
   if (prev_seg != NULL)
     {
-      seg    = now_seg;
-      subseg = now_subseg;
-      
+      segT seg = now_seg;
+      subsegT subseg = now_subseg;
+
       subseg_set (prev_seg, prev_subseg);
-      
+
       fill_insn (0);
-      
+
       subseg_set (seg, subseg);
     }
-  
+
+  if (done && debug_sym_link)
+    {
+      expand_debug_syms (debug_sym_link, 1);
+      debug_sym_link = (sym_linkS *) 0;
+    }
+
   return 1;
 }
 \f
@@ -291,26 +490,34 @@ void
 md_begin ()
 {
   flagword applicable;
-  segT     seg;
-  subsegT  subseg;
+  segT seg;
+  subsegT subseg;
 
   /* Initialize the `cgen' interface.  */
 
-  /* This is a callback from cgen to gas to parse operands.  */
-  cgen_parse_operand_fn = cgen_parse_operand;
-  
   /* Set the machine number and endian.  */
-  CGEN_SYM (init_asm) (0 /* mach number */,
-                      target_big_endian ?
-                      CGEN_ENDIAN_BIG : CGEN_ENDIAN_LITTLE);
+  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_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 (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);
     }
@@ -323,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
 
@@ -338,378 +546,643 @@ 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 ();
 }
 
-/* Returns non zero if the given instruction writes to a destination register.  */
-static int
-writes_to_dest_reg (insn)
-     const CGEN_INSN * insn;
-{
-  unsigned char * syntax = CGEN_SYNTAX_STRING (CGEN_INSN_SYNTAX (insn));
-  unsigned char   c;
-  
-  /* Scan the syntax string looking for a destination register.  */
-  while ((c = (* syntax ++)) != 0)
-    if (c == 128 + M32R_OPERAND_DR)
-      break;
+#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)))
 
-  return c;
-}
+/* 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.  */
 
-/* Returns non zero if the given instruction reads from a source register.
-   Syntax characters equal to 'ignore' are skipped as they have already been
-   processed.  (This works provided that no potential parallel instruction
-   can have more than 2 input registers).  */
 static int
-reads_from_src_reg (insn, ignore)
-     const CGEN_INSN * insn;
-     unsigned char     ignore;
+first_writes_to_seconds_operands (a, b, check_outputs)
+     m32r_insn *a;
+     m32r_insn *b;
+     const int check_outputs;
 {
-  unsigned char * syntax = CGEN_SYNTAX_STRING (CGEN_INSN_SYNTAX (insn));
-  unsigned char   c;
-  
-  /* Scan the syntax string looking for a source register.  */
-  while ((c = (* syntax ++)) != 0)
+  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;
+       a_operands->type != CGEN_OPINST_END;
+       a_index ++, a_operands ++)
     {
-      if (c == ignore)
-       continue;
-      
-      if (   c == 128 + M32R_OPERAND_SR
-         || c == 128 + M32R_OPERAND_SRC1
-         || c == 128 + M32R_OPERAND_SRC2)
-       break;
+      if (a_operands->type == CGEN_OPINST_OUTPUT)
+       {
+         int b_index;
+         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....  */
+
+         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;
+                  b_operands->type != CGEN_OPINST_END;
+                  b_index++, b_operands++)
+               {
+                 if ((b_operands->type
+                      == (check_outputs
+                          ? CGEN_OPINST_OUTPUT
+                          : CGEN_OPINST_INPUT))
+                     && OPERAND_IS_COND_BIT (b_operands, b->indices, b_index))
+                   return 1;
+               }
+           }
+         else
+           {
+             /* Scan operand list of 'b' looking for an operand that
+                references the same hardware element, and which goes in the
+                right direction.  */
+             for (b_index = 0;
+                  b_operands->type != CGEN_OPINST_END;
+                  b_index++, b_operands++)
+               {
+                 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 c;
+  return 0;
 }
 
-/* Returns the integer value of the destination register held in the fields. */
-#define get_dest_reg(fields) fields->f_r1
+/* Returns true if the insn can (potentially) alter the program counter.  */
 
-/* Returns an integer representing the source register of the given type.  */
 static int
-get_src_reg (syntax, fields)
-     unsigned char syntax;
-     CGEN_FIELDS * fields;
+writes_to_pc (a)
+     m32r_insn *a;
 {
-  switch (syntax)
+#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 (a_operands->type != CGEN_OPINST_END)
     {
-    case 128 + M32R_OPERAND_SR:    return fields->f_r2;
-      /* Relies upon the fact that no instruction with a $src1 operand
-        also has a $dr operand.  */
-    case 128 + M32R_OPERAND_SRC1:  return fields->f_r1;
-    case 128 + M32R_OPERAND_SRC2:  return fields->f_r2;
-    default:                       abort(); return -1;
+      if (a_operands->operand != NULL
+         && CGEN_OPERAND_INDEX (gas_cgen_cpu_desc,
+                                a_operands->operand) == M32R_OPERAND_PC)
+       return 1;
+
+      a_operands++;
     }
+#else
+  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;
 }
 
-/* start-sanitize-m32rx */
-/* 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, a_fields, b, b_fields, test_a_inputs, test_b_inputs)
-     const CGEN_INSN * a;
-     CGEN_FIELDS *     a_fields;
-     const CGEN_INSN * b;
-     CGEN_FIELDS *     b_fields;
-     int               test_a_inputs;
-     int               test_b_inputs;
+can_make_parallel (a, 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();
-  
-  a_pipe = CGEN_INSN_ATTR (a, CGEN_INSN_PIPE);
-  b_pipe = CGEN_INSN_ATTR (b, CGEN_INSN_PIPE);
-
-  if (   a_pipe == PIPE_NONE
+  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_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
       || b_pipe == PIPE_NONE)
-    return "Instructions do not use parallel execution pipelines.";
-  
-  if (   a_pipe == PIPE_S
+    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
       || b_pipe == PIPE_O)
-    return "Instructions share the same execution pipeline";
+    return _("Instructions share the same execution pipeline");
 
-  if (   writes_to_dest_reg (a)
-      && writes_to_dest_reg (b)
-      && (get_dest_reg (a_fields) == get_dest_reg (b_fields)))
-    return "Instructions write to the same destination register.";
+  return NULL;
+}
+
+/* Force the top bit of the second 16-bit insn to be set.  */
+
+static void
+make_parallel (buffer)
+     CGEN_INSN_BYTES_PTR buffer;
+{
+#if CGEN_INT_INSN_P
+  *buffer |= 0x8000;
+#else
+  buffer[CGEN_CPU_ENDIAN (gas_cgen_cpu_desc) == CGEN_ENDIAN_BIG ? 0 : 1]
+    |= 0x80;
+#endif
+}
+
+/* Same as make_parallel except buffer contains the bytes in target order.  */
 
-  /* If requested, make sure that the first instruction does not
-     overwrite the inputs of the second instruction.  */
-  if (test_b_inputs && writes_to_dest_reg (a))
+static void
+target_make_parallel (buffer)
+     char *buffer;
+{
+  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 *str3;
+  m32r_insn first;
+  m32r_insn second;
+  char *errmsg;
+  char save_str2 = *str2;
+
+  /* Seperate the two instructions.  */
+  *str2 = 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;
+
+  /* Parse the first instruction.  */
+  if (! (first.insn = m32r_cgen_assemble_insn
+        (gas_cgen_cpu_desc, str, & first.fields, first.buffer, & errmsg)))
     {
-      unsigned char skip = 1;
-      
-      while (skip = reads_from_src_reg (b, skip))
-       {
-         if (get_src_reg (skip, b_fields) == get_dest_reg (a_fields))
-           return "First instruction writes to register read by the second instruction";
-       }
+      as_bad (errmsg);
+      return;
+    }
+
+  /* Check it.  */
+  if (CGEN_FIELDS_BITSIZE (&first.fields) != 16)
+    {
+      /* 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_VALUE (first.insn, CGEN_INSN_MACH)
+              == (1 << MACH_M32RX)))
+    {
+      /* 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_VALUE (first.insn, CGEN_INSN_PIPE) == PIPE_NONE)
+    {
+      /* xgettext:c-format  */
+      as_bad (_("instruction '%s' cannot be executed in parallel."), str);
+      return;
+    }
+
+  /* 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++))
+      continue;
+
+    --s2;
+
+    while (ISALNUM (*s2))
+      {
+       *s2 = TOLOWER (*s2);
+       s2++;
+      }
+  }
+
+  /* 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
+     doesn't seem right.  Perhaps allow passing fields like we do insn.  */
+  /* FIXME: ALIAS insns do not have operands, so we use this function
+     to find the equivalent insn and overwrite the value stored in our
+     structure.  We still need the original insn, however, since this
+     may have certain attributes that are not present in the unaliased
+     version (eg relaxability).  When aliases behave differently this
+     may have to change.  */
+  first.orig_insn = first.insn;
+  {
+    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"));
+
+  second.debug_sym_link = NULL;
+
+  /* Parse the second instruction.  */
+  if (! (second.insn = m32r_cgen_assemble_insn
+        (gas_cgen_cpu_desc, str, & second.fields, second.buffer, & errmsg)))
+    {
+      as_bad (errmsg);
+      return;
+    }
+
+  /* Check it.  */
+  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  */
+      as_bad (_("unknown instruction '%s'"), str);
+      return;
+    }
+  else if (! enable_m32rx
+      && CGEN_INSN_ATTR_VALUE (second.insn, CGEN_INSN_MACH) == (1 << MACH_M32RX))
+    {
+      /* xgettext:c-format  */
+      as_bad (_("instruction '%s' is for the M32RX only"), str);
+      return;
     }
-  
-  /* Similarly, if requested, make sure that the second instruction
-     does not overwrite the inputs of the first instruction.  */
-  if (test_a_inputs && writes_to_dest_reg (b))
+
+  /* Check to see if this is an allowable parallel insn.  */
+  if (parallel_p
+      && CGEN_INSN_ATTR_VALUE (second.insn, CGEN_INSN_PIPE) == PIPE_NONE)
+    {
+      /* xgettext:c-format  */
+      as_bad (_("instruction '%s' cannot be executed in parallel."), str);
+      return;
+    }
+
+  if (parallel_p && ! enable_m32rx)
     {
-      unsigned char skip = 1;
-      
-      while (skip = reads_from_src_reg (a, skip))
+      if (CGEN_INSN_NUM (first.insn) != M32R_INSN_NOP
+         && CGEN_INSN_NUM (second.insn) != M32R_INSN_NOP)
        {
-         if (get_src_reg (skip, a_fields) == get_dest_reg (b_fields))
-           return "Second instruction writes to register read by the first instruction";
+         /* xgettext:c-format  */
+         as_bad (_("'%s': only the NOP instruction can be issued in parallel on the m32r"), str2);
+         return;
        }
     }
-  
-  return NULL;
-}
-/* end-sanitize-m32rx */
 
+  /* Get the indices of the operands of the instruction.  */
+  second.orig_insn = second.insn;
+  {
+    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"));
+
+  /* We assume that if the first instruction writes to a register that is
+     read by the second instruction it is because the programmer intended
+     this to happen, (after all they have explicitly requested that these
+     two instructions be executed in parallel).  Although if the global
+     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  */
+       as_warn (_("%s: output of 1st instruction is the same as an input to 2nd instruction - is this intentional ?"), str2);
 
-#ifdef CGEN_INT_INSN
-static void
-make_parallel (insn, buffer)
-     const CGEN_INSN * insn;
-     cgen_insn_t *     buffer;
-{
-  /* Force the top bit of the second insn to be set.  */
+      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);
+    }
 
-  bfd_vma value;
-      
-  if (CGEN_CURRENT_ENDIAN == CGEN_ENDIAN_BIG)
+  if (!parallel_p
+      || (errmsg = (char *) can_make_parallel (&first, &second)) == NULL)
     {
-      value = bfd_getb16 ((bfd_byte *) buffer);
-      value |= 0x8000;
-      bfd_putb16 (value, (char *) buffer);
+      /* Get the fixups for the first instruction.  */
+      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);
+
+      /* 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 (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);
+    }
+  /* Try swapping the instructions to see if they work that way.  */
+  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);
+
+      /* 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 (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);
     }
   else
     {
-      value = bfd_getl16 ((bfd_byte *) buffer);
-      value |= 0x8000;
-      bfd_putl16 (value, (char *) buffer);
+      as_bad ("'%s': %s", str2, errmsg);
+      return;
     }
-}
-#else
-static void
-make_parallel (insn, buffer)
-     const CGEN_INSN * insn;
-     char *            buffer;
-{
-  /* Force the top bit of the second insn to be set.  */
 
-  buffer [CGEN_CURRENT_ENDIAN == CGEN_ENDIAN_BIG ? 0 : 1] |= 0x80;
+  /* Set these so m32r_fill_insn can use them.  */
+  prev_seg    = now_seg;
+  prev_subseg = now_subseg;
 }
-#endif
-
 
 void
 md_assemble (str)
-     char * str;
+     char *str;
 {
-#ifdef CGEN_INT_INSN
-  cgen_insn_t              buffer [CGEN_MAX_INSN_SIZE / sizeof (cgen_insn_t)];
-  cgen_insn_t              prev_buffer [CGEN_MAX_INSN_SIZE / sizeof (cgen_insn_t)];
-#else
-  char                     buffer [CGEN_MAX_INSN_SIZE];
-  char                     prev_buffer [CGEN_MAX_INSN_SIZE];
-#endif
-  CGEN_FIELDS              fields;
-  const CGEN_INSN *        insn;
-  char *                   errmsg;
-  char *                   str2 = NULL;
-  int                     is_parallel = false;
-  
+  m32r_insn insn;
+  char *errmsg;
+  char *str2 = NULL;
+
   /* Initialize GAS's cgen interface for a new instruction.  */
-  cgen_asm_init_parse ();
+  gas_cgen_init_parse ();
 
   /* Look for a parallel instruction seperator.  */
   if ((str2 = strstr (str, "||")) != NULL)
     {
-      char * str3;
-      
-      * str2 = 0; /* Seperate the two instructions.  */
-
-      /* 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)
-       fill_insn (0);
+      assemble_two_insns (str, str2, 1);
+      return;
+    }
 
-      /* Assemble the first instruction.  */
-      prev_insn = CGEN_SYM (assemble_insn) (str, & prev_fields, prev_buffer,
-                                           & errmsg);
-      if (! prev_insn)
-       {
-         as_bad (errmsg);
-         return;
-       }
+  /* Also look for a sequential instruction seperator.  */
+  if ((str2 = strstr (str, "->")) != NULL)
+    {
+      assemble_two_insns (str, str2, 0);
+      return;
+    }
 
-/* start-sanitize-m32rx */
-      /* Check to see if this is an allowable parallel insn.  */
-      if (CGEN_INSN_ATTR (prev_insn, CGEN_INSN_PIPE) == PIPE_NONE)
-       {
-         as_bad ("instruction '%s' cannot be executed in parallel.", str);
-         return;
-       }
-       
-      if (! enable_m32rx && 
-         CGEN_INSN_ATTR (prev_insn, CGEN_INSN_MACH) == MACH_M32RX)
-       {
-         as_bad ("instruction '%s' is for the M32RX only", str);
-         return;
-       }
-/* end-sanitize-m32rx */
-  
-      /* fixups = fixups->next; */
+  insn.debug_sym_link = debug_sym_link;
+  debug_sym_link = (sym_linkS *) 0;
 
-      *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.  */
+  insn.insn = m32r_cgen_assemble_insn
+    (gas_cgen_cpu_desc, str, &insn.fields, insn.buffer, & errmsg);
 
-      is_parallel = true;
-    }
-  
-  insn = CGEN_SYM (assemble_insn) (str, & fields, buffer, & errmsg);
-  if (!insn)
+  if (!insn.insn)
     {
       as_bad (errmsg);
       return;
     }
 
-/* start-sanitize-m32rx */
-  if (! enable_m32rx && CGEN_INSN_ATTR (insn, CGEN_INSN_MACH) == (1 << MACH_M32RX))
+  if (! enable_special
+      && CGEN_INSN_ATTR_VALUE (insn.insn, CGEN_INSN_SPECIAL))
     {
-      as_bad ("instruction '%s' is for the M32RX only", str);
+      /* xgettext:c-format  */
+      as_bad (_("unknown instruction '%s'"), str);
       return;
     }
-/* end-sanitize-m32rx */
-  
-  if (is_parallel)
+  else if (! enable_m32rx
+          && CGEN_INSN_ATTR_VALUE (insn.insn, CGEN_INSN_MACH) == (1 << MACH_M32RX))
     {
-      int swap = false;
-      
-/* start-sanitize-m32rx */
-      if (! enable_m32rx)
-       {
-         if (strcmp (prev_insn->name, "nop") != 0
-             && strcmp (insn->name, "nop") != 0)
-           {
-             as_bad ("'%s': only the NOP instruction can be issued in parallel on the m32r", str2);
-             return;
-           }
-       }
-
-      /* We assume that if the first instruction writes to a register that is
-        read by the second instruction it is because the programmer intended
-        this to happen, (after all they have explicitly requested that these
-        two instructions be executed in parallel).  So we do not generate an
-        error if this happens.  */
-      if (can_make_parallel (prev_insn, & prev_fields, insn,
-                            & fields, false, false) != NULL)
-       {
-         if ((errmsg = (char *) can_make_parallel (insn, & fields, prev_insn,
-                                                   & prev_fields, false, false)) == NULL)
-           {
-             /* swap the two insns.  */
-             swap = true;
-           }
-         else
-           {
-             as_bad ("'%s': %s", str2, errmsg);
-             return;
-           }
-       }
-/* end-sanitize-m32rx */
-      
-      /* Generate the parallel instructions */
-      if (swap)
-       {
-         cgen_asm_finish_insn (insn, buffer, CGEN_FIELDS_BITSIZE (& fields));
-         
-         /* Force the top bit of the second insn to be set.  */
-         make_parallel (prev_insn, prev_buffer);
-
-         cgen_asm_finish_insn (prev_insn, prev_buffer,
-                               CGEN_FIELDS_BITSIZE (& prev_fields));
-       }
-      else
-       {
-         cgen_asm_finish_insn (prev_insn, prev_buffer,
-                               CGEN_FIELDS_BITSIZE (& prev_fields));
-
-         /* Force the top bit of the second insn to be set.  */
-         make_parallel (insn, buffer);
-      
-         cgen_asm_finish_insn (insn, buffer, CGEN_FIELDS_BITSIZE (& fields));
-       }
-      
-      /* Clear the prev_insn variable, since it only used if the insn was the first
-        16 bit insn in a 32 bit word.  */
-      prev_insn = NULL;
+      /* xgettext:c-format  */
+      as_bad (_("instruction '%s' is for the M32RX only"), str);
+      return;
     }
-  else if (CGEN_INSN_BITSIZE (insn) == 32)
+
+  if (CGEN_INSN_BITSIZE (insn.insn) == 32)
     {
       /* 32 bit insns must live on 32 bit boundaries.  */
-      if (prev_insn || seen_relaxable_p)
+      if (prev_insn.insn || seen_relaxable_p)
        {
-         /* FIXME: If calling fill_insn too many times turns us into a memory
-            pig, can we call assemble_nop instead of !seen_relaxable_p?  */
+         /* ??? If calling fill_insn too many times turns us into a memory
+            pig, can we call a fn to assemble a nop instead of
+            !seen_relaxable_p?  */
          fill_insn (0);
        }
-      
-      cgen_asm_finish_insn (insn, buffer, CGEN_FIELDS_BITSIZE (& fields));
+
+      expand_debug_syms (insn.debug_sym_link, 2);
+
+      /* 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);
     }
   else
     {
-      /* Keep track of whether we've seen a pair of 16 bit insns.
-        PREV_INSN is NULL when we're on a 32 bit boundary.  */
-      if (prev_insn)
+      int on_32bit_boundary_p;
+      int swap = false;
+
+      if (CGEN_INSN_BITSIZE (insn.insn) != 16)
+       abort ();
+
+      insn.orig_insn = insn.insn;
+
+      /* 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)
        {
-/* start-sanitize-m32rx */
-         if (can_make_parallel (prev_insn, & prev_fields, insn, & fields, false, true) == NULL)
-           make_parallel (insn, buffer);
-         else if (can_make_parallel (insn, & fields, prev_insn, & prev_fields, true, false) == NULL)
-           {
-             /* Swap instructions and make parallel.  */
-             /* XXX TODO .... */
-           }
-/* end-sanitize-m32rx */
-         
-         prev_insn = NULL;
+         /* Get the indices of the operands of the instruction.
+            FIXME: See assemble_parallel for notes on orig_insn.  */
+         {
+           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"));
        }
-      else
+
+      /* 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;
+
+      /* 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
+        program control, then it cannot be combined with the current
+        instruction.  If the current instruction is relaxable, then it
+        might be replaced with a longer version, so we cannot combine it.
+        Also if the output of the previous instruction is used as an
+        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_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)
+           make_parallel (insn.buffer);
+         else if (can_make_parallel (&insn, &prev_insn) == NULL)
+           swap = true;
+       }
+
+      expand_debug_syms (insn.debug_sym_link, 1);
+
+      {
+       int i;
+       finished_insnS fi;
+
+       /* Ensure each pair of 16 bit insns is in the same frag.  */
+       frag_grow (4);
+
+       gas_cgen_finish_insn (insn.orig_insn, insn.buffer,
+                             CGEN_FIELDS_BITSIZE (&insn.fields),
+                             1 /* relax_p  */, &fi);
+       insn.addr = fi.addr;
+       insn.frag = fi.frag;
+       insn.num_fixups = fi.num_fixups;
+       for (i = 0; i < fi.num_fixups; ++i)
+         insn.fixups[i] = fi.fixups[i];
+      }
+
+      if (swap)
        {
-         prev_insn   = insn;
-         prev_fields = fields;
+         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]);
+
+         target_make_parallel (insn.addr);
+
+         /* Swap any relaxable frags recorded for the two insns.  */
+         /* FIXME: Clarify.  relaxation precludes parallel insns */
+         if (prev_insn.frag->fr_opcode == prev_insn.addr)
+           prev_insn.frag->fr_opcode = insn.addr;
+         else if (insn.frag->fr_opcode == insn.addr)
+           insn.frag->fr_opcode = prev_insn.addr;
+
+         /* Update the addresses in any fixups.
+            Note that we don't have to handle the case where each insn is in
+            a different frag as we ensure they're in the same frag above.  */
+         for (i = 0; i < prev_insn.num_fixups; ++i)
+           prev_insn.fixups[i]->fx_where += 2;
+         for (i = 0; i < insn.num_fixups; ++i)
+           insn.fixups[i]->fx_where -= 2;
        }
-      
-      cgen_asm_finish_insn (insn, buffer, CGEN_FIELDS_BITSIZE (& fields));
+
+      /* 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.  */
+      if (on_32bit_boundary_p)
+       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 (prev_insn
-         && CGEN_INSN_ATTR (insn, CGEN_INSN_FILL_SLOT) != 0)
+      if (on_32bit_boundary_p
+         && 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, CGEN_INSN_RELAXABLE) != 0)
+        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_VALUE (insn.orig_insn, CGEN_INSN_RELAXABLE) != 0)
        seen_relaxable_p = 1;
     }
 
@@ -721,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);
@@ -743,7 +1216,7 @@ md_section_align (segment, size)
 
 symbolS *
 md_undefined_symbol (name)
-  char * name;
+     char *name;
 {
   return 0;
 }
@@ -751,65 +1224,68 @@ 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.");
+      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)
     {
-      as_warn (".SCOMMon length (%ld.) <0! Ignored.", (long) size);
+      /* 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)
        {
-         as_warn ("ignoring bad alignment");
+         as_warn (_("ignoring bad alignment"));
          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)
        {
-         as_bad ("Common alignment not a power of 2");
+         as_bad (_("Common alignment not a power of 2"));
          ignore_rest_of_line ();
          return;
        }
@@ -817,13 +1293,14 @@ 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))
     {
-      as_bad ("Ignoring attempt to re-define symbol `%s'.",
+      /* xgettext:c-format  */
+      as_bad (_("Ignoring attempt to re-define symbol `%s'."),
              S_GET_NAME (symbolP));
       ignore_rest_of_line ();
       return;
@@ -831,7 +1308,8 @@ m32r_scomm (ignore)
 
   if (S_GET_VALUE (symbolP) && S_GET_VALUE (symbolP) != (valueT) size)
     {
-      as_bad ("Length of .scomm \"%s\" is already %ld. Not changed to %ld.",
+      /* 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),
              (long) size);
@@ -840,25 +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);
@@ -906,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;
@@ -933,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)
@@ -954,17 +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;
-  char * opcode = fragP->fr_opcode;
-
   /* 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.
@@ -973,14 +1450,17 @@ 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.  */
-      opcode[0] |= 0x80;
+      fragP->fr_opcode[0] |= 0x80;
 
       /* Increase known (fixed) size of fragment.  */
       fragP->fr_fix += 2;
@@ -988,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.
-                 cgen_md_apply_fix3 can't handle it.  */
+                 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.
@@ -1009,21 +1490,22 @@ 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)
          abort ();
+
        fragP->fr_cgen.insn = insn;
        return 2;
       }
 #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.
@@ -1032,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;
 
@@ -1050,38 +1532,37 @@ 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.");
+       as_warn (_("Addend to unresolved symbol not on word boundary."));
       addend = fragP->fr_offset >> 2;
     }
   else
     {
       /* 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;
     }
 
@@ -1092,19 +1573,21 @@ md_convert_frag (abfd, sec, fragP)
     {
       assert (fragP->fr_subtype != 1);
       assert (fragP->fr_cgen.insn != 0);
-      cgen_record_fixup (fragP,
-                        /* Offset of branch insn in frag.  */
-                        fragP->fr_fix + extension - 4,
-                        fragP->fr_cgen.insn,
-                        4 /*length*/,
-                        /* FIXME: quick hack */
+      gas_cgen_record_fixup (fragP,
+                            /* Offset of branch insn in frag.  */
+                            fragP->fr_fix + extension - 4,
+                            fragP->fr_cgen.insn,
+                            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);
+                            fragP->fr_cgen.opinfo,
+                            fragP->fr_symbol, fragP->fr_offset);
     }
 
 #define SIZE_FROM_RELAX_STATE(n) ((n) == 1 ? 1 : 3)
@@ -1122,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)
@@ -1142,66 +1625,40 @@ md_pcrel_from_section (fixP, sec)
    *FIXP may be modified if desired.  */
 
 bfd_reloc_code_real_type
-CGEN_SYM (lookup_reloc) (insn, operand, fixP)
-     const CGEN_INSN *    insn;
-     const CGEN_OPERAND * operand;
-     fixS *               fixP;
+md_cgen_lookup_reloc (insn, operand, 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;
-    }
-  return BFD_RELOC_NONE;
-}
-
-/* Called while parsing an instruction to create a fixup.
-   We need to check for HI16 relocs and queue them up for later sorting.  */
-
-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;
-{
-  fixS * fixP = cgen_record_fixup_exp (frag, where, insn, length,
-                                     operand, opinfo, exp);
-
-  switch (CGEN_OPERAND_TYPE (operand))
-    {
-    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);
+    default:
+      /* Avoid -Wall warning.  */
       break;
     }
-
-  return fixP;
+  return BFD_RELOC_NONE;
 }
 
 /* Record a HI16 reloc for later matching with its LO16 cousin.  */
 
 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);
@@ -1211,14 +1668,46 @@ 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;
 }
 
+/* Called while parsing an instruction to create a fixup.
+   We need to check for HI16 relocs and queue them up for later sorting.  */
+
+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;
+{
+  fixS *fixP = gas_cgen_record_fixup_exp (frag, where, insn, length,
+                                         operand, opinfo, exp);
+
+  switch (operand->type)
+    {
+    case M32R_OPERAND_HI16:
+      /* If low/high/shigh/sda was used, it is recorded in `opinfo'.  */
+      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 */
+      break;
+    }
+
+  return fixP;
+}
+
 /* 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
@@ -1227,12 +1716,12 @@ m32r_record_hi16 (reloc_type, fixP, seg)
 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);
@@ -1252,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)
@@ -1267,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)
@@ -1294,9 +1783,10 @@ m32r_frob_file ()
          if (f != NULL)
            break;
 
-         if (pass == 1)
+         if (pass == 1
+             && warn_unmatched_high)
            as_warn_where (l->fixp->fx_file, l->fixp->fx_line,
-                          "Unmatched high/shigh reloc");
+                          _("Unmatched high/shigh reloc"));
        }
     }
 }
@@ -1307,22 +1797,25 @@ 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)
+    return 1;
+
   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);
@@ -1330,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 *
@@ -1344,12 +1837,11 @@ md_atof (type, litP, sizeP)
      char *litP;
      int *sizeP;
 {
-  int              i;
-  int              prec;
-  LITTLENUM_TYPE   words [MAX_LITTLENUMS];
-  LITTLENUM_TYPE * wordP;
-  char *           t;
-  char *           atof_ieee ();
+  int i;
+  int prec;
+  LITTLENUM_TYPE words[MAX_LITTLENUMS];
+  char *t;
+  char *atof_ieee ();
 
   switch (type)
     {
@@ -1367,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;
-      return "Bad call to md_atof()";
+      *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)
     {
@@ -1397,6 +1890,55 @@ md_atof (type, litP, sizeP)
          litP += sizeof (LITTLENUM_TYPE);
        }
     }
-     
+
   return 0;
 }
+
+void
+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.  */
+  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 (reloc_type == BFD_RELOC_VTABLE_INHERIT
+      || reloc_type == BFD_RELOC_VTABLE_ENTRY)
+    return 0;
+
+  return 1;
+}
This page took 0.049886 seconds and 4 git commands to generate.