Patchwork Add SPARC Solaris options from specs to new sparc/sol2.opt

login
register
mail settings
Submitter Joseph S. Myers
Date Feb. 4, 2011, 11:47 p.m.
Message ID <Pine.LNX.4.64.1102042347140.5791@digraph.polyomino.org.uk>
Download mbox | patch
Permalink /patch/81983/
State New
Headers show

Comments

Joseph S. Myers - Feb. 4, 2011, 11:47 p.m.
On Fri, 4 Feb 2011, Rainer Orth wrote:

> Eric Botcazou <ebotcazou@adacore.com> writes:
> 
> >> Tested building cc1 and xgcc for cross to sparc-solaris2.8.  OK to
> >> commit?  (Or, in the alternative, OK to commit the obvious patch to
> >> remove %{sun4:} %{target:} from specs in sol2-bi.h?)
> >
> > I'd personally be in favor of the latter solution.  What do you think, Rainer?
> 
> Me too.  I couldn't find any hint of -sun4 or -target (with or without
> sun4) in not-too-ancient Studio compilers.  This stuff seems like
> ancient (as in SunOS 4) history to me, so good riddance.

Is this alternative patch OK, then?

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

	* config/sparc/sol2-bi.h (CC1_SPEC): Remove %{sun4:} %{target:}.
Eric Botcazou - Feb. 5, 2011, 8:27 a.m.
> Is this alternative patch OK, then?
>
> 2011-02-04  Joseph Myers  <joseph@codesourcery.com>
>
> 	* config/sparc/sol2-bi.h (CC1_SPEC): Remove %{sun4:} %{target:}.

Sure, thanks.

Patch

Index: gcc/config/sparc/sol2-bi.h
===================================================================
--- gcc/config/sparc/sol2-bi.h	(revision 169841)
+++ gcc/config/sparc/sol2-bi.h	(working copy)
@@ -229,14 +229,12 @@  see the files COPYING3 and COPYING.RUNTI
 #undef	CC1_SPEC
 #if DEFAULT_ARCH32_P
 #define CC1_SPEC "\
-%{sun4:} %{target:} \
 %{m64:%{m32:%emay not use both -m32 and -m64}} \
 %{m64:-mptr64 -mstack-bias -mno-v8plus \
   %{!mcpu*:-%{!mv8plus:mcpu=v9}}} \
 "
 #else
 #define CC1_SPEC "\
-%{sun4:} %{target:} \
 %{m32:%{m64:%emay not use both -m32 and -m64}} \
 %{m32:-mptr32 -mno-stack-bias \
   %{!mcpu*:%{!mv8plus:-mcpu=v9}}} \