Patchwork cc/gcc: add patch for 4.7.2

login
register
mail settings
Submitter Richard Braun
Date Nov. 21, 2012, 12:41 p.m.
Message ID <20121121124134.GA22678@mail.sceen.net>
Download mbox | patch
Permalink /patch/200675/
State Changes Requested
Headers show

Comments

Florian Fainelli - Nov. 21, 2012, 1:13 p.m.
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
Richard Braun - Nov. 21, 2012, 1:48 p.m.
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.
Yann E. MORIN - Nov. 23, 2012, 5:33 p.m.
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.

Patch

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