Patchwork Add IRIX options from specs to new mips/iris6.opt

login
register
mail settings
Submitter Joseph S. Myers
Date Feb. 3, 2011, 8:51 p.m.
Message ID <Pine.LNX.4.64.1102032050410.18971@digraph.polyomino.org.uk>
Download mbox | patch
Permalink /patch/81716/
State New
Headers show

Comments

Joseph S. Myers - Feb. 3, 2011, 8:51 p.m.
In preparation for the driver only accepting options listed in .opt
files, and not other options that simply happen to match some spec,
this patch adds options from specs in mips/iris6.h to a new
iris6.opt.  (As noted in
<http://gcc.gnu.org/ml/gcc-patches/2011-01/msg00902.html>, I don't
actually know if all these options are IRIX linker options, but they
don't make much sense here if they aren't.)

Tested building cc1 and xgcc for cross to mips-sgi-irix6.5.  OK to
commit?

2011-02-03  Joseph Myers  <joseph@codesourcery.com>

	* config/mips/iris6.opt: New.
	* config.gcc (mips-sgi-irix6.5*): Use mips/iris6.opt.
Rainer Orth - Feb. 4, 2011, 3 p.m.
"Joseph S. Myers" <joseph@codesourcery.com> writes:

> In preparation for the driver only accepting options listed in .opt
> files, and not other options that simply happen to match some spec,
> this patch adds options from specs in mips/iris6.h to a new
> iris6.opt.  (As noted in
> <http://gcc.gnu.org/ml/gcc-patches/2011-01/msg00902.html>, I don't
> actually know if all these options are IRIX linker options, but they
> don't make much sense here if they aren't.)

They are, although -bestGnu, while accepted, isn't documented in ld(1).
Btw., what's our policy for handling linker options in the driver
directly instead of having to prefix them with -Wl,?  I'd suggest to
follow the lead of the native compiler here.

> Tested building cc1 and xgcc for cross to mips-sgi-irix6.5.  OK to
> commit?

Sure, thanks.

	Rainer

Patch

Index: gcc/config.gcc
===================================================================
--- gcc/config.gcc	(revision 169769)
+++ gcc/config.gcc	(working copy)
@@ -1817,7 +1817,7 @@  microblaze*-*-*)
 mips-sgi-irix6.5*)
 	tm_file="elfos.h ${tm_file} mips/iris6.h"
 	tmake_file="mips/t-iris mips/t-iris6 mips/t-slibgcc-irix"
-	extra_options="${extra_options} rpath.opt"
+	extra_options="${extra_options} rpath.opt mips/iris6.opt"
 	target_cpu_default="MASK_ABICALLS"
 	tm_defines="${tm_defines} MIPS_ISA_DEFAULT=3 MIPS_ABI_DEFAULT=ABI_N32"
 	# Only IRIX Development Foundation 1.3 for IRIX 6.5 provides stdint.h.
Index: gcc/config/mips/iris6.opt
===================================================================
--- gcc/config/mips/iris6.opt	(revision 0)
+++ gcc/config/mips/iris6.opt	(revision 0)
@@ -0,0 +1,45 @@ 
+; IRIX 6.5 options.
+
+; Copyright (C) 2011
+; Free Software Foundation, Inc.
+;
+; This file is part of GCC.
+;
+; GCC is free software; you can redistribute it and/or modify it under
+; the terms of the GNU General Public License as published by the Free
+; Software Foundation; either version 3, or (at your option) any later
+; version.
+;
+; GCC is distributed in the hope that it will be useful, but WITHOUT ANY
+; WARRANTY; without even the implied warranty of MERCHANTABILITY or
+; FITNESS FOR A PARTICULAR PURPOSE.  See the GNU General Public License
+; for more details.
+;
+; You should have received a copy of the GNU General Public License
+; along with GCC; see the file COPYING3.  If not see
+; <http://www.gnu.org/licenses/>.
+
+; See the GCC internals manual (options.texi) for a description of
+; this file's format.
+
+; Please try to keep this file in ASCII collating order.
+
+bestGnum
+Driver
+
+call_shared
+Driver
+
+exact_version
+Driver
+
+no_archive
+Driver
+
+non_shared
+Driver
+
+pthread
+Driver
+
+; This comment is to ensure we retain the blank line above.