tc-pdp11.c: remove useless code

if the condition is true then we know that str already points to a'\0' in the
string passed to the function.  Since we know the latter part of the function
doesn't modify that string, and str already points to a null byte there's no
point in changing str to point to a literal empty string.

gas/ChangeLog:

2016-03-22  Trevor Saunders  <tbsaunde+binutils@tbsaunde.org>

	* config/tc-pdp11.c (md_assemble): Remove useless if and assignment to
	str.

diff --git a/gas/ChangeLog b/gas/ChangeLog
index a6960a22e3cc285b9133337754a877f680383bf3..845eacd593177bed504059b1a50be92547a414c7 100644 (file)
--- a/gas/ChangeLog
+++ b/gas/ChangeLog
@@ -1,3 +1,8 @@
+2016-03-22  Trevor Saunders  <tbsaunde+binutils@tbsaunde.org>
+
+       * config/tc-pdp11.c (md_assemble): Remove useless if and assignment to
+       str.
+
 2016-03-22  Trevor Saunders  <tbsaunde+binutils@tbsaunde.org>
 
        * config/tc-sparc.c (sparc_regname_to_dw2regnum): Replace strchr ()

diff --git a/gas/config/tc-pdp11.c b/gas/config/tc-pdp11.c
index 89c75c32100eeb27b95540b480305b4f111f5547..f1447541d84fe91763757c8f48447194886dcc25 100644 (file)
--- a/gas/config/tc-pdp11.c
+++ b/gas/config/tc-pdp11.c
@@ -697,8 +697,6 @@ md_assemble (char *instruction_string)
     {
     case PDP11_OPCODE_NO_OPS:
       str = skip_whitespace (str);
-      if (*str == 0)
-       str = "";
       break;
 
     case PDP11_OPCODE_IMM3: