From owner-freebsd-x11@FreeBSD.ORG  Tue Mar  4 17:07:40 2014
Return-Path: <owner-freebsd-x11@FreeBSD.ORG>
Delivered-To: x11@FreeBSD.org
Received: by hub.freebsd.org (Postfix, from userid 1033)
 id 5E794F6D; Tue,  4 Mar 2014 17:07:40 +0000 (UTC)
Date: Tue, 4 Mar 2014 17:07:40 +0000
From: Alexey Dokuchaev <danfe@FreeBSD.org>
To: Tijl Coosemans <tijl@FreeBSD.org>
Subject: Re: svn commit: r342587 - head/x11/pixman
Message-ID: <20140304170740.GA82945@FreeBSD.org>
References: <201402041839.s14IdgAF097754@svn.freebsd.org>
 <20140204210959.044bc79a@kalimero.tijl.coosemans.org>
 <20140205121525.GB24764@FreeBSD.org>
MIME-Version: 1.0
Content-Type: text/plain; charset=us-ascii
Content-Disposition: inline
In-Reply-To: <20140205121525.GB24764@FreeBSD.org>
User-Agent: Mutt/1.5.22 (2013-10-16)
Cc: Koop Mast <kwm@FreeBSD.org>, x11@FreeBSD.org
X-BeenThere: freebsd-x11@freebsd.org
X-Mailman-Version: 2.1.17
Precedence: list
List-Id: X11 on FreeBSD -- maintaining and support <freebsd-x11.freebsd.org>
List-Unsubscribe: <http://lists.freebsd.org/mailman/options/freebsd-x11>,
 <mailto:freebsd-x11-request@freebsd.org?subject=unsubscribe>
List-Archive: <http://lists.freebsd.org/pipermail/freebsd-x11/>
List-Post: <mailto:freebsd-x11@freebsd.org>
List-Help: <mailto:freebsd-x11-request@freebsd.org?subject=help>
List-Subscribe: <http://lists.freebsd.org/mailman/listinfo/freebsd-x11>,
 <mailto:freebsd-x11-request@freebsd.org?subject=subscribe>
X-List-Received-Date: Tue, 04 Mar 2014 17:07:40 -0000

On Wed, Feb 05, 2014 at 12:15:25PM +0000, Alexey Dokuchaev wrote:
> On Tue, Feb 04, 2014 at 09:09:59PM +0100, Tijl Coosemans wrote:
> > The is a bit strange because if you turn off SIMD you still get MMX and
> > SSE2 support if your cpu supports it.  It should be something like this:
> > 
> > .if ! ${PORT_OPTIONS:MSIMD}
> > CONFIGURE_ARGS=	--disable-vmx --disable-arm-simd --disable-mmx \
> > 		--disable-sse2
> > .endif
> > 
> > But then you're still missing --disable-ssse3 and a number of ARM and
> > MIPS SIMD options (reported by ./configure --help).
> > 
> > I think it's best to just remove this whole block together with the
> > SIMD option because pixman tests at runtime what features the cpu
> > supports so in my opinion there's no reason to disable SIMD support.
> 
> +1.  I've always had to read inside the Makefile to understand what does
> this option do; it's very confusing up-front.  Please remove it and/or
> hide the logic behind the trunk, so it DTRT without annoying users with
> cryptic dialogs.

Do we have any consensus on this one now?  Can we just remove the whole
SIMD logic from the Makefile?

./danfe