]> git.ipfire.org Git - thirdparty/binutils-gdb.git/commitdiff
Revert "gas: have scrubber retain more whitespace"
authorH.J. Lu <hjl.tools@gmail.com>
Mon, 12 Aug 2024 15:43:21 +0000 (08:43 -0700)
committerH.J. Lu <hjl.tools@gmail.com>
Mon, 12 Aug 2024 17:42:02 +0000 (10:42 -0700)
This reverts commit 6ae8a30d44f016cafb46a75843b5109316eb1996.

This fixes PR gas/32073.

26 files changed:
gas/NEWS
gas/app.c
gas/config/tc-aarch64.c
gas/config/tc-arm.c
gas/config/tc-crx.c
gas/config/tc-csky.c
gas/config/tc-pru.c
gas/config/tc-sparc.c
gas/config/tc-v850.c
gas/testsuite/gas/all/macro.l
gas/testsuite/gas/all/macro.s
gas/testsuite/gas/i386/x86-64-apx-nf.s
opcodes/cgen-asm.in
opcodes/epiphany-asm.c
opcodes/fr30-asm.c
opcodes/frv-asm.c
opcodes/ip2k-asm.c
opcodes/iq2000-asm.c
opcodes/lm32-asm.c
opcodes/m32c-asm.c
opcodes/m32r-asm.c
opcodes/mep-asm.c
opcodes/mt-asm.c
opcodes/nds32-asm.c
opcodes/or1k-asm.c
opcodes/xstormy16-asm.c

index 66f8cc5cc8fff4716e3a816d8594a44cee175bff..69c6317c486aa9dfa8e0187222e198cb58fac164 100644 (file)
--- a/gas/NEWS
+++ b/gas/NEWS
@@ -2,13 +2,6 @@
 
 * Add support for RISC-V CORE-V extension (XCvBitmanip) with version 1.0.
 
-* The scrubber (pre-processor) now leaves in place more whitespace, to permit
-  various constructs not fitting the basic "insn opnd1,opnd2[,...]" scheme to
-  work.  This, however, means that macro invocations like "m 1 + 2", i.e. not
-  using double quotes or parentheses around the (apparently) sole argument,
-  will now be treated as passing three arguments.  Such lack of quotation /
-  parenthesization was never reliable to use.
-
 Changes in 2.43:
 
 * Add support for LoongArch .option for fine-grained control of assembly
