From: Ian Lance Taylor <ian@airs.com>
Date: Fri, 9 Jan 2004 02:34:04 +0000 (+0000)
Subject: 	* config/tc-mips.c (warn_nops): Remove static variable.
X-Git-Url: http://drtracing.org/?a=commitdiff_plain;h=78849248c05bbfd4aec63ab945195eac6e4224b0;p=deliverable%2Fbinutils-gdb.git

	* config/tc-mips.c (warn_nops): Remove static variable.
	(macro): Remove test of warn_nops.
	(md_shortops): Remove 'n'.
	(md_parse_option): Remove 'n' case.
	(md_show_usage): Remove -n.
	* doc/as.texinfo (Overview): Remove MIPS -n option.
	* doc/c-mips.texi (MIPS Opts): Remove mention -n.
	* NEWS: Mention removal of MIPS -n option.
---

diff --git a/gas/ChangeLog b/gas/ChangeLog
index 69cda1f7f4..e9efaf2b4d 100644
--- a/gas/ChangeLog
+++ b/gas/ChangeLog
@@ -1,5 +1,14 @@
 2004-01-08  Ian Lance Taylor  <ian@wasabisystems.com>
 
+	* config/tc-mips.c (warn_nops): Remove static variable.
+	(macro): Remove test of warn_nops.
+	(md_shortops): Remove 'n'.
+	(md_parse_option): Remove 'n' case.
+	(md_show_usage): Remove -n.
+	* doc/as.texinfo (Overview): Remove MIPS -n option.
+	* doc/c-mips.texi (MIPS Opts): Remove mention -n.
+	* NEWS: Mention removal of MIPS -n option.
+
 	* config/tc-mips.c (ISA_HAS_COPROC_DELAYS): Remove.
 	(cop_interlocks): Check ISA level.
 	(cop_mem_interlocks): Define.
diff --git a/gas/NEWS b/gas/NEWS
index cff68629eb..6257fae666 100644
--- a/gas/NEWS
+++ b/gas/NEWS
@@ -28,6 +28,9 @@
   with multi-byte nop instructions such as leal 0(%esi,1),%esi.  This
   switch disables the optimization.
 
+* Removed -n option from MIPS assembler.  It was not useful, and confused the
+  existing -non_shared option.
+
 Changes in 2.14:
 
 * Added support for MIPS32 Release 2.
diff --git a/gas/config/tc-mips.c b/gas/config/tc-mips.c
index 014245362c..da0ea76997 100644
--- a/gas/config/tc-mips.c
+++ b/gas/config/tc-mips.c
@@ -379,9 +379,6 @@ static int mips_32bitmode = 0;
 
 enum mips_pic_level mips_pic;
 
-/* Warn about all NOPS that the assembler generates.  */
-static int warn_nops = 0;
-
 /* 1 if we should generate 32 bit offsets from the $gp register in
    SVR4_PIC mode.  Currently has no meaning in other modes.  */
 static int mips_big_got = 0;
@@ -4196,19 +4193,9 @@ macro (struct mips_cl_insn *ip)
 	do_false:
 	  /* result is always false */
 	  if (! likely)
-	    {
-	      if (warn_nops)
-		as_warn (_("Branch %s is always false (nop)"),
-			 ip->insn_mo->name);
-	      macro_build (NULL, &icnt, NULL, "nop", "", 0);
-	    }
+	    macro_build (NULL, &icnt, NULL, "nop", "", 0);
 	  else
-	    {
-	      if (warn_nops)
-		as_warn (_("Branch likely %s is always false"),
-			 ip->insn_mo->name);
-	      macro_build (NULL, &icnt, &offset_expr, "bnel", "s,t,p", 0, 0);
-	    }
+	    macro_build (NULL, &icnt, &offset_expr, "bnel", "s,t,p", 0, 0);
 	  return;
 	}
       if (imm_expr.X_op != O_constant)
@@ -10428,7 +10415,7 @@ static int support_64bit_objects(void)
 }
 #endif /* OBJ_ELF */
 
-const char *md_shortopts = "nO::g::G:";
+const char *md_shortopts = "O::g::G:";
 
 struct option md_longopts[] =
 {
@@ -10613,10 +10600,6 @@ md_parse_option (int c, char *arg)
       target_big_endian = 0;
       break;
 
-    case 'n':
-      warn_nops = 1;
-      break;
-
     case 'O':
       if (arg && arg[1] == '0')
 	mips_optimize = 1;
@@ -14705,7 +14688,6 @@ MIPS options:\n\
 -mfp32			use 32-bit FPRs, regardless of the chosen ISA\n\
 -O0			remove unneeded NOPs, do not swap branches\n\
 -O			remove unneeded NOPs and swap branches\n\
--n			warn about NOPs generated from macros\n\
 --[no-]construct-floats [dis]allow floating point values to be constructed\n\
 --trap, --no-break	trap exception on div by 0 and mult overflow\n\
 --break, --no-trap	break exception on div by 0 and mult overflow\n"));
diff --git a/gas/doc/as.texinfo b/gas/doc/as.texinfo
index 463e23cd68..d9d23dff59 100644
--- a/gas/doc/as.texinfo
+++ b/gas/doc/as.texinfo
@@ -1,6 +1,6 @@
 \input texinfo @c                               -*-Texinfo-*-
 @c  Copyright 1991, 1992, 1993, 1994, 1995, 1996, 1997, 1998, 1999, 2000,
-@c  2001, 2002, 2003
+@c  2001, 2002, 2003, 2004
 @c  Free Software Foundation, Inc.
 @c UPDATE!!  On future updates--
 @c   (1)   check for new machine-dep cmdline options in
@@ -350,7 +350,7 @@ gcc(1), ld(1), and the Info entries for @file{binutils} and @file{ld}.
 @ifset MIPS
 
 @emph{Target MIPS options:}
-   [@b{-nocpp}] [@b{-EL}] [@b{-EB}] [@b{-n}] [@b{-O}[@var{optimization level}]]
+   [@b{-nocpp}] [@b{-EL}] [@b{-EB}] [@b{-O}[@var{optimization level}]]
    [@b{-g}[@var{debug level}]] [@b{-G} @var{num}] [@b{-KPIC}] [@b{-call_shared}]
    [@b{-non_shared}] [@b{-xgot}] [@b{--membedded-pic}]
    [@b{-mabi}=@var{ABI}] [@b{-32}] [@b{-n32}] [@b{-64}] [@b{-mfp32}] [@b{-mgp32}]
diff --git a/gas/doc/c-mips.texi b/gas/doc/c-mips.texi
index 6f8b737a29..dbea72300b 100644
--- a/gas/doc/c-mips.texi
+++ b/gas/doc/c-mips.texi
@@ -1,4 +1,5 @@
-@c Copyright 1991, 1992, 1993, 1994, 1995, 1997, 1999, 2000, 2001, 2002, 2003
+@c Copyright 1991, 1992, 1993, 1994, 1995, 1997, 1999, 2000, 2001,
+@c 2002, 2003, 2004
 @c Free Software Foundation, Inc.
 @c This is part of the GAS manual.
 @c For copying conditions, see the file as.texinfo.
@@ -237,10 +238,6 @@ error is detected.  This is the default.
 @itemx -mno-pdr
 Control generation of @code{.pdr} sections.  Off by default on IRIX, on
 elsewhere.
-
-@item -n
-When this option is used, @code{@value{AS}} will issue a warning every
-time it generates a nop instruction from a macro.
 @end table
 
 @node MIPS Object