From owner-svn-ports-head@FreeBSD.ORG Wed May 15 12:33:12 2013 Return-Path: Delivered-To: svn-ports-head@freebsd.org Received: from mx1.freebsd.org (mx1.freebsd.org [IPv6:2001:1900:2254:206a::19:1]) by hub.freebsd.org (Postfix) with ESMTP id 96170EFC; Wed, 15 May 2013 12:33:12 +0000 (UTC) (envelope-from miwi@bsdhash.org) Received: from bsdhash.org (bsdhash.org [94.23.250.27]) by mx1.freebsd.org (Postfix) with ESMTP id B604394C; Wed, 15 May 2013 12:33:11 +0000 (UTC) Received: from [192.168.0.101] (bgi-206-170.tm.net.my [202.188.206.170]) by bsdhash.org (Postfix) with ESMTPA id D572C5105E; Wed, 15 May 2013 20:33:07 +0800 (MYT) Mime-Version: 1.0 (Mac OS X Mail 6.2 \(1499\)) Subject: Re: svn commit: r318239 - in head: graphics/qt4-imageformats/files www/qt4-webkit/files x11-toolkits/qt4-gui/files x11/qt4-graphicssystems-opengl/files x11/qt4-opengl/files From: Martin Wilke In-Reply-To: <201305151140.r4FBed92003991@svn.freebsd.org> Date: Wed, 15 May 2013 20:33:02 +0800 Message-Id: References: <201305151140.r4FBed92003991@svn.freebsd.org> To: Raphael Kubo da Costa X-Mailer: Apple Mail (2.1499) Content-Type: text/plain; charset=us-ascii Content-Transfer-Encoding: quoted-printable X-Content-Filtered-By: Mailman/MimeDel 2.1.14 Cc: svn-ports-head@freebsd.org, Martin Wilke , svn-ports-all@freebsd.org, ports-committers@freebsd.org X-BeenThere: svn-ports-head@freebsd.org X-Mailman-Version: 2.1.14 Precedence: list List-Id: SVN commit messages for the ports tree for head List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , X-List-Received-Date: Wed, 15 May 2013 12:33:12 -0000 Big Thanks to all where have help to get this to work! On May 15, 2013, at 7:40 PM, Raphael Kubo da Costa = wrote: > Author: rakuco > Date: Wed May 15 11:40:38 2013 > New Revision: 318239 > URL: http://svnweb.freebsd.org/changeset/ports/318239 >=20 > Log: > Add the patch I added to qt4-corelib in r317896 to some other Qt = ports. >=20 > These ports which previously failed to build with clang-i386 use = their > private copies of the qsimd_p.h header, since this private header is = not > installed at all (which, in turns, makes the PORTREVISION bump in > qt4-corelib quite useless). >=20 > We thus need to add a copy of the same patch to all these ports to = make sure > the qsim_p.h they use when being built is actually the one we want. >=20 > Hopefully this gets miwi happy and people on clang-i386 can build = these > ports again. >=20 > Added: > head/graphics/qt4-imageformats/files/ > head/graphics/qt4-imageformats/files/patch-git_ccd1b2ee > - copied unchanged from r318237, = head/devel/qt4-corelib/files/patch-git_ccd1b2ee > head/www/qt4-webkit/files/patch-git_ccd1b2ee > - copied unchanged from r318237, = head/devel/qt4-corelib/files/patch-git_ccd1b2ee > head/x11-toolkits/qt4-gui/files/ > head/x11-toolkits/qt4-gui/files/patch-git_ccd1b2ee > - copied unchanged from r318237, = head/devel/qt4-corelib/files/patch-git_ccd1b2ee > head/x11/qt4-graphicssystems-opengl/files/ > head/x11/qt4-graphicssystems-opengl/files/patch-git_ccd1b2ee > - copied unchanged from r318237, = head/devel/qt4-corelib/files/patch-git_ccd1b2ee > head/x11/qt4-opengl/files/ > head/x11/qt4-opengl/files/patch-git_ccd1b2ee > - copied unchanged from r318237, = head/devel/qt4-corelib/files/patch-git_ccd1b2ee >=20 > Copied: head/graphics/qt4-imageformats/files/patch-git_ccd1b2ee (from = r318237, head/devel/qt4-corelib/files/patch-git_ccd1b2ee) > = =3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D= =3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D= =3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D= =3D=3D=3D > --- /dev/null 00:00:00 1970 (empty, because file is newly added) > +++ head/graphics/qt4-imageformats/files/patch-git_ccd1b2ee Wed May = 15 11:40:38 2013 (r318239, copy of r318237, = head/devel/qt4-corelib/files/patch-git_ccd1b2ee) > @@ -0,0 +1,35 @@ > +commit ccd1b2ee01c193cd157978c1f9f08f87d0f9c94b > +Author: Raphael Kubo da Costa > +Date: Fri May 10 01:40:23 2013 +0300 > + > + Check that 3DNow! instructions are supported before including = mm3dnow.h. > + =20 > + Include mm3dnow.h the way we already include the (S)SSE{3,4} and = AVX > + instruction headers: by checking that the compiler is set to = support those > + instructions besides verifying that QT_HAVE_3DNOW evalues to true = (which > + only means the compiler supported them when the build was being = configured). > + =20 > + This has not caused a problem in most cases so far because GCC = protects its > + mm3dnow.h header with an #ifdef __3dNOW__ (contrary to what it = does on, say, > + pmmintrin.h). clang's mm3dnow.h, on the other hand, does not have = that check > + and can fail if -march is set to, for example, i386 or i486. > + =20 > + Not backported from qt5 because qtbase because qsimd_p.h does not = include > + the 3DNow! headers after commit = a1b30b49ef09bef2e97b9a0622bf7ad622678fee. > + =20 > + Change-Id: I15ab5e936c71c55f89c3f25777ab27fbd262e9cd > + Reviewed-by: Thiago Macieira > + > +diff --git a/src/corelib/tools/qsimd_p.h = b/src/corelib/tools/qsimd_p.h > +index 87e26e1..6a32e8f 100644 > +--- ./src/corelib/tools/qsimd_p.h > ++++ ./src/corelib/tools/qsimd_p.h > +@@ -199,7 +199,7 @@ QT_BEGIN_HEADER > + #endif > +=20 > + // 3D now intrinsics > +-#if defined(QT_HAVE_3DNOW) > ++#if defined(QT_HAVE_3DNOW) && (defined(__3dNOW__) || = defined(Q_CC_MSVC)) > + #include > + #endif > +=20 >=20 > Copied: head/www/qt4-webkit/files/patch-git_ccd1b2ee (from r318237, = head/devel/qt4-corelib/files/patch-git_ccd1b2ee) > = =3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D= =3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D= =3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D= =3D=3D=3D > --- /dev/null 00:00:00 1970 (empty, because file is newly added) > +++ head/www/qt4-webkit/files/patch-git_ccd1b2ee Wed May 15 = 11:40:38 2013 (r318239, copy of r318237, = head/devel/qt4-corelib/files/patch-git_ccd1b2ee) > @@ -0,0 +1,35 @@ > +commit ccd1b2ee01c193cd157978c1f9f08f87d0f9c94b > +Author: Raphael Kubo da Costa > +Date: Fri May 10 01:40:23 2013 +0300 > + > + Check that 3DNow! instructions are supported before including = mm3dnow.h. > + =20 > + Include mm3dnow.h the way we already include the (S)SSE{3,4} and = AVX > + instruction headers: by checking that the compiler is set to = support those > + instructions besides verifying that QT_HAVE_3DNOW evalues to true = (which > + only means the compiler supported them when the build was being = configured). > + =20 > + This has not caused a problem in most cases so far because GCC = protects its > + mm3dnow.h header with an #ifdef __3dNOW__ (contrary to what it = does on, say, > + pmmintrin.h). clang's mm3dnow.h, on the other hand, does not have = that check > + and can fail if -march is set to, for example, i386 or i486. > + =20 > + Not backported from qt5 because qtbase because qsimd_p.h does not = include > + the 3DNow! headers after commit = a1b30b49ef09bef2e97b9a0622bf7ad622678fee. > + =20 > + Change-Id: I15ab5e936c71c55f89c3f25777ab27fbd262e9cd > + Reviewed-by: Thiago Macieira > + > +diff --git a/src/corelib/tools/qsimd_p.h = b/src/corelib/tools/qsimd_p.h > +index 87e26e1..6a32e8f 100644 > +--- ./src/corelib/tools/qsimd_p.h > ++++ ./src/corelib/tools/qsimd_p.h > +@@ -199,7 +199,7 @@ QT_BEGIN_HEADER > + #endif > +=20 > + // 3D now intrinsics > +-#if defined(QT_HAVE_3DNOW) > ++#if defined(QT_HAVE_3DNOW) && (defined(__3dNOW__) || = defined(Q_CC_MSVC)) > + #include > + #endif > +=20 >=20 > Copied: head/x11-toolkits/qt4-gui/files/patch-git_ccd1b2ee (from = r318237, head/devel/qt4-corelib/files/patch-git_ccd1b2ee) > = =3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D= =3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D= =3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D= =3D=3D=3D > --- /dev/null 00:00:00 1970 (empty, because file is newly added) > +++ head/x11-toolkits/qt4-gui/files/patch-git_ccd1b2ee Wed May = 15 11:40:38 2013 (r318239, copy of r318237, = head/devel/qt4-corelib/files/patch-git_ccd1b2ee) > @@ -0,0 +1,35 @@ > +commit ccd1b2ee01c193cd157978c1f9f08f87d0f9c94b > +Author: Raphael Kubo da Costa > +Date: Fri May 10 01:40:23 2013 +0300 > + > + Check that 3DNow! instructions are supported before including = mm3dnow.h. > + =20 > + Include mm3dnow.h the way we already include the (S)SSE{3,4} and = AVX > + instruction headers: by checking that the compiler is set to = support those > + instructions besides verifying that QT_HAVE_3DNOW evalues to true = (which > + only means the compiler supported them when the build was being = configured). > + =20 > + This has not caused a problem in most cases so far because GCC = protects its > + mm3dnow.h header with an #ifdef __3dNOW__ (contrary to what it = does on, say, > + pmmintrin.h). clang's mm3dnow.h, on the other hand, does not have = that check > + and can fail if -march is set to, for example, i386 or i486. > + =20 > + Not backported from qt5 because qtbase because qsimd_p.h does not = include > + the 3DNow! headers after commit = a1b30b49ef09bef2e97b9a0622bf7ad622678fee. > + =20 > + Change-Id: I15ab5e936c71c55f89c3f25777ab27fbd262e9cd > + Reviewed-by: Thiago Macieira > + > +diff --git a/src/corelib/tools/qsimd_p.h = b/src/corelib/tools/qsimd_p.h > +index 87e26e1..6a32e8f 100644 > +--- ./src/corelib/tools/qsimd_p.h > ++++ ./src/corelib/tools/qsimd_p.h > +@@ -199,7 +199,7 @@ QT_BEGIN_HEADER > + #endif > +=20 > + // 3D now intrinsics > +-#if defined(QT_HAVE_3DNOW) > ++#if defined(QT_HAVE_3DNOW) && (defined(__3dNOW__) || = defined(Q_CC_MSVC)) > + #include > + #endif > +=20 >=20 > Copied: head/x11/qt4-graphicssystems-opengl/files/patch-git_ccd1b2ee = (from r318237, head/devel/qt4-corelib/files/patch-git_ccd1b2ee) > = =3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D= =3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D= =3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D= =3D=3D=3D > --- /dev/null 00:00:00 1970 (empty, because file is newly added) > +++ head/x11/qt4-graphicssystems-opengl/files/patch-git_ccd1b2ee = Wed May 15 11:40:38 2013 (r318239, copy of r318237, = head/devel/qt4-corelib/files/patch-git_ccd1b2ee) > @@ -0,0 +1,35 @@ > +commit ccd1b2ee01c193cd157978c1f9f08f87d0f9c94b > +Author: Raphael Kubo da Costa > +Date: Fri May 10 01:40:23 2013 +0300 > + > + Check that 3DNow! instructions are supported before including = mm3dnow.h. > + =20 > + Include mm3dnow.h the way we already include the (S)SSE{3,4} and = AVX > + instruction headers: by checking that the compiler is set to = support those > + instructions besides verifying that QT_HAVE_3DNOW evalues to true = (which > + only means the compiler supported them when the build was being = configured). > + =20 > + This has not caused a problem in most cases so far because GCC = protects its > + mm3dnow.h header with an #ifdef __3dNOW__ (contrary to what it = does on, say, > + pmmintrin.h). clang's mm3dnow.h, on the other hand, does not have = that check > + and can fail if -march is set to, for example, i386 or i486. > + =20 > + Not backported from qt5 because qtbase because qsimd_p.h does not = include > + the 3DNow! headers after commit = a1b30b49ef09bef2e97b9a0622bf7ad622678fee. > + =20 > + Change-Id: I15ab5e936c71c55f89c3f25777ab27fbd262e9cd > + Reviewed-by: Thiago Macieira > + > +diff --git a/src/corelib/tools/qsimd_p.h = b/src/corelib/tools/qsimd_p.h > +index 87e26e1..6a32e8f 100644 > +--- ./src/corelib/tools/qsimd_p.h > ++++ ./src/corelib/tools/qsimd_p.h > +@@ -199,7 +199,7 @@ QT_BEGIN_HEADER > + #endif > +=20 > + // 3D now intrinsics > +-#if defined(QT_HAVE_3DNOW) > ++#if defined(QT_HAVE_3DNOW) && (defined(__3dNOW__) || = defined(Q_CC_MSVC)) > + #include > + #endif > +=20 >=20 > Copied: head/x11/qt4-opengl/files/patch-git_ccd1b2ee (from r318237, = head/devel/qt4-corelib/files/patch-git_ccd1b2ee) > = =3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D= =3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D= =3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D= =3D=3D=3D > --- /dev/null 00:00:00 1970 (empty, because file is newly added) > +++ head/x11/qt4-opengl/files/patch-git_ccd1b2ee Wed May 15 = 11:40:38 2013 (r318239, copy of r318237, = head/devel/qt4-corelib/files/patch-git_ccd1b2ee) > @@ -0,0 +1,35 @@ > +commit ccd1b2ee01c193cd157978c1f9f08f87d0f9c94b > +Author: Raphael Kubo da Costa > +Date: Fri May 10 01:40:23 2013 +0300 > + > + Check that 3DNow! instructions are supported before including = mm3dnow.h. > + =20 > + Include mm3dnow.h the way we already include the (S)SSE{3,4} and = AVX > + instruction headers: by checking that the compiler is set to = support those > + instructions besides verifying that QT_HAVE_3DNOW evalues to true = (which > + only means the compiler supported them when the build was being = configured). > + =20 > + This has not caused a problem in most cases so far because GCC = protects its > + mm3dnow.h header with an #ifdef __3dNOW__ (contrary to what it = does on, say, > + pmmintrin.h). clang's mm3dnow.h, on the other hand, does not have = that check > + and can fail if -march is set to, for example, i386 or i486. > + =20 > + Not backported from qt5 because qtbase because qsimd_p.h does not = include > + the 3DNow! headers after commit = a1b30b49ef09bef2e97b9a0622bf7ad622678fee. > + =20 > + Change-Id: I15ab5e936c71c55f89c3f25777ab27fbd262e9cd > + Reviewed-by: Thiago Macieira > + > +diff --git a/src/corelib/tools/qsimd_p.h = b/src/corelib/tools/qsimd_p.h > +index 87e26e1..6a32e8f 100644 > +--- ./src/corelib/tools/qsimd_p.h > ++++ ./src/corelib/tools/qsimd_p.h > +@@ -199,7 +199,7 @@ QT_BEGIN_HEADER > + #endif > +=20 > + // 3D now intrinsics > +-#if defined(QT_HAVE_3DNOW) > ++#if defined(QT_HAVE_3DNOW) && (defined(__3dNOW__) || = defined(Q_CC_MSVC)) > + #include > + #endif > +=20 >=20 +-----------------oOO--(_)--OOo-------------------------+ With best Regards, Martin Wilke (miwi_(at)_FreeBSD.org) Mess with the Best, Die like the Rest