index b88b4c96137e427e99f1f2fa9511e25320745295..41ba4163a2aa84cbaf17679cc54573f04fd6b935 100644 (file)
--- a/gas/app.c
+++ b/gas/app.c
@@ -467,18 +467,16 @@ do_scrub_chars (size_t (*get) (char *, size_t), char *tostart, size_t tolen,
 
   /*State 0: beginning of normal line
          1: After first whitespace on line (flush more white)
-         2: After first non-white (opcode or maybe label when they're followed
-            by colons) on line (keep 1white)
-         3: after subsequent white on line (typically into operands)
-            (flush more white)
+         2: After first non-white (opcode) on line (keep 1white)
+         3: after second white on line (into operands) (flush white)
          4: after putting out a .linefile, put out digits
          5: parsing a string, then go to old-state
          6: putting out \ escape in a "d string.
          7: no longer used
          8: no longer used
-         9: After seeing non-white in state 3 (keep 1white)
-        10: no longer used
-        11: After seeing a non-white character in state 0 (eg a label definition)
+         9: After seeing symbol char in state 3 (keep 1white after symchar)
+        10: After seeing whitespace in state 9 (keep white before symchar)
+        11: After seeing a symbol character in state 0 (eg a label definition)
         -1: output string in out_string and go to the state in old_state
         12: no longer used
 #ifdef DOUBLEBAR_PARALLEL
@@ -941,11 +939,7 @@ do_scrub_chars (size_t (*get) (char *, size_t), char *tostart, size_t tolen,
                  && (state < 1 || strchr (tc_comment_chars, ch)))
              || IS_NEWLINE (ch)
              || IS_LINE_SEPARATOR (ch)
-             || IS_PARALLEL_SEPARATOR (ch)
-             /* See comma related comment near the bottom of the function.
-                Reasoning equally applies to whitespace preceding a comma in
-                most cases.  */
-             || (ch == ',' && state > 2 && state != 11))
+             || IS_PARALLEL_SEPARATOR (ch))
            {
              if (scrub_m68k_mri)
                {
@@ -988,7 +982,6 @@ do_scrub_chars (size_t (*get) (char *, size_t), char *tostart, size_t tolen,
                 character at the beginning of a line.  */
              goto recycle;
            case 2:
-           case 9:
              state = 3;
              if (to + 1 < toend)
                {
@@ -1012,6 +1005,20 @@ do_scrub_chars (size_t (*get) (char *, size_t), char *tostart, size_t tolen,
                  break;
                }
              goto recycle;     /* Sp in operands */
+           case 9:
+           case 10:
+#ifndef TC_KEEP_OPERAND_SPACES
+             if (scrub_m68k_mri)
+#endif
+               {
+                 /* In MRI mode, we keep these spaces.  */
+                 state = 3;
+                 UNGET (ch);
+                 PUT (' ');
+                 break;
+               }
+             state = 10;       /* Sp after symbol char */
+             goto recycle;
            case 11:
              if (LABELS_WITHOUT_COLONS || flag_m68k_mri)
                state = 1;
@@ -1082,17 +1089,27 @@ do_scrub_chars (size_t (*get) (char *, size_t), char *tostart, size_t tolen,
            {
              if (ch2 != EOF)
                UNGET (ch2);
-             if (state == 1)
-               state = 2;
-             else if (state == 3)
-               state = 9;
+             if (state == 9 || state == 10)
+               state = 3;
              PUT (ch);
            }
          break;
 
        case LEX_IS_STRINGQUOTE:
          quotechar = ch;
-         if (state == 3)
+         if (state == 10)
+           {
+             /* Preserve the whitespace in foo "bar".  */
+             UNGET (ch);
+             state = 3;
+             PUT (' ');
+
+             /* PUT didn't jump out.  We could just break, but we
+                know what will happen, so optimize a bit.  */
+             ch = GET ();
+             old_state = 9;
+           }
+         else if (state == 3)
            old_state = 9;
          else if (state == 0)
            old_state = 11; /* Now seeing label definition.  */
@@ -1113,6 +1130,14 @@ do_scrub_chars (size_t (*get) (char *, size_t), char *tostart, size_t tolen,
              UNGET (c);
            }
 #endif
+         if (state == 10)
+           {
+             /* Preserve the whitespace in foo 'b'.  */
+             UNGET (ch);
+             state = 3;
+             PUT (' ');
+             break;
+           }
          ch = GET ();
          if (ch == EOF)
            {
@@ -1147,7 +1172,10 @@ do_scrub_chars (size_t (*get) (char *, size_t), char *tostart, size_t tolen,
              PUT (out_buf[0]);
              break;
            }
-         old_state = state;
+         if (state == 9)
+           old_state = 3;
+         else
+           old_state = state;
          state = -1;
          out_string = out_buf;
          PUT (*out_string++);
@@ -1157,10 +1185,10 @@ do_scrub_chars (size_t (*get) (char *, size_t), char *tostart, size_t tolen,
 #ifdef KEEP_WHITE_AROUND_COLON
          state = 9;
 #else
-         if (state == 2 || state == 11)
+         if (state == 9 || state == 10)
+           state = 3;
+         else if (state != 3)
            state = 1;
-         else
-           state = 9;
 #endif
          PUT (ch);
          break;
@@ -1285,6 +1313,20 @@ do_scrub_chars (size_t (*get) (char *, size_t), char *tostart, size_t tolen,
              break;
            }
 
+#ifdef TC_D10V
+         /* All insns end in a char for which LEX_IS_SYMBOL_COMPONENT is true.
+            Trap is the only short insn that has a first operand that is
+            neither register nor label.
+            We must prevent exef0f ||trap #1 to degenerate to exef0f ||trap#1 .
+            We can't make '#' LEX_IS_SYMBOL_COMPONENT because it is
+            already LEX_IS_LINE_COMMENT_START.  However, it is the
+            only character in line_comment_chars for d10v, hence we
+            can recognize it as such.  */
+         /* An alternative approach would be to reset the state to 1 when
+            we see '||', '<'- or '->', but that seems to be overkill.  */
+         if (state == 10)
+           PUT (' ');
+#endif
          /* We have a line comment character which is not at the
             start of a line.  If this is also a normal comment
             character, fall through.  Otherwise treat it as a default
@@ -1348,6 +1390,17 @@ do_scrub_chars (size_t (*get) (char *, size_t), char *tostart, size_t tolen,
          /* Fall through.  */
 
        case LEX_IS_SYMBOL_COMPONENT:
+         if (state == 10)
+           {
+             /* This is a symbol character following another symbol
+                character, with whitespace in between.  We skipped
+                the whitespace earlier, so output it now.  */
+             UNGET (ch);
+             state = 3;
+             PUT (' ');
+             break;
+           }
+
 #ifdef TC_Z80
          /* "af'" is a symbol containing '\''.  */
          if (state == 3 && (ch == 'a' || ch == 'A'))
@@ -1373,16 +1426,7 @@ do_scrub_chars (size_t (*get) (char *, size_t), char *tostart, size_t tolen,
                }
            }
 #endif
-
-         /* Fall through.  */
-       default:
-       de_fault:
-         /* Some relatively `normal' character.  */
-         if (state == 0)
-           state = 11; /* Now seeing label definition.  */
-         else if (state == 1)
-           state = 2;  /* Ditto.  */
-         else if (state == 3)
+         if (state == 3)
            state = 9;
 
          /* This is a common case.  Quickly copy CH and all the
@@ -1391,10 +1435,6 @@ do_scrub_chars (size_t (*get) (char *, size_t), char *tostart, size_t tolen,
              && mri_state == NULL
 #if defined TC_ARM && defined OBJ_ELF
              && symver_state == NULL
-#endif
-#ifdef TC_Z80
-             /* See comma related comment below.  */
-             && ch != ','
 #endif
              )
            {
@@ -1410,12 +1450,6 @@ do_scrub_chars (size_t (*get) (char *, size_t), char *tostart, size_t tolen,
                  if (type != 0
                      && type != LEX_IS_SYMBOL_COMPONENT)
                    break;
-#ifdef TC_Z80
-                 /* Need to split at commas, to be able to enter state 16
-                    when needed.  */
-                 if (ch2 == ',')
-                   break;
-#endif
                }
 
              if (s > from)
@@ -1440,15 +1474,52 @@ do_scrub_chars (size_t (*get) (char *, size_t), char *tostart, size_t tolen,
                }
            }
 
-         /* As a special case, to limit the delta to previous behavior, e.g.
-            also affecting listings, go straight to state 3 when seeing a
-            comma. Commas are special: While they can be used to separate
-            macro parameters or arguments, they cannot (on their own, i.e.
-            without quoting) be arguments (or parameter default values).
-            Hence successive whitespace is not meaningful there.  */
-         if (ch == ',' && state == 9)
-           state = 3;
+         /* Fall through.  */
+       default:
+       de_fault:
+         /* Some relatively `normal' character.  */
+         if (state == 0)
+           {
+             state = 11;       /* Now seeing label definition.  */
+           }
+         else if (state == 1)
+           {
+             state = 2;        /* Ditto.  */
+           }
+         else if (state == 9)
+           {
+             if (!IS_SYMBOL_COMPONENT (ch))
+               state = 3;
+           }
+         else if (state == 10)
+           {
+             if (ch == '\\')
+               {
+                 /* Special handling for backslash: a backslash may
+                    be the beginning of a formal parameter (of a
+                    macro) following another symbol character, with
+                    whitespace in between.  If that is the case, we
+                    output a space before the parameter.  Strictly
+                    speaking, correct handling depends upon what the
+                    macro parameter expands into; if the parameter
+                    expands into something which does not start with
+                    an operand character, then we don't want to keep
+                    the space.  We don't have enough information to
+                    make the right choice, so here we are making the
+                    choice which is more likely to be correct.  */
+                 if (to + 1 >= toend)
+                   {
+                     /* If we're near the end of the buffer, save the
+                        character for the next time round.  Otherwise
+                        we'll lose our state.  */
+                     UNGET (ch);
+                     goto tofull;
+                   }
+                 *to++ = ' ';
+               }
 
+             state = 3;
+           }
          PUT (ch);
          break;
        }
index c6ac66bcc5cd128659bd12076326d525b603f569..e94a0cff406aaaf1800979a27991ccbb7e92e917 100644 (file)
@@ -643,7 +643,6 @@ const char FLT_CHARS[] = "rRsSfFdDxXeEpPhHb";
 static inline bool
 skip_past_char (char **str, char c)
 {
-  skip_whitespace (*str);
   if (**str == c)
     {
       (*str)++;
@@ -894,7 +893,6 @@ parse_reg (char **ccp)
   start++;
 #endif
 
-  skip_whitespace (start);
   p = start;
   if (!ISALPHA (*p) || !is_name_beginner (*p))
     return NULL;
@@ -1204,17 +1202,13 @@ parse_typed_reg (char **ccp, aarch64_reg_type type,
                 struct vector_type_el *typeinfo, unsigned int flags)
 {
   char *str = *ccp;
-  bool is_alpha;
-  const reg_entry *reg;
+  bool is_alpha = ISALPHA (*str);
+  const reg_entry *reg = parse_reg (&str);
   struct vector_type_el atype;
   struct vector_type_el parsetype;
   bool is_typed_vecreg = false;
   unsigned int err_flags = (flags & PTR_IN_REGLIST) ? SEF_IN_REGLIST : 0;
 
-  skip_whitespace (str);
-  is_alpha = ISALPHA (*str);
-  reg = parse_reg (&str);
-
   atype.defined = 0;
   atype.type = NT_invtype;
   atype.width = -1;
@@ -1435,7 +1429,10 @@ parse_vector_reg_list (char **ccp, aarch64_reg_type type,
   do
     {
       if (in_range)
-       val_range = val;
+       {
+         str++;                /* skip over '-' */
+         val_range = val;
+       }
 
       const reg_entry *reg;
       if (has_qualifier)
@@ -1497,8 +1494,7 @@ parse_vector_reg_list (char **ccp, aarch64_reg_type type,
       in_range = 0;
       ptr_flags |= PTR_GOOD_MATCH;
     }
-  while (skip_past_comma (&str)
-        || (in_range = 1, skip_past_char (&str, '-')));
+  while (skip_past_comma (&str) || (in_range = 1, *str == '-'));
 
   skip_whitespace (str);
   if (*str != '}')
@@ -8293,7 +8289,6 @@ parse_operands (char *str, const aarch64_opcode *opcode)
     }
 
   /* Check if we have parsed all the operands.  */
-  skip_whitespace (str);
   if (*str != '\0' && ! error_p ())
     {
       /* Set I to the index of the last present operand; this is
index 14d729558c5648bcfb6559752cbac325a6b20508..540ab4873843a49214d85642b59520ef7a7dd971 100644 (file)
@@ -1148,8 +1148,6 @@ my_get_expression (expressionS * ep, char ** str, int prefix_mode)
     prefix_mode = (prefix_mode == GE_OPT_PREFIX_BIG) ? prefix_mode
                  : GE_OPT_PREFIX;
 
-  skip_whitespace (*str);
-
   switch (prefix_mode)
     {
     case GE_NO_PREFIX: break;
index 4673c947f2fec747e57eeab07fa0ac79beead9cd..3a47354c4737d42afb6dc5c9f2afdb9b7d6aaad1 100644 (file)
@@ -1723,12 +1723,8 @@ preprocess_reglist (char *param, int *allocated)
 
   while (*paramP != '}')
     {
-      memset (&reg_name, '\0', sizeof (reg_name));
-
-      while (ISSPACE (*paramP))
-       paramP++;
-
       regP = paramP;
+      memset (&reg_name, '\0', sizeof (reg_name));
 
       while (ISALNUM (*paramP))
        paramP++;
index 0eb1805aacf2f6018fcd27d6911826413647ecca..0dc8bfd45dc1c6e01ec28c11dd7d0494a9a6a1d2 100644 (file)
@@ -2409,18 +2409,10 @@ parse_rt (char *s,
     /* Indicate nothing there.  */
     ep->X_op = O_absent;
 
-  /* Skip whitespace.  */
-  while (ISSPACE (*s))
-    ++s;
-
   if (*s == '[')
     {
       s = parse_exp (s + 1, &e);
 
-      /* Skip whitespace.  */
-      while (ISSPACE (*s))
-       ++s;
-
       if (*s == ']')
        s++;
       else
@@ -2943,11 +2935,6 @@ is_reg_lshift_illegal (char **oper, int is_float)
     }
 
   *oper += len;
-
-  /* Skip whitespace.  */
-  while (ISSPACE (**oper))
-    ++*oper;
-
   if ((*oper)[0] != '<' || (*oper)[1] != '<')
     {
       SET_ERROR_STRING (ERROR_UNDEFINE,
@@ -3474,9 +3461,6 @@ get_operand_value (struct csky_opcode_info *op,
          return false;
        }
 
-      while (ISSPACE (**oper))
-       ++*oper;
-
       if (!get_operand_value (op, oper, &soprnd->subs[0]))
        {
          *s = rc;
@@ -3497,7 +3481,7 @@ get_operand_value (struct csky_opcode_info *op,
        }
 
       *s = rc;
-      *oper = s + 1;
+      *oper += 1;
       return true;
     }
 
@@ -4293,16 +4277,11 @@ get_operand_value (struct csky_opcode_info *op,
     case OPRND_TYPE_VREG_WITH_INDEX:
       if (parse_type_freg (oper, 0))
        {
-         /* Skip whitespace.  */
-         while (ISSPACE (**oper))
-           ++*oper;
          if (**oper == '[')
            {
              (*oper)++;
              if (is_imm_within_range (oper, 0, 0xf))
                {
-                 while (ISSPACE (**oper))
-                   ++*oper;
                  if (**oper == ']')
                    {
                      unsigned int idx = --csky_insn.idx;
index 6789686718aedbf0f805c9a382d14a5d9036d1ec..99a3c1ef88ce23189a259dfbfbea5605290b726c 100644 (file)
@@ -1399,6 +1399,7 @@ pru_parse_args (pru_insn_infoS *insn ATTRIBUTE_UNUSED, char *argstr,
                  const char *parsestr, char **parsed_args)
 {
   char *p;
+  char *end = NULL;
   int i;
   p = argstr;
   i = 0;
@@ -1425,7 +1426,14 @@ pru_parse_args (pru_insn_infoS *insn ATTRIBUTE_UNUSED, char *argstr,
       else
        {
          /* Check that the argument string has no trailing arguments.  */
-         if (strpbrk (p, ",") != NULL)
+         /* If we've got a %pmem relocation, we've zapped the parens with
+            spaces.  */
+         if (strprefix (p, "%pmem") || strprefix (p, "%label"))
+           end = strpbrk (p, ",");
+         else
+           end = strpbrk (p, " ,");
+
+         if (end != NULL)
            as_bad (_("too many arguments"));
        }
 
index d78aa0e6e3dae46b0ad4e4f07801e7b933cec20a..e37189e7c5ed13de6b3b299d1d61d1ab74983bcd 100644 (file)
@@ -1778,9 +1778,6 @@ sparc_ip (char *str, const struct sparc_opcode **pinsn)
          operands match.  */
       for (args = insn->args;; ++args)
        {
-         if (*s == ' ' && *args != ' ')
-           ++s;
-
          switch (*args)
            {
            case 'K':
@@ -2721,6 +2718,11 @@ sparc_ip (char *str, const struct sparc_opcode **pinsn)
                   'symbols' in the input string.  Try not to create U
                   symbols for registers, etc.  */
 
+               /* This stuff checks to see if the expression ends in
+                  +%reg.  If it does, it removes the register from
+                  the expression, and re-sets 's' to point to the
+                  right place.  */
+
                if (op_arg)
                  {
                    int npar = 0;
@@ -2750,8 +2752,6 @@ sparc_ip (char *str, const struct sparc_opcode **pinsn)
                        return special_case;
                      }
                    s = s1 + 1;
-                   if (*s == ' ')
-                     s++;
                    if (*s == ',' || *s == ']' || !*s)
                      continue;
                    if (*s != '+' && *s != '-')
@@ -2765,45 +2765,17 @@ sparc_ip (char *str, const struct sparc_opcode **pinsn)
                    memset (&the_insn.exp, 0, sizeof (the_insn.exp));
                  }
 
-               /* This stuff checks to see if the expression ends in
-                  +%reg.  If it does, it removes the register from
-                  the expression, and re-sets 's' to point to the
-                  right place.  */
-
                for (s1 = s; *s1 && *s1 != ',' && *s1 != ']'; s1++)
                  ;
 
-               if (s1 != s && s1[-1] == ' ')
-                 --s1;
                if (s1 != s && ISDIGIT (s1[-1]))
                  {
                    if (s1[-2] == '%' && s1[-3] == '+')
-                     {
-                       if (s1[-3] == '+')
-                         s1 -= 3;
-                       else if (s1[-3] == ' ' && s1[-4] == '+')
-                         s1 -= 4;
-                       else
-                         s1 = NULL;
-                     }
-                   else if (strchr ("golir0123456789", s1[-2]) && s1[-3] == '%')
-                     {
-                       if (s1[-4] == '+')
-                         s1 -= 4;
-                       else if (s1[-4] == ' ' && s1[-5] == '+')
-                         s1 -= 5;
-                       else
-                         s1 = NULL;
-                     }
-                   else if (s1[-3] == 'r' && s1[-4] == '%')
-                     {
-                       if (s1[-5] == '+')
-                         s1 -= 5;
-                       else if (s1[-5] == ' ' && s1[-6] == '+')
-                         s1 -= 6;
-                       else
-                         s1 = NULL;
-                     }
+                     s1 -= 3;
+                   else if (strchr ("golir0123456789", s1[-2]) && s1[-3] == '%' && s1[-4] == '+')
+                     s1 -= 4;
+                   else if (s1[-3] == 'r' && s1[-4] == '%' && s1[-5] == '+')
+                     s1 -= 5;
                    else
                      s1 = NULL;
                    if (s1)
index 63b69a7e1f9f865e8b15d4d7e10e613a34794ace..cb518a7da05694c2421efdf455184fcf5a90e51e 100644 (file)
@@ -1456,8 +1456,6 @@ parse_register_list (unsigned long *insn,
            }
        }
 
-      skip_white_space ();
-
       if (*input_line_pointer == '}')
        {
          input_line_pointer++;
@@ -1477,8 +1475,6 @@ parse_register_list (unsigned long *insn,
          /* Skip the dash.  */
          ++input_line_pointer;
 
-         skip_white_space ();
-
          /* Get the second register in the range.  */
          if (! register_name (&exp2))
            {
index c62d34d2baaf7e3f2e655c157d29de7f5837da26..75fe338f1323d0ebedbc37e1b051dcd618957340 100644 (file)
 [      ]*[1-9][0-9]*[  ]+.... 0+70*[   ]+>  .byte 7
 [      ]*[1-9][0-9]*[  ]+.... 0+80*[   ]+>  .byte 8
 [      ]*[1-9][0-9]*[  ]+m[    ]+""[   ]+""[   ]+""
-[      ]*[1-9][0-9]*[  ]+
-[      ]*[1-9][0-9]*[  ]+m[    ]+1[    ]+\+2
-[      ]*[1-9][0-9]*[  ]+.... 0+10*[   ]+>  .byte 1
-[      ]*[1-9][0-9]*[  ]+.... 0+20*[   ]+>  .byte \+2
-[      ]*[1-9][0-9]*[  ]+m[    ]+\(3\)[        ]+\+4
-[      ]*[1-9][0-9]*[  ]+.... 0+30*[   ]+>  .byte \(3\)
-[      ]*[1-9][0-9]*[  ]+.... 0+40*[   ]+>  .byte \+4
-[      ]*[1-9][0-9]*[  ]+m[    ]+\(5\)[        ]+\(6\)
-[      ]*[1-9][0-9]*[  ]+.... 0+50*[   ]+>  .byte \(5\)
-[      ]*[1-9][0-9]*[  ]+.... 0+60*[   ]+>  .byte \(6\)
 #pass
index 109bcc56c9919a0203196f2a006ecb58360e43c7..9e70f3067b755129ccdbb748b41ee43b9ca52ba6 100644 (file)
@@ -9,8 +9,8 @@
        m "7" "8"
        m "" "" ""
 
+       .if 0
        m 1 +2
        m (3) +4
        m (5) (6)
-
-       .byte -1
+       .endif
index 99ae1c7aaa64ff984baa25f7529b2c35dd1387a7..fbd4cadd9834070678ab06b971d7dc4a153e2a91 100644 (file)
@@ -1390,13 +1390,13 @@ optimize:
        {nf}    \op     $128, %ecx, %edx
        {nf}    \op     $128, %r9
        {nf}    \op     $128, %r9, %r31
-       {nf}    \op\(b) $128, (%rax)
+       {nf}    \op\()b $128, (%rax)
        {nf}    \op     $128, (%rax), %bl
-       {nf}    \op\(w) $128, (%rax)
+       {nf}    \op\()w $128, (%rax)
        {nf}    \op     $128, (%rax), %dx
-       {nf}    \op\(l) $128, (%rax)
+       {nf}    \op\()l $128, (%rax)
        {nf}    \op     $128, (%rax), %ecx
-       {nf}    \op\(q) $128, (%rax)
+       {nf}    \op\()q $128, (%rax)
        {nf}    \op     $128, (%rax), %r9
 
        {nf}    \op     $1, %bl
@@ -1407,13 +1407,13 @@ optimize:
        {nf}    \op     $1, %ecx, %edx
        {nf}    \op     $1, %r9
        {nf}    \op     $1, %r9, %r31
-       {nf}    \op\(b) $1, (%rax)
+       {nf}    \op\()b $1, (%rax)
        {nf}    \op     $1, (%rax), %bl
-       {nf}    \op\(w) $1, (%rax)
+       {nf}    \op\()w $1, (%rax)
        {nf}    \op     $1, (%rax), %dx
-       {nf}    \op\(l) $1, (%rax)
+       {nf}    \op\()l $1, (%rax)
        {nf}    \op     $1, (%rax), %ecx
-       {nf}    \op\(q) $1, (%rax)
+       {nf}    \op\()q $1, (%rax)
        {nf}    \op     $1, (%rax), %r9
 
        {nf}    \op     $0xff, %bl
@@ -1424,13 +1424,13 @@ optimize:
        {nf}    \op     $-1, %ecx, %edx
        {nf}    \op     $-1, %r9
        {nf}    \op     $-1, %r9, %r31
-       {nf}    \op\(b) $0xff, (%rax)
+       {nf}    \op\()b $0xff, (%rax)
        {nf}    \op     $-1, (%rax), %bl
-       {nf}    \op\(w) $0xffff, (%rax)
+       {nf}    \op\()w $0xffff, (%rax)
        {nf}    \op     $-1, (%rax), %dx
-       {nf}    \op\(l) $0xffffffff, (%rax)
+       {nf}    \op\()l $0xffffffff, (%rax)
        {nf}    \op     $-1, (%rax), %ecx
-       {nf}    \op\(q) $-1, (%rax)
+       {nf}    \op\()q $-1, (%rax)
        {nf}    \op     $-1, (%rax), %r9
        .endr
 
@@ -1444,13 +1444,13 @@ optimize:
        {nf}    ro\dir  $63, %rdx
        {nf}    ro\dir  $63, %rdx, %rax
 
-       {nf}    ro\dir\(b)      $7, (%rdx)
+       {nf}    ro\dir\()b      $7, (%rdx)
        {nf}    ro\dir          $7, (%rdx), %al
-       {nf}    ro\dir\(w)      $15, (%rdx)
+       {nf}    ro\dir\()w      $15, (%rdx)
        {nf}    ro\dir          $15, (%rdx), %ax
-       {nf}    ro\dir\(l)      $31, (%rdx)
+       {nf}    ro\dir\()l      $31, (%rdx)
        {nf}    ro\dir          $31, (%rdx), %eax
-       {nf}    ro\dir\(q)      $63, (%rdx)
+       {nf}    ro\dir\()q      $63, (%rdx)
        {nf}    ro\dir          $63, (%rdx), %rax
        .endr
 
@@ -1476,10 +1476,10 @@ optimize:
        # Note: 2-6 want leaving alone with -Os.
        .irp n, 1, 2, 6, 7
        # Note: 16-bit 3-operand src!=dst non-ZU form needs leaving alone.
-       {nf} imul $1<<\n, %\r\(dx), %\r\(cx)
-       {nf} imul $1<<\n, (%rdx), %\r\(cx)
-       {nf} imul $1<<\n, %\r\(cx), %\r\(cx)
-       {nf} imul $1<<\n, %\r\(cx)
+       {nf} imul $1<<\n, %\r\()dx, %\r\()cx
+       {nf} imul $1<<\n, (%rdx), %\r\()cx
+       {nf} imul $1<<\n, %\r\()cx, %\r\()cx
+       {nf} imul $1<<\n, %\r\()cx
 
        .ifeqs "\r",""
        {nf} imulzu $1<<\n, %dx, %cx
index 8aab7e1ab83a12ef25c0a56c1cb576b16087228c..3f5180434df447cbc3184c649446107f44ef68f0 100644 (file)
@@ -68,7 +68,6 @@ char *
   char rxbuf[CGEN_MAX_RX_ELEMENTS];
   char *rx = rxbuf;
   const CGEN_SYNTAX_CHAR_TYPE *syn;
-  char prev_syntax_char = 0;
   int reg_err;
 
   syn = CGEN_SYNTAX_STRING (CGEN_OPCODE_SYNTAX (opc));
@@ -106,15 +105,6 @@ char *
        {
          char c = CGEN_SYNTAX_CHAR (* syn);
 
-         /* See whitespace related comments in parse_insn_normal().  */
-         if (c != ' ' && prev_syntax_char != ' '
-             && (!ISALNUM (c) || !ISALNUM (prev_syntax_char)))
-           {
-             *rx++ = ' ';
-             *rx++ = '*';
-           }
-         prev_syntax_char = c;
-
          switch (c)
            {
              /* Escape any regex metacharacters in the syntax.  */
@@ -148,7 +138,6 @@ char *
          /* Replace non-syntax fields with globs.  */
          *rx++ = '.';
          *rx++ = '*';
-         prev_syntax_char = 0;
        }
     }
 
@@ -206,8 +195,10 @@ parse_insn_normal (CGEN_CPU_DESC cd,
   const char *errmsg;
   const char *p;
   const CGEN_SYNTAX_CHAR_TYPE * syn;
-  char prev_syntax_char = 0;
-  bool past_opcode_p;
+#ifdef CGEN_MNEMONIC_OPERANDS
+  /* FIXME: wip */
+  int past_opcode_p;
+#endif
 
   /* For now we assume the mnemonic is first (there are no leading operands).
      We can parse it without needing to set up operand parsing.
@@ -223,13 +214,13 @@ parse_insn_normal (CGEN_CPU_DESC cd,
 #ifndef CGEN_MNEMONIC_OPERANDS
   if (* str && ! ISSPACE (* str))
     return _("unrecognized instruction");
-  past_opcode_p = true;
-#else
-  past_opcode_p = false;
 #endif
 
   CGEN_INIT_PARSE (cd);
   cgen_init_parse_operand (cd);
+#ifdef CGEN_MNEMONIC_OPERANDS
+  past_opcode_p = 0;
+#endif
 
   /* We don't check for (*str != '\0') here because we want to parse
      any trailing fake arguments in the syntax string.  */
@@ -243,28 +234,18 @@ parse_insn_normal (CGEN_CPU_DESC cd,
 
   while (* syn != 0)
     {
-      char c = CGEN_SYNTAX_CHAR_P (*syn) ? CGEN_SYNTAX_CHAR (*syn) : 0;
-
-      /* FIXME: Despite this check we may still take inappropriate advantage of
-        the fact that GAS's input scrubber will remove extraneous whitespace.
-        We may also be a little too lax with this now, yet being more strict
-        would require targets to indicate where whitespace is permissible.  */
-      if (past_opcode_p && c != ' ' && ISSPACE (*str)
-         /* No whitespace between consecutive alphanumeric syntax elements.  */
-         && (!ISALNUM (c) || !ISALNUM (prev_syntax_char)))
-       ++str;
-      prev_syntax_char = c;
-
       /* Non operand chars must match exactly.  */
-      if (c != 0)
+      if (CGEN_SYNTAX_CHAR_P (* syn))
        {
          /* FIXME: While we allow for non-GAS callers above, we assume the
             first char after the mnemonic part is a space.  */
-         if (TOLOWER (*str) == TOLOWER (c))
+         /* FIXME: We also take inappropriate advantage of the fact that
+            GAS's input scrubber will remove extraneous blanks.  */
+         if (TOLOWER (*str) == TOLOWER (CGEN_SYNTAX_CHAR (* syn)))
            {
 #ifdef CGEN_MNEMONIC_OPERANDS
-             if (c == ' ')
-               past_opcode_p = true;
+             if (CGEN_SYNTAX_CHAR(* syn) == ' ')
+               past_opcode_p = 1;
 #endif
              ++ syn;
              ++ str;
@@ -276,7 +257,7 @@ parse_insn_normal (CGEN_CPU_DESC cd,
 
              /* xgettext:c-format */
              sprintf (msg, _("syntax error (expected char `%c', found `%c')"),
-                      c, *str);
+                      CGEN_SYNTAX_CHAR(*syn), *str);
              return msg;
            }
          else
@@ -286,12 +267,15 @@ parse_insn_normal (CGEN_CPU_DESC cd,
 
              /* xgettext:c-format */
              sprintf (msg, _("syntax error (expected char `%c', found end of instruction)"),
-                      c);
+                      CGEN_SYNTAX_CHAR(*syn));
              return msg;
            }
          continue;
        }
 
+#ifdef CGEN_MNEMONIC_OPERANDS
+      (void) past_opcode_p;
+#endif
       /* We have an operand of some sort.  */
       errmsg = cd->parse_operand (cd, CGEN_SYNTAX_FIELD (*syn), &str, fields);
       if (errmsg)
index ce10f48d6b3c4af0f465fd85245c6d97d662358b..6c3f549705d4c8509ac06d2216fe842713ef6d02 100644 (file)
@@ -499,7 +499,6 @@ epiphany_cgen_build_insn_regex (CGEN_INSN *insn)
   char rxbuf[CGEN_MAX_RX_ELEMENTS];
   char *rx = rxbuf;
   const CGEN_SYNTAX_CHAR_TYPE *syn;
-  char prev_syntax_char = 0;
   int reg_err;
 
   syn = CGEN_SYNTAX_STRING (CGEN_OPCODE_SYNTAX (opc));
@@ -537,15 +536,6 @@ epiphany_cgen_build_insn_regex (CGEN_INSN *insn)
        {
          char c = CGEN_SYNTAX_CHAR (* syn);
 
-         /* See whitespace related comments in parse_insn_normal().  */
-         if (c != ' ' && prev_syntax_char != ' '
-             && (!ISALNUM (c) || !ISALNUM (prev_syntax_char)))
-           {
-             *rx++ = ' ';
-             *rx++ = '*';
-           }
-         prev_syntax_char = c;
-
          switch (c)
            {
              /* Escape any regex metacharacters in the syntax.  */
@@ -579,7 +569,6 @@ epiphany_cgen_build_insn_regex (CGEN_INSN *insn)
          /* Replace non-syntax fields with globs.  */
          *rx++ = '.';
          *rx++ = '*';
-         prev_syntax_char = 0;
        }
     }
 
@@ -637,8 +626,10 @@ parse_insn_normal (CGEN_CPU_DESC cd,
   const char *errmsg;
   const char *p;
   const CGEN_SYNTAX_CHAR_TYPE * syn;
-  char prev_syntax_char = 0;
-  bool past_opcode_p;
+#ifdef CGEN_MNEMONIC_OPERANDS
+  /* FIXME: wip */
+  int past_opcode_p;
+#endif
 
   /* For now we assume the mnemonic is first (there are no leading operands).
      We can parse it without needing to set up operand parsing.
@@ -654,13 +645,13 @@ parse_insn_normal (CGEN_CPU_DESC cd,
 #ifndef CGEN_MNEMONIC_OPERANDS
   if (* str && ! ISSPACE (* str))
     return _("unrecognized instruction");
-  past_opcode_p = true;
-#else
-  past_opcode_p = false;
 #endif
 
   CGEN_INIT_PARSE (cd);
   cgen_init_parse_operand (cd);
+#ifdef CGEN_MNEMONIC_OPERANDS
+  past_opcode_p = 0;
+#endif
 
   /* We don't check for (*str != '\0') here because we want to parse
      any trailing fake arguments in the syntax string.  */
@@ -674,28 +665,18 @@ parse_insn_normal (CGEN_CPU_DESC cd,
 
   while (* syn != 0)
     {
-      char c = CGEN_SYNTAX_CHAR_P (*syn) ? CGEN_SYNTAX_CHAR (*syn) : 0;
-
-      /* FIXME: Despite this check we may still take inappropriate advantage of
-        the fact that GAS's input scrubber will remove extraneous whitespace.
-        We may also be a little too lax with this now, yet being more strict
-        would require targets to indicate where whitespace is permissible.  */
-      if (past_opcode_p && c != ' ' && ISSPACE (*str)
-         /* No whitespace between consecutive alphanumeric syntax elements.  */
-         && (!ISALNUM (c) || !ISALNUM (prev_syntax_char)))
-       ++str;
-      prev_syntax_char = c;
-
       /* Non operand chars must match exactly.  */
-      if (c != 0)
+      if (CGEN_SYNTAX_CHAR_P (* syn))
        {
          /* FIXME: While we allow for non-GAS callers above, we assume the
             first char after the mnemonic part is a space.  */
-         if (TOLOWER (*str) == TOLOWER (c))
+         /* FIXME: We also take inappropriate advantage of the fact that
+            GAS's input scrubber will remove extraneous blanks.  */
+         if (TOLOWER (*str) == TOLOWER (CGEN_SYNTAX_CHAR (* syn)))
            {
 #ifdef CGEN_MNEMONIC_OPERANDS
-             if (c == ' ')
-               past_opcode_p = true;
+             if (CGEN_SYNTAX_CHAR(* syn) == ' ')
+               past_opcode_p = 1;
 #endif
              ++ syn;
              ++ str;
@@ -707,7 +688,7 @@ parse_insn_normal (CGEN_CPU_DESC cd,
 
              /* xgettext:c-format */
              sprintf (msg, _("syntax error (expected char `%c', found `%c')"),
-                      c, *str);
+                      CGEN_SYNTAX_CHAR(*syn), *str);
              return msg;
            }
          else
@@ -717,12 +698,15 @@ parse_insn_normal (CGEN_CPU_DESC cd,
 
              /* xgettext:c-format */
              sprintf (msg, _("syntax error (expected char `%c', found end of instruction)"),
-                      c);
+                      CGEN_SYNTAX_CHAR(*syn));
              return msg;
            }
          continue;
        }
 
+#ifdef CGEN_MNEMONIC_OPERANDS
+      (void) past_opcode_p;
+#endif
       /* We have an operand of some sort.  */
       errmsg = cd->parse_operand (cd, CGEN_SYNTAX_FIELD (*syn), &str, fields);
       if (errmsg)
index 966d36fbe2c9cc6ddfcec91eca3c2a476061aa30..9ef15fed4c24c2a57f1208e999e969d6ab7099a3 100644 (file)
@@ -354,7 +354,6 @@ fr30_cgen_build_insn_regex (CGEN_INSN *insn)
   char rxbuf[CGEN_MAX_RX_ELEMENTS];
   char *rx = rxbuf;
   const CGEN_SYNTAX_CHAR_TYPE *syn;
-  char prev_syntax_char = 0;
   int reg_err;
 
   syn = CGEN_SYNTAX_STRING (CGEN_OPCODE_SYNTAX (opc));
@@ -392,15 +391,6 @@ fr30_cgen_build_insn_regex (CGEN_INSN *insn)
        {
          char c = CGEN_SYNTAX_CHAR (* syn);
 
-         /* See whitespace related comments in parse_insn_normal().  */
-         if (c != ' ' && prev_syntax_char != ' '
-             && (!ISALNUM (c) || !ISALNUM (prev_syntax_char)))
-           {
-             *rx++ = ' ';
-             *rx++ = '*';
-           }
-         prev_syntax_char = c;
-
          switch (c)
            {
              /* Escape any regex metacharacters in the syntax.  */
@@ -434,7 +424,6 @@ fr30_cgen_build_insn_regex (CGEN_INSN *insn)
          /* Replace non-syntax fields with globs.  */
          *rx++ = '.';
          *rx++ = '*';
-         prev_syntax_char = 0;
        }
     }
 
@@ -492,8 +481,10 @@ parse_insn_normal (CGEN_CPU_DESC cd,
   const char *errmsg;
   const char *p;
   const CGEN_SYNTAX_CHAR_TYPE * syn;
-  char prev_syntax_char = 0;
-  bool past_opcode_p;
+#ifdef CGEN_MNEMONIC_OPERANDS
+  /* FIXME: wip */
+  int past_opcode_p;
+#endif
 
   /* For now we assume the mnemonic is first (there are no leading operands).
      We can parse it without needing to set up operand parsing.
@@ -509,13 +500,13 @@ parse_insn_normal (CGEN_CPU_DESC cd,
 #ifndef CGEN_MNEMONIC_OPERANDS
   if (* str && ! ISSPACE (* str))
     return _("unrecognized instruction");
-  past_opcode_p = true;
-#else
-  past_opcode_p = false;
 #endif
 
   CGEN_INIT_PARSE (cd);
   cgen_init_parse_operand (cd);
+#ifdef CGEN_MNEMONIC_OPERANDS
+  past_opcode_p = 0;
+#endif
 
   /* We don't check for (*str != '\0') here because we want to parse
      any trailing fake arguments in the syntax string.  */
@@ -529,28 +520,18 @@ parse_insn_normal (CGEN_CPU_DESC cd,
 
   while (* syn != 0)
     {
-      char c = CGEN_SYNTAX_CHAR_P (*syn) ? CGEN_SYNTAX_CHAR (*syn) : 0;
-
-      /* FIXME: Despite this check we may still take inappropriate advantage of
-        the fact that GAS's input scrubber will remove extraneous whitespace.
-        We may also be a little too lax with this now, yet being more strict
-        would require targets to indicate where whitespace is permissible.  */
-      if (past_opcode_p && c != ' ' && ISSPACE (*str)
-         /* No whitespace between consecutive alphanumeric syntax elements.  */
-         && (!ISALNUM (c) || !ISALNUM (prev_syntax_char)))
-       ++str;
-      prev_syntax_char = c;
-
       /* Non operand chars must match exactly.  */
-      if (c != 0)
+      if (CGEN_SYNTAX_CHAR_P (* syn))
        {
          /* FIXME: While we allow for non-GAS callers above, we assume the
             first char after the mnemonic part is a space.  */
-         if (TOLOWER (*str) == TOLOWER (c))
+         /* FIXME: We also take inappropriate advantage of the fact that
+            GAS's input scrubber will remove extraneous blanks.  */
+         if (TOLOWER (*str) == TOLOWER (CGEN_SYNTAX_CHAR (* syn)))
            {
 #ifdef CGEN_MNEMONIC_OPERANDS
-             if (c == ' ')
-               past_opcode_p = true;
+             if (CGEN_SYNTAX_CHAR(* syn) == ' ')
+               past_opcode_p = 1;
 #endif
              ++ syn;
              ++ str;
@@ -562,7 +543,7 @@ parse_insn_normal (CGEN_CPU_DESC cd,
 
              /* xgettext:c-format */
              sprintf (msg, _("syntax error (expected char `%c', found `%c')"),
-                      c, *str);
+                      CGEN_SYNTAX_CHAR(*syn), *str);
              return msg;
            }
          else
@@ -572,12 +553,15 @@ parse_insn_normal (CGEN_CPU_DESC cd,
 
              /* xgettext:c-format */
              sprintf (msg, _("syntax error (expected char `%c', found end of instruction)"),
-                      c);
+                      CGEN_SYNTAX_CHAR(*syn));
              return msg;
            }
          continue;
        }
 
+#ifdef CGEN_MNEMONIC_OPERANDS
+      (void) past_opcode_p;
+#endif
       /* We have an operand of some sort.  */
       errmsg = cd->parse_operand (cd, CGEN_SYNTAX_FIELD (*syn), &str, fields);
       if (errmsg)
index 9d125345048cee59f5e8b5cd336def6abc1e0be2..6b64bf452992872506f5de2a8734aa93c4fe79fb 100644 (file)
@@ -1307,7 +1307,6 @@ frv_cgen_build_insn_regex (CGEN_INSN *insn)
   char rxbuf[CGEN_MAX_RX_ELEMENTS];
   char *rx = rxbuf;
   const CGEN_SYNTAX_CHAR_TYPE *syn;
-  char prev_syntax_char = 0;
   int reg_err;
 
   syn = CGEN_SYNTAX_STRING (CGEN_OPCODE_SYNTAX (opc));
@@ -1345,15 +1344,6 @@ frv_cgen_build_insn_regex (CGEN_INSN *insn)
        {
          char c = CGEN_SYNTAX_CHAR (* syn);
 
-         /* See whitespace related comments in parse_insn_normal().  */
-         if (c != ' ' && prev_syntax_char != ' '
-             && (!ISALNUM (c) || !ISALNUM (prev_syntax_char)))
-           {
-             *rx++ = ' ';
-             *rx++ = '*';
-           }
-         prev_syntax_char = c;
-
          switch (c)
            {
              /* Escape any regex metacharacters in the syntax.  */
@@ -1387,7 +1377,6 @@ frv_cgen_build_insn_regex (CGEN_INSN *insn)
          /* Replace non-syntax fields with globs.  */
          *rx++ = '.';
          *rx++ = '*';
-         prev_syntax_char = 0;
        }
     }
 
@@ -1445,8 +1434,10 @@ parse_insn_normal (CGEN_CPU_DESC cd,
   const char *errmsg;
   const char *p;
   const CGEN_SYNTAX_CHAR_TYPE * syn;
-  char prev_syntax_char = 0;
-  bool past_opcode_p;
+#ifdef CGEN_MNEMONIC_OPERANDS
+  /* FIXME: wip */
+  int past_opcode_p;
+#endif
 
   /* For now we assume the mnemonic is first (there are no leading operands).
      We can parse it without needing to set up operand parsing.
@@ -1462,13 +1453,13 @@ parse_insn_normal (CGEN_CPU_DESC cd,
 #ifndef CGEN_MNEMONIC_OPERANDS
   if (* str && ! ISSPACE (* str))
     return _("unrecognized instruction");
-  past_opcode_p = true;
-#else
-  past_opcode_p = false;
 #endif
 
   CGEN_INIT_PARSE (cd);
   cgen_init_parse_operand (cd);
+#ifdef CGEN_MNEMONIC_OPERANDS
+  past_opcode_p = 0;
+#endif
 
   /* We don't check for (*str != '\0') here because we want to parse
      any trailing fake arguments in the syntax string.  */
@@ -1482,28 +1473,18 @@ parse_insn_normal (CGEN_CPU_DESC cd,
 
   while (* syn != 0)
     {
-      char c = CGEN_SYNTAX_CHAR_P (*syn) ? CGEN_SYNTAX_CHAR (*syn) : 0;
-
-      /* FIXME: Despite this check we may still take inappropriate advantage of
-        the fact that GAS's input scrubber will remove extraneous whitespace.
-        We may also be a little too lax with this now, yet being more strict
-        would require targets to indicate where whitespace is permissible.  */
-      if (past_opcode_p && c != ' ' && ISSPACE (*str)
-         /* No whitespace between consecutive alphanumeric syntax elements.  */
-         && (!ISALNUM (c) || !ISALNUM (prev_syntax_char)))
-       ++str;
-      prev_syntax_char = c;
-
       /* Non operand chars must match exactly.  */
-      if (c != 0)
+      if (CGEN_SYNTAX_CHAR_P (* syn))
        {
          /* FIXME: While we allow for non-GAS callers above, we assume the
             first char after the mnemonic part is a space.  */
-         if (TOLOWER (*str) == TOLOWER (c))
+         /* FIXME: We also take inappropriate advantage of the fact that
+            GAS's input scrubber will remove extraneous blanks.  */
+         if (TOLOWER (*str) == TOLOWER (CGEN_SYNTAX_CHAR (* syn)))
            {
 #ifdef CGEN_MNEMONIC_OPERANDS
-             if (c == ' ')
-               past_opcode_p = true;
+             if (CGEN_SYNTAX_CHAR(* syn) == ' ')
+               past_opcode_p = 1;
 #endif
              ++ syn;
              ++ str;
@@ -1515,7 +1496,7 @@ parse_insn_normal (CGEN_CPU_DESC cd,
 
              /* xgettext:c-format */
              sprintf (msg, _("syntax error (expected char `%c', found `%c')"),
-                      c, *str);
+                      CGEN_SYNTAX_CHAR(*syn), *str);
              return msg;
            }
          else
@@ -1525,12 +1506,15 @@ parse_insn_normal (CGEN_CPU_DESC cd,
 
              /* xgettext:c-format */
              sprintf (msg, _("syntax error (expected char `%c', found end of instruction)"),
-                      c);
+                      CGEN_SYNTAX_CHAR(*syn));
              return msg;
            }
          continue;
        }
 
+#ifdef CGEN_MNEMONIC_OPERANDS
+      (void) past_opcode_p;
+#endif
       /* We have an operand of some sort.  */
       errmsg = cd->parse_operand (cd, CGEN_SYNTAX_FIELD (*syn), &str, fields);
       if (errmsg)
index 1799025e5a49a757b3bc57fa75a68a30c0cb7429..003b78d971456d20449a7f54b203e7a50310ed7b 100644 (file)
@@ -555,7 +555,6 @@ ip2k_cgen_build_insn_regex (CGEN_INSN *insn)
   char rxbuf[CGEN_MAX_RX_ELEMENTS];
   char *rx = rxbuf;
   const CGEN_SYNTAX_CHAR_TYPE *syn;
-  char prev_syntax_char = 0;
   int reg_err;
 
   syn = CGEN_SYNTAX_STRING (CGEN_OPCODE_SYNTAX (opc));
@@ -593,15 +592,6 @@ ip2k_cgen_build_insn_regex (CGEN_INSN *insn)
        {
          char c = CGEN_SYNTAX_CHAR (* syn);
 
-         /* See whitespace related comments in parse_insn_normal().  */
-         if (c != ' ' && prev_syntax_char != ' '
-             && (!ISALNUM (c) || !ISALNUM (prev_syntax_char)))
-           {
-             *rx++ = ' ';
-             *rx++ = '*';
-           }
-         prev_syntax_char = c;
-
          switch (c)
            {
              /* Escape any regex metacharacters in the syntax.  */
@@ -635,7 +625,6 @@ ip2k_cgen_build_insn_regex (CGEN_INSN *insn)
          /* Replace non-syntax fields with globs.  */
          *rx++ = '.';
          *rx++ = '*';
-         prev_syntax_char = 0;
        }
     }
 
@@ -693,8 +682,10 @@ parse_insn_normal (CGEN_CPU_DESC cd,
   const char *errmsg;
   const char *p;
   const CGEN_SYNTAX_CHAR_TYPE * syn;
-  char prev_syntax_char = 0;
-  bool past_opcode_p;
+#ifdef CGEN_MNEMONIC_OPERANDS
+  /* FIXME: wip */
+  int past_opcode_p;
+#endif
 
   /* For now we assume the mnemonic is first (there are no leading operands).
      We can parse it without needing to set up operand parsing.
@@ -710,13 +701,13 @@ parse_insn_normal (CGEN_CPU_DESC cd,
 #ifndef CGEN_MNEMONIC_OPERANDS
   if (* str && ! ISSPACE (* str))
     return _("unrecognized instruction");
-  past_opcode_p = true;
-#else
-  past_opcode_p = false;
 #endif
 
   CGEN_INIT_PARSE (cd);
   cgen_init_parse_operand (cd);
+#ifdef CGEN_MNEMONIC_OPERANDS
+  past_opcode_p = 0;
+#endif
 
   /* We don't check for (*str != '\0') here because we want to parse
      any trailing fake arguments in the syntax string.  */
@@ -730,28 +721,18 @@ parse_insn_normal (CGEN_CPU_DESC cd,
 
   while (* syn != 0)
     {
-      char c = CGEN_SYNTAX_CHAR_P (*syn) ? CGEN_SYNTAX_CHAR (*syn) : 0;
-
-      /* FIXME: Despite this check we may still take inappropriate advantage of
-        the fact that GAS's input scrubber will remove extraneous whitespace.
-        We may also be a little too lax with this now, yet being more strict
-        would require targets to indicate where whitespace is permissible.  */
-      if (past_opcode_p && c != ' ' && ISSPACE (*str)
-         /* No whitespace between consecutive alphanumeric syntax elements.  */
-         && (!ISALNUM (c) || !ISALNUM (prev_syntax_char)))
-       ++str;
-      prev_syntax_char = c;
-
       /* Non operand chars must match exactly.  */
-      if (c != 0)
+      if (CGEN_SYNTAX_CHAR_P (* syn))
        {
          /* FIXME: While we allow for non-GAS callers above, we assume the
             first char after the mnemonic part is a space.  */
-         if (TOLOWER (*str) == TOLOWER (c))
+         /* FIXME: We also take inappropriate advantage of the fact that
+            GAS's input scrubber will remove extraneous blanks.  */
+         if (TOLOWER (*str) == TOLOWER (CGEN_SYNTAX_CHAR (* syn)))
            {
 #ifdef CGEN_MNEMONIC_OPERANDS
-             if (c == ' ')
-               past_opcode_p = true;
+             if (CGEN_SYNTAX_CHAR(* syn) == ' ')
+               past_opcode_p = 1;
 #endif
              ++ syn;
              ++ str;
@@ -763,7 +744,7 @@ parse_insn_normal (CGEN_CPU_DESC cd,
 
              /* xgettext:c-format */
              sprintf (msg, _("syntax error (expected char `%c', found `%c')"),
-                      c, *str);
+                      CGEN_SYNTAX_CHAR(*syn), *str);
              return msg;
            }
          else
@@ -773,12 +754,15 @@ parse_insn_normal (CGEN_CPU_DESC cd,
 
              /* xgettext:c-format */
              sprintf (msg, _("syntax error (expected char `%c', found end of instruction)"),
-                      c);
+                      CGEN_SYNTAX_CHAR(*syn));
              return msg;
            }
          continue;
        }
 
+#ifdef CGEN_MNEMONIC_OPERANDS
+      (void) past_opcode_p;
+#endif
       /* We have an operand of some sort.  */
       errmsg = cd->parse_operand (cd, CGEN_SYNTAX_FIELD (*syn), &str, fields);
       if (errmsg)
index 1a329c2cf852e175a124114a010693b80436b812..66d828fd2582f73e17be77e18d71cc7e94c07ac7 100644 (file)
@@ -503,7 +503,6 @@ iq2000_cgen_build_insn_regex (CGEN_INSN *insn)
   char rxbuf[CGEN_MAX_RX_ELEMENTS];
   char *rx = rxbuf;
   const CGEN_SYNTAX_CHAR_TYPE *syn;
-  char prev_syntax_char = 0;
   int reg_err;
 
   syn = CGEN_SYNTAX_STRING (CGEN_OPCODE_SYNTAX (opc));
@@ -541,15 +540,6 @@ iq2000_cgen_build_insn_regex (CGEN_INSN *insn)
        {
          char c = CGEN_SYNTAX_CHAR (* syn);
 
-         /* See whitespace related comments in parse_insn_normal().  */
-         if (c != ' ' && prev_syntax_char != ' '
-             && (!ISALNUM (c) || !ISALNUM (prev_syntax_char)))
-           {
-             *rx++ = ' ';
-             *rx++ = '*';
-           }
-         prev_syntax_char = c;
-
          switch (c)
            {
              /* Escape any regex metacharacters in the syntax.  */
@@ -583,7 +573,6 @@ iq2000_cgen_build_insn_regex (CGEN_INSN *insn)
          /* Replace non-syntax fields with globs.  */
          *rx++ = '.';
          *rx++ = '*';
-         prev_syntax_char = 0;
        }
     }
 
@@ -641,8 +630,10 @@ parse_insn_normal (CGEN_CPU_DESC cd,
   const char *errmsg;
   const char *p;
   const CGEN_SYNTAX_CHAR_TYPE * syn;
-  char prev_syntax_char = 0;
-  bool past_opcode_p;
+#ifdef CGEN_MNEMONIC_OPERANDS
+  /* FIXME: wip */
+  int past_opcode_p;
+#endif
 
   /* For now we assume the mnemonic is first (there are no leading operands).
      We can parse it without needing to set up operand parsing.
@@ -658,13 +649,13 @@ parse_insn_normal (CGEN_CPU_DESC cd,
 #ifndef CGEN_MNEMONIC_OPERANDS
   if (* str && ! ISSPACE (* str))
     return _("unrecognized instruction");
-  past_opcode_p = true;
-#else
-  past_opcode_p = false;
 #endif
 
   CGEN_INIT_PARSE (cd);
   cgen_init_parse_operand (cd);
+#ifdef CGEN_MNEMONIC_OPERANDS
+  past_opcode_p = 0;
+#endif
 
   /* We don't check for (*str != '\0') here because we want to parse
      any trailing fake arguments in the syntax string.  */
@@ -678,28 +669,18 @@ parse_insn_normal (CGEN_CPU_DESC cd,
 
   while (* syn != 0)
     {
-      char c = CGEN_SYNTAX_CHAR_P (*syn) ? CGEN_SYNTAX_CHAR (*syn) : 0;
-
-      /* FIXME: Despite this check we may still take inappropriate advantage of
-        the fact that GAS's input scrubber will remove extraneous whitespace.
-        We may also be a little too lax with this now, yet being more strict
-        would require targets to indicate where whitespace is permissible.  */
-      if (past_opcode_p && c != ' ' && ISSPACE (*str)
-         /* No whitespace between consecutive alphanumeric syntax elements.  */
-         && (!ISALNUM (c) || !ISALNUM (prev_syntax_char)))
-       ++str;
-      prev_syntax_char = c;
-
       /* Non operand chars must match exactly.  */
-      if (c != 0)
+      if (CGEN_SYNTAX_CHAR_P (* syn))
        {
          /* FIXME: While we allow for non-GAS callers above, we assume the
             first char after the mnemonic part is a space.  */
-         if (TOLOWER (*str) == TOLOWER (c))
+         /* FIXME: We also take inappropriate advantage of the fact that
+            GAS's input scrubber will remove extraneous blanks.  */
+         if (TOLOWER (*str) == TOLOWER (CGEN_SYNTAX_CHAR (* syn)))
            {
 #ifdef CGEN_MNEMONIC_OPERANDS
-             if (c == ' ')
-               past_opcode_p = true;
+             if (CGEN_SYNTAX_CHAR(* syn) == ' ')
+               past_opcode_p = 1;
 #endif
              ++ syn;
              ++ str;
@@ -711,7 +692,7 @@ parse_insn_normal (CGEN_CPU_DESC cd,
 
              /* xgettext:c-format */
              sprintf (msg, _("syntax error (expected char `%c', found `%c')"),
-                      c, *str);
+                      CGEN_SYNTAX_CHAR(*syn), *str);
              return msg;
            }
          else
@@ -721,12 +702,15 @@ parse_insn_normal (CGEN_CPU_DESC cd,
 
              /* xgettext:c-format */
              sprintf (msg, _("syntax error (expected char `%c', found end of instruction)"),
-                      c);
+                      CGEN_SYNTAX_CHAR(*syn));
              return msg;
            }
          continue;
        }
 
+#ifdef CGEN_MNEMONIC_OPERANDS
+      (void) past_opcode_p;
+#endif
       /* We have an operand of some sort.  */
       errmsg = cd->parse_operand (cd, CGEN_SYNTAX_FIELD (*syn), &str, fields);
       if (errmsg)
index 5c4f9f4b69fd7e091ae1c5ec6789b1d06c0b4ed8..d5e25f9e8b94f1e367b042eeb519f652ce4d7975 100644 (file)
@@ -393,7 +393,6 @@ lm32_cgen_build_insn_regex (CGEN_INSN *insn)
   char rxbuf[CGEN_MAX_RX_ELEMENTS];
   char *rx = rxbuf;
   const CGEN_SYNTAX_CHAR_TYPE *syn;
-  char prev_syntax_char = 0;
   int reg_err;
 
   syn = CGEN_SYNTAX_STRING (CGEN_OPCODE_SYNTAX (opc));
@@ -431,15 +430,6 @@ lm32_cgen_build_insn_regex (CGEN_INSN *insn)
        {
          char c = CGEN_SYNTAX_CHAR (* syn);
 
-         /* See whitespace related comments in parse_insn_normal().  */
-         if (c != ' ' && prev_syntax_char != ' '
-             && (!ISALNUM (c) || !ISALNUM (prev_syntax_char)))
-           {
-             *rx++ = ' ';
-             *rx++ = '*';
-           }
-         prev_syntax_char = c;
-
          switch (c)
            {
              /* Escape any regex metacharacters in the syntax.  */
@@ -473,7 +463,6 @@ lm32_cgen_build_insn_regex (CGEN_INSN *insn)
          /* Replace non-syntax fields with globs.  */
          *rx++ = '.';
          *rx++ = '*';
-         prev_syntax_char = 0;
        }
     }
 
@@ -531,8 +520,10 @@ parse_insn_normal (CGEN_CPU_DESC cd,
   const char *errmsg;
   const char *p;
   const CGEN_SYNTAX_CHAR_TYPE * syn;
-  char prev_syntax_char = 0;
-  bool past_opcode_p;
+#ifdef CGEN_MNEMONIC_OPERANDS
+  /* FIXME: wip */
+  int past_opcode_p;
+#endif
 
   /* For now we assume the mnemonic is first (there are no leading operands).
      We can parse it without needing to set up operand parsing.
@@ -548,13 +539,13 @@ parse_insn_normal (CGEN_CPU_DESC cd,
 #ifndef CGEN_MNEMONIC_OPERANDS
   if (* str && ! ISSPACE (* str))
     return _("unrecognized instruction");
-  past_opcode_p = true;
-#else
-  past_opcode_p = false;
 #endif
 
   CGEN_INIT_PARSE (cd);
   cgen_init_parse_operand (cd);
+#ifdef CGEN_MNEMONIC_OPERANDS
+  past_opcode_p = 0;
+#endif
 
   /* We don't check for (*str != '\0') here because we want to parse
      any trailing fake arguments in the syntax string.  */
@@ -568,28 +559,18 @@ parse_insn_normal (CGEN_CPU_DESC cd,
 
   while (* syn != 0)
     {
-      char c = CGEN_SYNTAX_CHAR_P (*syn) ? CGEN_SYNTAX_CHAR (*syn) : 0;
-
-      /* FIXME: Despite this check we may still take inappropriate advantage of
-        the fact that GAS's input scrubber will remove extraneous whitespace.
-        We may also be a little too lax with this now, yet being more strict
-        would require targets to indicate where whitespace is permissible.  */
-      if (past_opcode_p && c != ' ' && ISSPACE (*str)
-         /* No whitespace between consecutive alphanumeric syntax elements.  */
-         && (!ISALNUM (c) || !ISALNUM (prev_syntax_char)))
-       ++str;
-      prev_syntax_char = c;
-
       /* Non operand chars must match exactly.  */
-      if (c != 0)
+      if (CGEN_SYNTAX_CHAR_P (* syn))
        {
          /* FIXME: While we allow for non-GAS callers above, we assume the
             first char after the mnemonic part is a space.  */
-         if (TOLOWER (*str) == TOLOWER (c))
+         /* FIXME: We also take inappropriate advantage of the fact that
+            GAS's input scrubber will remove extraneous blanks.  */
+         if (TOLOWER (*str) == TOLOWER (CGEN_SYNTAX_CHAR (* syn)))
            {
 #ifdef CGEN_MNEMONIC_OPERANDS
-             if (c == ' ')
-               past_opcode_p = true;
+             if (CGEN_SYNTAX_CHAR(* syn) == ' ')
+               past_opcode_p = 1;
 #endif
              ++ syn;
              ++ str;
@@ -601,7 +582,7 @@ parse_insn_normal (CGEN_CPU_DESC cd,
 
              /* xgettext:c-format */
              sprintf (msg, _("syntax error (expected char `%c', found `%c')"),
-                      c, *str);
+                      CGEN_SYNTAX_CHAR(*syn), *str);
              return msg;
            }
          else
@@ -611,12 +592,15 @@ parse_insn_normal (CGEN_CPU_DESC cd,
 
              /* xgettext:c-format */
              sprintf (msg, _("syntax error (expected char `%c', found end of instruction)"),
-                      c);
+                      CGEN_SYNTAX_CHAR(*syn));
              return msg;
            }
          continue;
        }
 
+#ifdef CGEN_MNEMONIC_OPERANDS
+      (void) past_opcode_p;
+#endif
       /* We have an operand of some sort.  */
       errmsg = cd->parse_operand (cd, CGEN_SYNTAX_FIELD (*syn), &str, fields);
       if (errmsg)
index aebb59f0e26858dee95a61ace0adb1c34a6eb3f4..8a7983c4c5bdc5d3932ddf9c40dbf80ff6421307 100644 (file)
@@ -1628,7 +1628,6 @@ m32c_cgen_build_insn_regex (CGEN_INSN *insn)
   char rxbuf[CGEN_MAX_RX_ELEMENTS];
   char *rx = rxbuf;
   const CGEN_SYNTAX_CHAR_TYPE *syn;
-  char prev_syntax_char = 0;
   int reg_err;
 
   syn = CGEN_SYNTAX_STRING (CGEN_OPCODE_SYNTAX (opc));
@@ -1666,15 +1665,6 @@ m32c_cgen_build_insn_regex (CGEN_INSN *insn)
        {
          char c = CGEN_SYNTAX_CHAR (* syn);
 
-         /* See whitespace related comments in parse_insn_normal().  */
-         if (c != ' ' && prev_syntax_char != ' '
-             && (!ISALNUM (c) || !ISALNUM (prev_syntax_char)))
-           {
-             *rx++ = ' ';
-             *rx++ = '*';
-           }
-         prev_syntax_char = c;
-
          switch (c)
            {
              /* Escape any regex metacharacters in the syntax.  */
@@ -1708,7 +1698,6 @@ m32c_cgen_build_insn_regex (CGEN_INSN *insn)
          /* Replace non-syntax fields with globs.  */
          *rx++ = '.';
          *rx++ = '*';
-         prev_syntax_char = 0;
        }
     }
 
@@ -1766,8 +1755,10 @@ parse_insn_normal (CGEN_CPU_DESC cd,
   const char *errmsg;
   const char *p;
   const CGEN_SYNTAX_CHAR_TYPE * syn;
-  char prev_syntax_char = 0;
-  bool past_opcode_p;
+#ifdef CGEN_MNEMONIC_OPERANDS
+  /* FIXME: wip */
+  int past_opcode_p;
+#endif
 
   /* For now we assume the mnemonic is first (there are no leading operands).
      We can parse it without needing to set up operand parsing.
@@ -1783,13 +1774,13 @@ parse_insn_normal (CGEN_CPU_DESC cd,
 #ifndef CGEN_MNEMONIC_OPERANDS
   if (* str && ! ISSPACE (* str))
     return _("unrecognized instruction");
-  past_opcode_p = true;
-#else
-  past_opcode_p = false;
 #endif
 
   CGEN_INIT_PARSE (cd);
   cgen_init_parse_operand (cd);
+#ifdef CGEN_MNEMONIC_OPERANDS
+  past_opcode_p = 0;
+#endif
 
   /* We don't check for (*str != '\0') here because we want to parse
      any trailing fake arguments in the syntax string.  */
@@ -1803,28 +1794,18 @@ parse_insn_normal (CGEN_CPU_DESC cd,
 
   while (* syn != 0)
     {
-      char c = CGEN_SYNTAX_CHAR_P (*syn) ? CGEN_SYNTAX_CHAR (*syn) : 0;
-
-      /* FIXME: Despite this check we may still take inappropriate advantage of
-        the fact that GAS's input scrubber will remove extraneous whitespace.
-        We may also be a little too lax with this now, yet being more strict
-        would require targets to indicate where whitespace is permissible.  */
-      if (past_opcode_p && c != ' ' && ISSPACE (*str)
-         /* No whitespace between consecutive alphanumeric syntax elements.  */
-         && (!ISALNUM (c) || !ISALNUM (prev_syntax_char)))
-       ++str;
-      prev_syntax_char = c;
-
       /* Non operand chars must match exactly.  */
-      if (c != 0)
+      if (CGEN_SYNTAX_CHAR_P (* syn))
        {
          /* FIXME: While we allow for non-GAS callers above, we assume the
             first char after the mnemonic part is a space.  */
-         if (TOLOWER (*str) == TOLOWER (c))
+         /* FIXME: We also take inappropriate advantage of the fact that
+            GAS's input scrubber will remove extraneous blanks.  */
+         if (TOLOWER (*str) == TOLOWER (CGEN_SYNTAX_CHAR (* syn)))
            {
 #ifdef CGEN_MNEMONIC_OPERANDS
-             if (c == ' ')
-               past_opcode_p = true;
+             if (CGEN_SYNTAX_CHAR(* syn) == ' ')
+               past_opcode_p = 1;
 #endif
              ++ syn;
              ++ str;
@@ -1836,7 +1817,7 @@ parse_insn_normal (CGEN_CPU_DESC cd,
 
              /* xgettext:c-format */
              sprintf (msg, _("syntax error (expected char `%c', found `%c')"),
-                      c, *str);
+                      CGEN_SYNTAX_CHAR(*syn), *str);
              return msg;
            }
          else
@@ -1846,12 +1827,15 @@ parse_insn_normal (CGEN_CPU_DESC cd,
 
              /* xgettext:c-format */
              sprintf (msg, _("syntax error (expected char `%c', found end of instruction)"),
-                      c);
+                      CGEN_SYNTAX_CHAR(*syn));
              return msg;
            }
          continue;
        }
 
+#ifdef CGEN_MNEMONIC_OPERANDS
+      (void) past_opcode_p;
+#endif
       /* We have an operand of some sort.  */
       errmsg = cd->parse_operand (cd, CGEN_SYNTAX_FIELD (*syn), &str, fields);
       if (errmsg)
index ae6ed4991429659224b409ae385cd18bf9bf1603..4d19babdb007da24e9c16ce618f4a2303a715c94 100644 (file)
@@ -372,7 +372,6 @@ m32r_cgen_build_insn_regex (CGEN_INSN *insn)
   char rxbuf[CGEN_MAX_RX_ELEMENTS];
   char *rx = rxbuf;
   const CGEN_SYNTAX_CHAR_TYPE *syn;
-  char prev_syntax_char = 0;
   int reg_err;
 
   syn = CGEN_SYNTAX_STRING (CGEN_OPCODE_SYNTAX (opc));
@@ -410,15 +409,6 @@ m32r_cgen_build_insn_regex (CGEN_INSN *insn)
        {
          char c = CGEN_SYNTAX_CHAR (* syn);
 
-         /* See whitespace related comments in parse_insn_normal().  */
-         if (c != ' ' && prev_syntax_char != ' '
-             && (!ISALNUM (c) || !ISALNUM (prev_syntax_char)))
-           {
-             *rx++ = ' ';
-             *rx++ = '*';
-           }
-         prev_syntax_char = c;
-
          switch (c)
            {
              /* Escape any regex metacharacters in the syntax.  */
@@ -452,7 +442,6 @@ m32r_cgen_build_insn_regex (CGEN_INSN *insn)
          /* Replace non-syntax fields with globs.  */
          *rx++ = '.';
          *rx++ = '*';
-         prev_syntax_char = 0;
        }
     }
 
@@ -510,8 +499,10 @@ parse_insn_normal (CGEN_CPU_DESC cd,
   const char *errmsg;
   const char *p;
   const CGEN_SYNTAX_CHAR_TYPE * syn;
-  char prev_syntax_char = 0;
-  bool past_opcode_p;
+#ifdef CGEN_MNEMONIC_OPERANDS
+  /* FIXME: wip */
+  int past_opcode_p;
+#endif
 
   /* For now we assume the mnemonic is first (there are no leading operands).
      We can parse it without needing to set up operand parsing.
@@ -527,13 +518,13 @@ parse_insn_normal (CGEN_CPU_DESC cd,
 #ifndef CGEN_MNEMONIC_OPERANDS
   if (* str && ! ISSPACE (* str))
     return _("unrecognized instruction");
-  past_opcode_p = true;
-#else
-  past_opcode_p = false;
 #endif
 
   CGEN_INIT_PARSE (cd);
   cgen_init_parse_operand (cd);
+#ifdef CGEN_MNEMONIC_OPERANDS
+  past_opcode_p = 0;
+#endif
 
   /* We don't check for (*str != '\0') here because we want to parse
      any trailing fake arguments in the syntax string.  */
@@ -547,28 +538,18 @@ parse_insn_normal (CGEN_CPU_DESC cd,
 
   while (* syn != 0)
     {
-      char c = CGEN_SYNTAX_CHAR_P (*syn) ? CGEN_SYNTAX_CHAR (*syn) : 0;
-
-      /* FIXME: Despite this check we may still take inappropriate advantage of
-        the fact that GAS's input scrubber will remove extraneous whitespace.
-        We may also be a little too lax with this now, yet being more strict
-        would require targets to indicate where whitespace is permissible.  */
-      if (past_opcode_p && c != ' ' && ISSPACE (*str)
-         /* No whitespace between consecutive alphanumeric syntax elements.  */
-         && (!ISALNUM (c) || !ISALNUM (prev_syntax_char)))
-       ++str;
-      prev_syntax_char = c;
-
       /* Non operand chars must match exactly.  */
-      if (c != 0)
+      if (CGEN_SYNTAX_CHAR_P (* syn))
        {
          /* FIXME: While we allow for non-GAS callers above, we assume the
             first char after the mnemonic part is a space.  */
-         if (TOLOWER (*str) == TOLOWER (c))
+         /* FIXME: We also take inappropriate advantage of the fact that
+            GAS's input scrubber will remove extraneous blanks.  */
+         if (TOLOWER (*str) == TOLOWER (CGEN_SYNTAX_CHAR (* syn)))
            {
 #ifdef CGEN_MNEMONIC_OPERANDS
-             if (c == ' ')
-               past_opcode_p = true;
+             if (CGEN_SYNTAX_CHAR(* syn) == ' ')
+               past_opcode_p = 1;
 #endif
              ++ syn;
              ++ str;
@@ -580,7 +561,7 @@ parse_insn_normal (CGEN_CPU_DESC cd,
 
              /* xgettext:c-format */
              sprintf (msg, _("syntax error (expected char `%c', found `%c')"),
-                      c, *str);
+                      CGEN_SYNTAX_CHAR(*syn), *str);
              return msg;
            }
          else
@@ -590,12 +571,15 @@ parse_insn_normal (CGEN_CPU_DESC cd,
 
              /* xgettext:c-format */
              sprintf (msg, _("syntax error (expected char `%c', found end of instruction)"),
-                      c);
+                      CGEN_SYNTAX_CHAR(*syn));
              return msg;
            }
          continue;
        }
 
+#ifdef CGEN_MNEMONIC_OPERANDS
+      (void) past_opcode_p;
+#endif
       /* We have an operand of some sort.  */
       errmsg = cd->parse_operand (cd, CGEN_SYNTAX_FIELD (*syn), &str, fields);
       if (errmsg)
index 87af40ca844c5b0316c19d373f10ff4296421aa7..6e72ddba5a1173915b3da5f31e34046f1b9ba8e4 100644 (file)
@@ -1331,7 +1331,6 @@ mep_cgen_build_insn_regex (CGEN_INSN *insn)
   char rxbuf[CGEN_MAX_RX_ELEMENTS];
   char *rx = rxbuf;
   const CGEN_SYNTAX_CHAR_TYPE *syn;
-  char prev_syntax_char = 0;
   int reg_err;
 
   syn = CGEN_SYNTAX_STRING (CGEN_OPCODE_SYNTAX (opc));
@@ -1369,15 +1368,6 @@ mep_cgen_build_insn_regex (CGEN_INSN *insn)
        {
          char c = CGEN_SYNTAX_CHAR (* syn);
 
-         /* See whitespace related comments in parse_insn_normal().  */
-         if (c != ' ' && prev_syntax_char != ' '
-             && (!ISALNUM (c) || !ISALNUM (prev_syntax_char)))
-           {
-             *rx++ = ' ';
-             *rx++ = '*';
-           }
-         prev_syntax_char = c;
-
          switch (c)
            {
              /* Escape any regex metacharacters in the syntax.  */
@@ -1411,7 +1401,6 @@ mep_cgen_build_insn_regex (CGEN_INSN *insn)
          /* Replace non-syntax fields with globs.  */
          *rx++ = '.';
          *rx++ = '*';
-         prev_syntax_char = 0;
        }
     }
 
@@ -1469,8 +1458,10 @@ parse_insn_normal (CGEN_CPU_DESC cd,
   const char *errmsg;
   const char *p;
   const CGEN_SYNTAX_CHAR_TYPE * syn;
-  char prev_syntax_char = 0;
-  bool past_opcode_p;
+#ifdef CGEN_MNEMONIC_OPERANDS
+  /* FIXME: wip */
+  int past_opcode_p;
+#endif
 
   /* For now we assume the mnemonic is first (there are no leading operands).
      We can parse it without needing to set up operand parsing.
@@ -1486,13 +1477,13 @@ parse_insn_normal (CGEN_CPU_DESC cd,
 #ifndef CGEN_MNEMONIC_OPERANDS
   if (* str && ! ISSPACE (* str))
     return _("unrecognized instruction");
-  past_opcode_p = true;
-#else
-  past_opcode_p = false;
 #endif
 
   CGEN_INIT_PARSE (cd);
   cgen_init_parse_operand (cd);
+#ifdef CGEN_MNEMONIC_OPERANDS
+  past_opcode_p = 0;
+#endif
 
   /* We don't check for (*str != '\0') here because we want to parse
      any trailing fake arguments in the syntax string.  */
@@ -1506,28 +1497,18 @@ parse_insn_normal (CGEN_CPU_DESC cd,
 
   while (* syn != 0)
     {
-      char c = CGEN_SYNTAX_CHAR_P (*syn) ? CGEN_SYNTAX_CHAR (*syn) : 0;
-
-      /* FIXME: Despite this check we may still take inappropriate advantage of
-        the fact that GAS's input scrubber will remove extraneous whitespace.
-        We may also be a little too lax with this now, yet being more strict
-        would require targets to indicate where whitespace is permissible.  */
-      if (past_opcode_p && c != ' ' && ISSPACE (*str)
-         /* No whitespace between consecutive alphanumeric syntax elements.  */
-         && (!ISALNUM (c) || !ISALNUM (prev_syntax_char)))
-       ++str;
-      prev_syntax_char = c;
-
       /* Non operand chars must match exactly.  */
-      if (c != 0)
+      if (CGEN_SYNTAX_CHAR_P (* syn))
        {
          /* FIXME: While we allow for non-GAS callers above, we assume the
             first char after the mnemonic part is a space.  */
-         if (TOLOWER (*str) == TOLOWER (c))
+         /* FIXME: We also take inappropriate advantage of the fact that
+            GAS's input scrubber will remove extraneous blanks.  */
+         if (TOLOWER (*str) == TOLOWER (CGEN_SYNTAX_CHAR (* syn)))
            {
 #ifdef CGEN_MNEMONIC_OPERANDS
-             if (c == ' ')
-               past_opcode_p = true;
+             if (CGEN_SYNTAX_CHAR(* syn) == ' ')
+               past_opcode_p = 1;
 #endif
              ++ syn;
              ++ str;
@@ -1539,7 +1520,7 @@ parse_insn_normal (CGEN_CPU_DESC cd,
 
              /* xgettext:c-format */
              sprintf (msg, _("syntax error (expected char `%c', found `%c')"),
-                      c, *str);
+                      CGEN_SYNTAX_CHAR(*syn), *str);
              return msg;
            }
          else
@@ -1549,12 +1530,15 @@ parse_insn_normal (CGEN_CPU_DESC cd,
 
              /* xgettext:c-format */
              sprintf (msg, _("syntax error (expected char `%c', found end of instruction)"),
-                      c);
+                      CGEN_SYNTAX_CHAR(*syn));
              return msg;
            }
          continue;
        }
 
+#ifdef CGEN_MNEMONIC_OPERANDS
+      (void) past_opcode_p;
+#endif
       /* We have an operand of some sort.  */
       errmsg = cd->parse_operand (cd, CGEN_SYNTAX_FIELD (*syn), &str, fields);
       if (errmsg)
index 8263670f7ac04995c253684ea0e29b1616df572e..074c1ab57ace470bbd6191a0e01188e1fef28140 100644 (file)
@@ -639,7 +639,6 @@ mt_cgen_build_insn_regex (CGEN_INSN *insn)
   char rxbuf[CGEN_MAX_RX_ELEMENTS];
   char *rx = rxbuf;
   const CGEN_SYNTAX_CHAR_TYPE *syn;
-  char prev_syntax_char = 0;
   int reg_err;
 
   syn = CGEN_SYNTAX_STRING (CGEN_OPCODE_SYNTAX (opc));
@@ -677,15 +676,6 @@ mt_cgen_build_insn_regex (CGEN_INSN *insn)
        {
          char c = CGEN_SYNTAX_CHAR (* syn);
 
-         /* See whitespace related comments in parse_insn_normal().  */
-         if (c != ' ' && prev_syntax_char != ' '
-             && (!ISALNUM (c) || !ISALNUM (prev_syntax_char)))
-           {
-             *rx++ = ' ';
-             *rx++ = '*';
-           }
-         prev_syntax_char = c;
-
          switch (c)
            {
              /* Escape any regex metacharacters in the syntax.  */
@@ -719,7 +709,6 @@ mt_cgen_build_insn_regex (CGEN_INSN *insn)
          /* Replace non-syntax fields with globs.  */
          *rx++ = '.';
          *rx++ = '*';
-         prev_syntax_char = 0;
        }
     }
 
@@ -777,8 +766,10 @@ parse_insn_normal (CGEN_CPU_DESC cd,
   const char *errmsg;
   const char *p;
   const CGEN_SYNTAX_CHAR_TYPE * syn;
-  char prev_syntax_char = 0;
-  bool past_opcode_p;
+#ifdef CGEN_MNEMONIC_OPERANDS
+  /* FIXME: wip */
+  int past_opcode_p;
+#endif
 
   /* For now we assume the mnemonic is first (there are no leading operands).
      We can parse it without needing to set up operand parsing.
@@ -794,13 +785,13 @@ parse_insn_normal (CGEN_CPU_DESC cd,
 #ifndef CGEN_MNEMONIC_OPERANDS
   if (* str && ! ISSPACE (* str))
     return _("unrecognized instruction");
-  past_opcode_p = true;
-#else
-  past_opcode_p = false;
 #endif
 
   CGEN_INIT_PARSE (cd);
   cgen_init_parse_operand (cd);
+#ifdef CGEN_MNEMONIC_OPERANDS
+  past_opcode_p = 0;
+#endif
 
   /* We don't check for (*str != '\0') here because we want to parse
      any trailing fake arguments in the syntax string.  */
@@ -814,28 +805,18 @@ parse_insn_normal (CGEN_CPU_DESC cd,
 
   while (* syn != 0)
     {
-      char c = CGEN_SYNTAX_CHAR_P (*syn) ? CGEN_SYNTAX_CHAR (*syn) : 0;
-
-      /* FIXME: Despite this check we may still take inappropriate advantage of
-        the fact that GAS's input scrubber will remove extraneous whitespace.
-        We may also be a little too lax with this now, yet being more strict
-        would require targets to indicate where whitespace is permissible.  */
-      if (past_opcode_p && c != ' ' && ISSPACE (*str)
-         /* No whitespace between consecutive alphanumeric syntax elements.  */
-         && (!ISALNUM (c) || !ISALNUM (prev_syntax_char)))
-       ++str;
-      prev_syntax_char = c;
-
       /* Non operand chars must match exactly.  */
-      if (c != 0)
+      if (CGEN_SYNTAX_CHAR_P (* syn))
        {
          /* FIXME: While we allow for non-GAS callers above, we assume the
             first char after the mnemonic part is a space.  */
-         if (TOLOWER (*str) == TOLOWER (c))
+         /* FIXME: We also take inappropriate advantage of the fact that
+            GAS's input scrubber will remove extraneous blanks.  */
+         if (TOLOWER (*str) == TOLOWER (CGEN_SYNTAX_CHAR (* syn)))
            {
 #ifdef CGEN_MNEMONIC_OPERANDS
-             if (c == ' ')
-               past_opcode_p = true;
+             if (CGEN_SYNTAX_CHAR(* syn) == ' ')
+               past_opcode_p = 1;
 #endif
              ++ syn;
              ++ str;
@@ -847,7 +828,7 @@ parse_insn_normal (CGEN_CPU_DESC cd,
 
              /* xgettext:c-format */
              sprintf (msg, _("syntax error (expected char `%c', found `%c')"),
-                      c, *str);
+                      CGEN_SYNTAX_CHAR(*syn), *str);
              return msg;
            }
          else
@@ -857,12 +838,15 @@ parse_insn_normal (CGEN_CPU_DESC cd,
 
              /* xgettext:c-format */
              sprintf (msg, _("syntax error (expected char `%c', found end of instruction)"),
-                      c);
+                      CGEN_SYNTAX_CHAR(*syn));
              return msg;
            }
          continue;
        }
 
+#ifdef CGEN_MNEMONIC_OPERANDS
+      (void) past_opcode_p;
+#endif
       /* We have an operand of some sort.  */
       errmsg = cd->parse_operand (cd, CGEN_SYNTAX_FIELD (*syn), &str, fields);
       if (errmsg)
index f7a5e54f985dae66bde721655ececae501649504..67a983365303ecfd31f862a1cad47e588800f9d3 100644 (file)
@@ -2486,9 +2486,6 @@ parse_insn (nds32_asm_desc_t *pdesc, nds32_asm_insn_t *pinsn,
 
       while (*plex)
        {
-         if (ISSPACE (*p))
-           ++p;
-
          if (IS_LEX_CHAR (*plex))
            {
              /* If it's a plain char, just compare it.  */
@@ -2533,8 +2530,6 @@ parse_insn (nds32_asm_desc_t *pdesc, nds32_asm_insn_t *pinsn,
        }
 
       /* Check whether this syntax is accepted.  */
-      if (ISSPACE (*p))
-       ++p;
       if (*plex == 0 && (*p == '\0' || *p == '!' || *p == '#'))
        return 1;
 
index 9351c1b407b1beae802bc8255552fdd4734adf69..0839366788d9569ae23aa70bdf1b91cdffd9c4e5 100644 (file)
@@ -619,7 +619,6 @@ or1k_cgen_build_insn_regex (CGEN_INSN *insn)
   char rxbuf[CGEN_MAX_RX_ELEMENTS];
   char *rx = rxbuf;
   const CGEN_SYNTAX_CHAR_TYPE *syn;
-  char prev_syntax_char = 0;
   int reg_err;
 
   syn = CGEN_SYNTAX_STRING (CGEN_OPCODE_SYNTAX (opc));
@@ -657,15 +656,6 @@ or1k_cgen_build_insn_regex (CGEN_INSN *insn)
        {
          char c = CGEN_SYNTAX_CHAR (* syn);
 
-         /* See whitespace related comments in parse_insn_normal().  */
-         if (c != ' ' && prev_syntax_char != ' '
-             && (!ISALNUM (c) || !ISALNUM (prev_syntax_char)))
-           {
-             *rx++ = ' ';
-             *rx++ = '*';
-           }
-         prev_syntax_char = c;
-
          switch (c)
            {
              /* Escape any regex metacharacters in the syntax.  */
@@ -699,7 +689,6 @@ or1k_cgen_build_insn_regex (CGEN_INSN *insn)
          /* Replace non-syntax fields with globs.  */
          *rx++ = '.';
          *rx++ = '*';
-         prev_syntax_char = 0;
        }
     }
 
@@ -757,8 +746,10 @@ parse_insn_normal (CGEN_CPU_DESC cd,
   const char *errmsg;
   const char *p;
   const CGEN_SYNTAX_CHAR_TYPE * syn;
-  char prev_syntax_char = 0;
-  bool past_opcode_p;
+#ifdef CGEN_MNEMONIC_OPERANDS
+  /* FIXME: wip */
+  int past_opcode_p;
+#endif
 
   /* For now we assume the mnemonic is first (there are no leading operands).
      We can parse it without needing to set up operand parsing.
@@ -774,13 +765,13 @@ parse_insn_normal (CGEN_CPU_DESC cd,
 #ifndef CGEN_MNEMONIC_OPERANDS
   if (* str && ! ISSPACE (* str))
     return _("unrecognized instruction");
-  past_opcode_p = true;
-#else
-  past_opcode_p = false;
 #endif
 
   CGEN_INIT_PARSE (cd);
   cgen_init_parse_operand (cd);
+#ifdef CGEN_MNEMONIC_OPERANDS
+  past_opcode_p = 0;
+#endif
 
   /* We don't check for (*str != '\0') here because we want to parse
      any trailing fake arguments in the syntax string.  */
@@ -794,28 +785,18 @@ parse_insn_normal (CGEN_CPU_DESC cd,
 
   while (* syn != 0)
     {
-      char c = CGEN_SYNTAX_CHAR_P (*syn) ? CGEN_SYNTAX_CHAR (*syn) : 0;
-
-      /* FIXME: Despite this check we may still take inappropriate advantage of
-        the fact that GAS's input scrubber will remove extraneous whitespace.
-        We may also be a little too lax with this now, yet being more strict
-        would require targets to indicate where whitespace is permissible.  */
-      if (past_opcode_p && c != ' ' && ISSPACE (*str)
-         /* No whitespace between consecutive alphanumeric syntax elements.  */
-         && (!ISALNUM (c) || !ISALNUM (prev_syntax_char)))
-       ++str;
-      prev_syntax_char = c;
-
       /* Non operand chars must match exactly.  */
-      if (c != 0)
+      if (CGEN_SYNTAX_CHAR_P (* syn))
        {
          /* FIXME: While we allow for non-GAS callers above, we assume the
             first char after the mnemonic part is a space.  */
-         if (TOLOWER (*str) == TOLOWER (c))
+         /* FIXME: We also take inappropriate advantage of the fact that
+            GAS's input scrubber will remove extraneous blanks.  */
+         if (TOLOWER (*str) == TOLOWER (CGEN_SYNTAX_CHAR (* syn)))
            {
 #ifdef CGEN_MNEMONIC_OPERANDS
-             if (c == ' ')
-               past_opcode_p = true;
+             if (CGEN_SYNTAX_CHAR(* syn) == ' ')
+               past_opcode_p = 1;
 #endif
              ++ syn;
              ++ str;
@@ -827,7 +808,7 @@ parse_insn_normal (CGEN_CPU_DESC cd,
 
              /* xgettext:c-format */
              sprintf (msg, _("syntax error (expected char `%c', found `%c')"),
-                      c, *str);
+                      CGEN_SYNTAX_CHAR(*syn), *str);
              return msg;
            }
          else
@@ -837,12 +818,15 @@ parse_insn_normal (CGEN_CPU_DESC cd,
 
              /* xgettext:c-format */
              sprintf (msg, _("syntax error (expected char `%c', found end of instruction)"),
-                      c);
+                      CGEN_SYNTAX_CHAR(*syn));
              return msg;
            }
          continue;
        }
 
+#ifdef CGEN_MNEMONIC_OPERANDS
+      (void) past_opcode_p;
+#endif
       /* We have an operand of some sort.  */
       errmsg = cd->parse_operand (cd, CGEN_SYNTAX_FIELD (*syn), &str, fields);
       if (errmsg)
index 40c9265da188a0ae167f057cef933ad74db314d6..8a9f33cdf93ee85f4d171f89ba26e3823010a333 100644 (file)
@@ -320,7 +320,6 @@ xstormy16_cgen_build_insn_regex (CGEN_INSN *insn)
   char rxbuf[CGEN_MAX_RX_ELEMENTS];
   char *rx = rxbuf;
   const CGEN_SYNTAX_CHAR_TYPE *syn;
-  char prev_syntax_char = 0;
   int reg_err;
 
   syn = CGEN_SYNTAX_STRING (CGEN_OPCODE_SYNTAX (opc));
@@ -358,15 +357,6 @@ xstormy16_cgen_build_insn_regex (CGEN_INSN *insn)
        {
          char c = CGEN_SYNTAX_CHAR (* syn);
 
-         /* See whitespace related comments in parse_insn_normal().  */
-         if (c != ' ' && prev_syntax_char != ' '
-             && (!ISALNUM (c) || !ISALNUM (prev_syntax_char)))
-           {
-             *rx++ = ' ';
-             *rx++ = '*';
-           }
-         prev_syntax_char = c;
-
          switch (c)
            {
              /* Escape any regex metacharacters in the syntax.  */
@@ -400,7 +390,6 @@ xstormy16_cgen_build_insn_regex (CGEN_INSN *insn)
          /* Replace non-syntax fields with globs.  */
          *rx++ = '.';
          *rx++ = '*';
-         prev_syntax_char = 0;
        }
     }
 
@@ -458,8 +447,10 @@ parse_insn_normal (CGEN_CPU_DESC cd,
   const char *errmsg;
   const char *p;
   const CGEN_SYNTAX_CHAR_TYPE * syn;
-  char prev_syntax_char = 0;
-  bool past_opcode_p;
+#ifdef CGEN_MNEMONIC_OPERANDS
+  /* FIXME: wip */
+  int past_opcode_p;
+#endif
 
   /* For now we assume the mnemonic is first (there are no leading operands).
      We can parse it without needing to set up operand parsing.
@@ -475,13 +466,13 @@ parse_insn_normal (CGEN_CPU_DESC cd,
 #ifndef CGEN_MNEMONIC_OPERANDS
   if (* str && ! ISSPACE (* str))
     return _("unrecognized instruction");
-  past_opcode_p = true;
-#else
-  past_opcode_p = false;
 #endif
 
   CGEN_INIT_PARSE (cd);
   cgen_init_parse_operand (cd);
+#ifdef CGEN_MNEMONIC_OPERANDS
+  past_opcode_p = 0;
+#endif
 
   /* We don't check for (*str != '\0') here because we want to parse
      any trailing fake arguments in the syntax string.  */
@@ -495,28 +486,18 @@ parse_insn_normal (CGEN_CPU_DESC cd,
 
   while (* syn != 0)
     {
-      char c = CGEN_SYNTAX_CHAR_P (*syn) ? CGEN_SYNTAX_CHAR (*syn) : 0;
-
-      /* FIXME: Despite this check we may still take inappropriate advantage of
-        the fact that GAS's input scrubber will remove extraneous whitespace.
-        We may also be a little too lax with this now, yet being more strict
-        would require targets to indicate where whitespace is permissible.  */
-      if (past_opcode_p && c != ' ' && ISSPACE (*str)
-         /* No whitespace between consecutive alphanumeric syntax elements.  */
-         && (!ISALNUM (c) || !ISALNUM (prev_syntax_char)))
-       ++str;
-      prev_syntax_char = c;
-
       /* Non operand chars must match exactly.  */
-      if (c != 0)
+      if (CGEN_SYNTAX_CHAR_P (* syn))
        {
          /* FIXME: While we allow for non-GAS callers above, we assume the
             first char after the mnemonic part is a space.  */
-         if (TOLOWER (*str) == TOLOWER (c))
+         /* FIXME: We also take inappropriate advantage of the fact that
+            GAS's input scrubber will remove extraneous blanks.  */
+         if (TOLOWER (*str) == TOLOWER (CGEN_SYNTAX_CHAR (* syn)))
            {
 #ifdef CGEN_MNEMONIC_OPERANDS
-             if (c == ' ')
-               past_opcode_p = true;
+             if (CGEN_SYNTAX_CHAR(* syn) == ' ')
+               past_opcode_p = 1;
 #endif
              ++ syn;
              ++ str;
@@ -528,7 +509,7 @@ parse_insn_normal (CGEN_CPU_DESC cd,
 
              /* xgettext:c-format */
              sprintf (msg, _("syntax error (expected char `%c', found `%c')"),
-                      c, *str);
+                      CGEN_SYNTAX_CHAR(*syn), *str);
              return msg;
            }
          else
@@ -538,12 +519,15 @@ parse_insn_normal (CGEN_CPU_DESC cd,
 
              /* xgettext:c-format */
              sprintf (msg, _("syntax error (expected char `%c', found end of instruction)"),
-                      c);
+                      CGEN_SYNTAX_CHAR(*syn));
              return msg;
            }
          continue;
        }
 
+#ifdef CGEN_MNEMONIC_OPERANDS
+      (void) past_opcode_p;
+#endif
       /* We have an operand of some sort.  */
       errmsg = cd->parse_operand (cd, CGEN_SYNTAX_FIELD (*syn), &str, fields);
       if (errmsg)