]> git.ipfire.org Git - thirdparty/binutils-gdb.git/commitdiff
gas: drop tic6x scrubber special case
authorJan Beulich <jbeulich@suse.com>
Fri, 19 Jul 2024 09:53:24 +0000 (11:53 +0200)
committerJan Beulich <jbeulich@suse.com>
Fri, 19 Jul 2024 09:53:24 +0000 (11:53 +0200)
Two successive PUT() without a state change in between can't be right:
The first PUT() may take the "goto tofull" path, leading to the
subsequent character being processed later in the previously set state
(1 in this case), rather than the state we were in upon entry to the
switch() (13 in this case).

However, the original purpose of that logic appears to be to not mistake
"|| ^" for "||^". This effect, sadly, looks to not have been achieved.
Therefore drop the special case altogether; something that actually
achieves the (presumably) intended effect may then be introduced down
the road.

gas/app.c

index 57398b3f2d32d0e20859ea6a9eeb7746043f8ab8..3ac5e2a13acfc8704867bce97b1af8cfafcd32c0 100644 (file)
--- a/gas/app.c
+++ b/gas/app.c
@@ -769,16 +769,6 @@ do_scrub_chars (size_t (*get) (char *, size_t), char *tostart, size_t tolen,
             line from just after the first white space.  */
          state = 1;
          PUT ('|');
-#ifdef TC_TIC6X
-         /* "||^" is used for SPMASKed instructions.  */
-         ch = GET ();
-         if (ch == EOF)
-           goto fromeof;
-         else if (ch == '^')
-           PUT ('^');
-         else
-           UNGET (ch);
-#endif
          continue;
 #endif
 #ifdef TC_Z80