Message ID | 20121121124134.GA22678@mail.sceen.net |
---|---|
State | Changes Requested |
Headers | show |
Hello Richard, On Wednesday 21 November 2012 13:41:34 Richard Braun wrote: > > Do you have a link to the PR on GCC's bugzilla if any? It is always useful so one can know whether this patch should be forward ported to a new version. Thank you! -- Florian -- For unsubscribe information see http://sourceware.org/lists.html#faq
On Wed, Nov 21, 2012 at 02:13:57PM +0100, Florian Fainelli wrote: > Do you have a link to the PR on GCC's bugzilla if any? It is always useful so > one can know whether this patch should be forward ported to a new version. Right, sorry. Here is the link: http://gcc.gnu.org/bugzilla/show_bug.cgi?id=52695 Upstream hasn't yet fixed it, but comments in the report in addition to my experience here indicate the patch is very likely OK.
Richard, All,
On Wednesday 21 November 2012 Richard Braun wrote:
> (nothing)
This patch is missing you Signed-off-by line.
While you are at it, please add the BZ URL in the ptch body.
Also, could you use 'hg send-email' to send your patches, instead of
attaching them, please? It makes it easier to review.
Regards,
Yann E. MORIN.
exporting patch: # HG changeset patch # User Richard Braun <rbraun@sceen.net> # Date 1353500353 -3600 # Node ID c777aafa8c093c723ee317fdd5ca0c35ebe29790 # Parent e097000b5420234168203460dae6ee13560e565e cc/gcc: add patch for 4.7.2 This patch fixes compilation of gcc when C++ is enabled and MMX is available, but not SSE/SSE2/AVX. diff -r e097000b5420 -r c777aafa8c09 patches/gcc/4.7.2/000-libitm-fix-definition-of-__m64.patch --- /dev/null Thu Jan 01 00:00:00 1970 +0000 +++ b/patches/gcc/4.7.2/000-libitm-fix-definition-of-__m64.patch Wed Nov 21 13:19:13 2012 +0100 @@ -0,0 +1,26 @@ +From fd81a24e5a46badfaca606be0781e097aa3cb2e6 Mon Sep 17 00:00:00 2001 +From: Richard Braun <rbraun@sceen.net> +Date: Wed, 21 Nov 2012 11:58:16 +0100 +Subject: [PATCH] libitm: fix definition of __m64 + +--- + libitm/config/x86/target.h | 3 +++ + 1 files changed, 3 insertions(+), 0 deletions(-) + +diff --git a/libitm/config/x86/target.h b/libitm/config/x86/target.h +index 5c7e6fb..6254d8a 100644 +--- a/libitm/config/x86/target.h ++++ b/libitm/config/x86/target.h +@@ -73,6 +73,9 @@ cpu_relax (void) + /* ??? It's broken for C++. */ + #include <x86intrin.h> + #else ++# ifdef __MMX__ ++# include <mmintrin.h> ++# endif + # ifdef __SSE2__ + # include <emmintrin.h> + # elif defined(__SSE__) +-- +1.7.2.5 + -- For unsubscribe information see http://sourceware.org/lists.html#faq