From owner-freebsd-current@FreeBSD.ORG  Thu Mar 10 08:27:40 2011
Return-Path: <owner-freebsd-current@FreeBSD.ORG>
Delivered-To: current@freebsd.org
Received: by hub.freebsd.org (Postfix, from userid 1233)
	id 71AF51065676; Thu, 10 Mar 2011 08:27:40 +0000 (UTC)
Date: Thu, 10 Mar 2011 08:27:40 +0000
From: Alexander Best <arundel@freebsd.org>
To: Dimitry Andric <dim@FreeBSD.org>
Message-ID: <20110310082740.GA74075@freebsd.org>
References: <4D7799FD.9040008@FreeBSD.org> <20110309211820.GA46520@zim.MIT.EDU>
	<4D7810B2.4090207@FreeBSD.org>
Mime-Version: 1.0
Content-Type: text/plain; charset=us-ascii
Content-Disposition: inline
In-Reply-To: <4D7810B2.4090207@FreeBSD.org>
Cc: gerald@freebsd.org, Martin Matuska <mm@freebsd.org>, current@freebsd.org
Subject: Re: [TESTING] ssse3 backport from gcc 4.3
X-BeenThere: freebsd-current@freebsd.org
X-Mailman-Version: 2.1.5
Precedence: list
List-Id: Discussions about the use of FreeBSD-current
	<freebsd-current.freebsd.org>
List-Unsubscribe: <http://lists.freebsd.org/mailman/listinfo/freebsd-current>, 
	<mailto:freebsd-current-request@freebsd.org?subject=unsubscribe>
List-Archive: <http://lists.freebsd.org/pipermail/freebsd-current>
List-Post: <mailto:freebsd-current@freebsd.org>
List-Help: <mailto:freebsd-current-request@freebsd.org?subject=help>
List-Subscribe: <http://lists.freebsd.org/mailman/listinfo/freebsd-current>,
	<mailto:freebsd-current-request@freebsd.org?subject=subscribe>
X-List-Received-Date: Thu, 10 Mar 2011 08:27:40 -0000

On Thu Mar 10 11, Dimitry Andric wrote:
> On 2011-03-09 22:18, David Schultz wrote:
> >The proliferation of -mno-sse -mno-sse2 -mno-sse3 options in the
> >makefiles is probably not needed: I'm pretty sure -mno-sse implies
> >the other two, unless -msse3 is specified explicitly.
> 
> Indeed, contrib/gcc/config/i386/i386.c has:
> 
> static bool
> ix86_handle_option (size_t code, const char *arg ATTRIBUTE_UNUSED, int 
> value)
> {
> ...
>     case OPT_msse:
>       if (!value)
>         {
>           target_flags &= ~(MASK_SSE2 | MASK_SSE3);
>           target_flags_explicit |= MASK_SSE2 | MASK_SSE3;
>         }
>       return true;
> 
> E.g -mno-sse disables SSE2 and SSE3, unless you explicitly add -msse2 or
> -msse3 options.  In Martin's patch, this fragment uses MASK_SSSE3 too.

+1 for removing -mno-sse2, -mno-sse3 and -mno-ssse3 throughout the source
tree.

...also -mfpmath=387 is still present for amd64 and i don't think it's
necessary. with -mno-sse i don't think -mfpmath=sse remains the default for
amd64.

> 
> Clang has a similar mechanism, -mno-sse disables all 'higher' SSE
> variants too.

-- 
a13x