Patchwork Add Solaris options from specs to sol2.opt

login
register
mail settings
Submitter Joseph S. Myers
Date Dec. 31, 2010, 3:49 p.m.
Message ID <Pine.LNX.4.64.1012311547520.16308@digraph.polyomino.org.uk>
Download mbox | patch
Permalink /patch/77088/
State New
Headers show

Comments

Joseph S. Myers - Dec. 31, 2010, 3:49 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 Solaris options matched by specs in sol2.h to
sol2.opt.

Tested building cc1 and xgcc for crosses to: i686-solaris2.10
sparc-solaris2.10.  OK to commit?

2010-12-31  Joseph Myers  <joseph@codesourcery.com>

	* config/sol2.opt (G, YP,, Ym,, compat-bsd, pthread, pthreads,
	rdynamic, threads): New Driver options.
Joseph S. Myers - Jan. 6, 2011, 12:12 a.m.
Ping.  This patch 
<http://gcc.gnu.org/ml/gcc-patches/2010-12/msg02038.html> is pending 
review.
Rainer Orth - Jan. 6, 2011, 6:37 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 Solaris options matched by specs in sol2.h to
> sol2.opt.
>
> Tested building cc1 and xgcc for crosses to: i686-solaris2.10
> sparc-solaris2.10.  OK to commit?

Looks good to me.  As was mentioned in some other thread, I'd consider
changes like this obvious.

Thanks.
	Rainer

Patch

Index: gcc/config/sol2.opt
===================================================================
--- gcc/config/sol2.opt	(revision 168378)
+++ gcc/config/sol2.opt	(working copy)
@@ -18,6 +18,30 @@ 
 ; along with GCC; see the file COPYING3.  If not see
 ; <http://www.gnu.org/licenses/>.
 
+G
+Driver
+
+YP,
+Driver Joined
+
+Ym,
+Driver Joined
+
+compat-bsd
+Driver
+
 mimpure-text
 Target Report
 Pass -z text to linker
+
+pthread
+Driver
+
+pthreads
+Driver
+
+rdynamic
+Driver
+
+threads
+Driver