* config/tc-m68k.c (parse_mri_control_operand): Change leftstart

	and rightstart to not be const.
	(parse_mri_control_expression): Likewise.
	(build_mri_control_operand): Likewise.  If the left side of the
	comparison is a register, and the right side is not, swap the two
	sides.
	* config/m68k-parse.y (m68k_reg_parse): Make globally visible.
	* config/m68k-parse.h (m68k_reg_parse): Declare.

diff --git a/gas/ChangeLog b/gas/ChangeLog
index c8df738cb2b35cfc0d657b1b2890b819010d8d8f..27d5e78574665be5b5a85c2a06b962e5b8ae931b 100644 (file)
--- a/gas/ChangeLog
+++ b/gas/ChangeLog
@@ -1,5 +1,14 @@
 Mon Sep 18 15:22:28 1995  Ian Lance Taylor  <ian@cygnus.com>
 
+       * config/tc-m68k.c (parse_mri_control_operand): Change leftstart
+       and rightstart to not be const.
+       (parse_mri_control_expression): Likewise.
+       (build_mri_control_operand): Likewise.  If the left side of the
+       comparison is a register, and the right side is not, swap the two
+       sides.
+       * config/m68k-parse.y (m68k_reg_parse): Make globally visible.
+       * config/m68k-parse.h (m68k_reg_parse): Declare.
+
        * read.c (mri_comment_field): New function.
        (mri_comment_end): New function.
        (s_align_bytes): Use mri_comment_field.

diff --git a/gas/config/m68k-parse.h b/gas/config/m68k-parse.h
index 95a1cd82b36c00be703bfdc9a652526795b07585..31132cf735c138890b2d4b8e0293d4f93c75bf88 100644 (file)
--- a/gas/config/m68k-parse.h
+++ b/gas/config/m68k-parse.h
@@ -205,20 +205,6 @@ struct m68k_exp
   expressionS exp;
 };
 
-/* See whether an expression is a signed eight bit value.  */
-
-#define expr8(ex)                              \
-  ((ex)->exp.X_op == O_constant                        \
-   && (ex)->exp.X_add_number >= -0x80          \
-   && (ex)->exp.X_add_number < 0x80)
-
-/* See whether an expression is a signed sixteen bit value.  */
-
-#define expr16(ex)                             \
-  ((ex)->exp.X_op == O_constant                        \
-   && (ex)->exp.X_add_number >= -0x8000                \
-   && (ex)->exp.X_add_number < 0x8000)
-
 /* The operand modes.  */
 
 enum m68k_operand_type
@@ -267,6 +253,10 @@ struct m68k_op
 
 #endif /* ! defined (M68K_PARSE_H) */
 
+/* Parse a register.  */
+
+extern enum m68k_register m68k_reg_parse PARAMS ((char **));
+
 /* The parsing function.  */
 
 extern int m68k_ip_op PARAMS ((char *, struct m68k_op *));

diff --git a/gas/config/m68k-parse.y b/gas/config/m68k-parse.y
index d3d690a858013664e8be05d31c4e605d7d5b025f..dea1a53865010adcbf0e4366d0a0e827d2ebf126 100644 (file)
--- a/gas/config/m68k-parse.y
+++ b/gas/config/m68k-parse.y
@@ -81,7 +81,6 @@
 
 /* Internal functions.  */
 
-static enum m68k_register m68k_reg_parse PARAMS ((char **));
 static int yylex PARAMS (());
 static void yyerror PARAMS ((const char *));
 
@@ -210,12 +209,17 @@ motorola_operand:
                  else
                    op->mode = DISP;
                }
+       | '(' LPC ')'
+               {
+                 op->mode = DISP;
+                 op->reg = $2;
+               }
        | '(' ZAR ')'
                {
                  op->mode = BASE;
                  op->reg = $2;
                }
