Date: Wed, 15 May 2013 13:56:55 +0000 (UTC) From: Raphael Kubo da Costa <rakuco@FreeBSD.org> To: ports-committers@freebsd.org, svn-ports-all@freebsd.org, svn-ports-head@freebsd.org Subject: svn commit: r318241 - head/graphics/qt4-svg/files Message-ID: <201305151356.r4FDutqR049931@svn.freebsd.org>
next in thread | raw e-mail | index | archive | help
Author: rakuco Date: Wed May 15 13:56:55 2013 New Revision: 318241 URL: http://svnweb.freebsd.org/changeset/ports/318241 Log: Add the patch from r317896 to qt4-svg too per miwi's request. From r318239: 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). 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. Hopefully this gets miwi happy and people on clang-i386 can build these ports again. Added: head/graphics/qt4-svg/files/ head/graphics/qt4-svg/files/patch-git_ccd1b2ee - copied unchanged from r318237, head/devel/qt4-corelib/files/patch-git_ccd1b2ee Copied: head/graphics/qt4-svg/files/patch-git_ccd1b2ee (from r318237, head/devel/qt4-corelib/files/patch-git_ccd1b2ee) ============================================================================== --- /dev/null 00:00:00 1970 (empty, because file is newly added) +++ head/graphics/qt4-svg/files/patch-git_ccd1b2ee Wed May 15 13:56:55 2013 (r318241, copy of r318237, head/devel/qt4-corelib/files/patch-git_ccd1b2ee) @@ -0,0 +1,35 @@ +commit ccd1b2ee01c193cd157978c1f9f08f87d0f9c94b +Author: Raphael Kubo da Costa <rakuco@FreeBSD.org> +Date: Fri May 10 01:40:23 2013 +0300 + + Check that 3DNow! instructions are supported before including mm3dnow.h. + + 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). + + 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. + + Not backported from qt5 because qtbase because qsimd_p.h does not include + the 3DNow! headers after commit a1b30b49ef09bef2e97b9a0622bf7ad622678fee. + + Change-Id: I15ab5e936c71c55f89c3f25777ab27fbd262e9cd + Reviewed-by: Thiago Macieira <thiago.macieira@intel.com> + +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 + + // 3D now intrinsics +-#if defined(QT_HAVE_3DNOW) ++#if defined(QT_HAVE_3DNOW) && (defined(__3dNOW__) || defined(Q_CC_MSVC)) + #include <mm3dnow.h> + #endif +
Want to link to this message? Use this URL: <https://mail-archive.FreeBSD.org/cgi/mid.cgi?201305151356.r4FDutqR049931>