From: Doug Evans <dje@google.com>
Date: Wed, 28 Jan 1998 17:31:22 +0000 (+0000)
Subject: (assemble_one_insn): tweak lexing of suffix
X-Git-Url: https://git.libre-soc.org/?a=commitdiff_plain;h=aa788b629fd46dfe12972619914feb1848364174;p=binutils-gdb.git

(assemble_one_insn): tweak lexing of suffix
---

diff --git a/gas/config/tc-dvp.c b/gas/config/tc-dvp.c
index 06b516c5263..075d91cf31c 100644
--- a/gas/config/tc-dvp.c
+++ b/gas/config/tc-dvp.c
@@ -495,8 +495,8 @@ assemble_one_insn (cpu, opcode, operand_table, pstr, insn_buf)
 	      /* Pick the suffix out and parse it.  */
 	      /* ??? Hmmm ... there may not be any need to nul-terminate the
 		 string, and it may in fact complicate things.  */
-	      for (t = (*s == '.' || *s == '/') ? s + 1 : s;
-		   *t && (isalnum (*t) || *t == '[' || *t == ']');
+	      for (t = (*s == '.' || *s == '/' || *s == '[') ? s + 1 : s;
+		   *t && (isalnum (*t) || *t == ']');
 		   ++t)
 		continue;
 	      c = *t;
@@ -520,6 +520,9 @@ assemble_one_insn (cpu, opcode, operand_table, pstr, insn_buf)
 	      else
 		*insn_buf |= suf_value << operand->shift;
 
+	      /* FIXME: For suffixes that have a null "" value,
+		 this next line is wrong as we will skip over something
+		 we're not supposed to.  */
 	      str = t;
 	      ++syn;
 	    }