-       | '(' zpc ')'
+       | '(' LZPC ')'
                {
                  op->mode = BASE;
                  op->reg = $2;
@@ -633,7 +637,7 @@ static char *strorig;
 /* If *CCP could be a register, return the register number and advance
    *CCP.  Otherwise don't change *CCP, and return 0.  */
 
-static enum m68k_register
+enum m68k_register
 m68k_reg_parse (ccp)
      register char **ccp;
 {

diff --git a/gas/config/tc-m68k.c b/gas/config/tc-m68k.c
index e4a1ff896baa01dfb2231b0d6c6412f2ddbcb229..652b54af18a5cd5a9c22a5e8ed24b2749eb76d7f 100644 (file)
--- a/gas/config/tc-m68k.c
+++ b/gas/config/tc-m68k.c
@@ -4708,11 +4708,11 @@ static struct mri_control_info *push_mri_control
 static void pop_mri_control PARAMS ((void));
 static int parse_mri_condition PARAMS ((int *));
 static int parse_mri_control_operand
-  PARAMS ((int *, const char **, const char **, const char **, const char **));
+  PARAMS ((int *, char **, const char **, char **, const char **));
 static int swap_mri_condition PARAMS ((int));
 static int reverse_mri_condition PARAMS ((int));
 static void build_mri_control_operand
-  PARAMS ((int, int, const char *, const char *, const char *, const char *,
+  PARAMS ((int, int, char *, const char *, char *, const char *,
           const char *, const char *, int));
 static void parse_mri_control_expression
   PARAMS ((char *, int, const char *, const char *, int));
@@ -4809,9 +4809,9 @@ parse_mri_condition (pcc)
 static int
 parse_mri_control_operand (pcc, leftstart, leftstop, rightstart, rightstop)
      int *pcc;
-     const char **leftstart;
+     char **leftstart;
      const char **leftstop;
-     const char **rightstart;
+     char **rightstart;
      const char **rightstop;
 {
   char *s;
@@ -4936,30 +4936,53 @@ build_mri_control_operand (qual, cc, leftstart, leftstop, rightstart,
                           rightstop, truelab, falselab, extent)
      int qual;
      int cc;
-     const char *leftstart;
+     char *leftstart;
      const char *leftstop;
-     const char *rightstart;
+     char *rightstart;
      const char *rightstop;
      const char *truelab;
      const char *falselab;
      int extent;
 {
+  int leftreg, rightreg;
   char *buf;
   char *s;
 
-  /* The 68k can't do a general comparision with an immediate operand
-     on the right hand side.  */
-  if (rightstart != NULL && *rightstart == '#')
+  /* See which sides of the comparison are plain registers.  */
+  if (leftstart == NULL)
+    leftreg = 0;
+  else
+    {
+      char *l;
+
+      l = leftstart;
+      leftreg = m68k_reg_parse (&l) != 0;
+    }
+  if (rightstart == NULL)
+    rightreg = 0;
+  else
+    {
+      char *l;
+
+      l = rightstart;
+      rightreg = m68k_reg_parse (&l) != 0;
+    }
+
+  /* Swap the compare operands, if necessary, to produce a legal m68k
+     compare instruction.  */
+  if ((leftreg && ! rightreg)
+      || (rightstart != NULL && *rightstart == '#'))
     {
-      const char *temp;
+      char *temp;
+      const char *tempc;
 
       cc = swap_mri_condition (cc);
       temp = leftstart;
       leftstart = rightstart;
       rightstart = temp;
-      temp = leftstop;
+      tempc = leftstop;
       leftstop = rightstop;
-      rightstop = temp;
+      rightstop = tempc;
     }
 
   if (truelab == NULL)
@@ -5020,9 +5043,9 @@ parse_mri_control_expression (stop, qual, truelab, falselab, extent)
 {
   int c;
   int cc;
-  const char *leftstart;
+  char *leftstart;
   const char *leftstop;
-  const char *rightstart;
+  char *rightstart;
   const char *rightstop;
 
   c = *stop;