Skip site navigation (1)Skip section navigation (2)
Date:      Thu, 10 Mar 2011 23:28:39 +0000
From:      Alexander Best <arundel@freebsd.org>
To:        Dimitry Andric <dim@FreeBSD.org>
Cc:        gerald@freebsd.org, Martin Matuska <mm@freebsd.org>, current@freebsd.org
Subject:   Re: [TESTING] ssse3 backport from gcc 4.3
Message-ID:  <20110310232839.GA5378@freebsd.org>
In-Reply-To: <20110310082740.GA74075@freebsd.org>
References:  <4D7799FD.9040008@FreeBSD.org> <20110309211820.GA46520@zim.MIT.EDU> <4D7810B2.4090207@FreeBSD.org> <20110310082740.GA74075@freebsd.org>

next in thread | previous in thread | raw e-mail | index | archive | help
On Thu Mar 10 11, Alexander Best wrote:
> 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.

i've compiled two kernels. one with the standard -mno-sse* options in kernel.mk
and one without -mno-sse2, -mno-sse3, -mno-ssse3 and -mfpmath=387.

CPUYTE ?= core2.

diffing showed no major differences for gcc and clang as compiler

cheers.
alex

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

-- 
a13x



Want to link to this message? Use this URL: <https://mail-archive.FreeBSD.org/cgi/mid.cgi?20110310232839.GA5378>