Patchwork [meta-fsl-ppc,master] Revert "e500v2: workaround for ICE of gcc-4.8.x"

login
register
mail settings
Submitter chunrong guo
Date Sept. 3, 2013, 2:58 a.m.
Message ID <B21730BB6DA1D24B83348D9301EA417E10455ACB@039-SN2MPN1-022.039d.mgd.msft.net>
Download mbox | patch
Permalink /patch/57251/
State Not Applicable
Headers show

Comments

chunrong guo - Sept. 3, 2013, 2:58 a.m.
Hello, zhenhua
Please review the patch because the below patch was applied.
"gcc-4.8: fix ICE of cross-compile for PowerPC e500v2 targets" 
http://git.yoctoproject.org/cgit/cgit.cgi/poky/commit/?id=03325fe9c51d9a198c0333da5fedd83cf833a32b


thanks,
chunrong
-----Original Message-----
From: Guo Chunrong-B40290 
Sent: Wednesday, August 28, 2013 5:30 PM
To: meta-freescale@yoctoproject.org
Cc: Liu Ting-B28495; Luo Zhenhua-B19537; Guo Chunrong-B40290; Guo Chunrong-B40290
Subject: [meta-fsl-ppc master][PATCH] Revert "e500v2: workaround for ICE of gcc-4.8.x"

  *This reverts commit 	93c57323edc118eb8905980348ad3888664f2382
   and commit efd344473c277eab8c1bae56120779983aba6176
  *The reason for reverting this is:
   the below patch fix this problem.
   the patch url is : http://patches.openembedded.org/patch/56803/

Signed-off-by: Chunrong Guo <B40290@freescale.com>
---
 conf/machine/e500v2.inc                            |    6 ---
 recipes-core/coreutils/coreutils_8.21.bbappend     |    4 --
 .../coreutils-fix-dd-segfault-for-e500v2.patch     |   37 --------------------
 3 files changed, 0 insertions(+), 47 deletions(-)  delete mode 100644 recipes-core/coreutils/coreutils_8.21.bbappend
 delete mode 100644 recipes-core/coreutils/files/coreutils-fix-dd-segfault-for-e500v2.patch

Patch

diff --git a/conf/machine/e500v2.inc b/conf/machine/e500v2.inc index aabca6e..9688784 100644
--- a/conf/machine/e500v2.inc
+++ b/conf/machine/e500v2.inc
@@ -3,12 +3,6 @@  require conf/machine/include/tune-ppce500v2.inc
 MACHINE_FEATURES = "kernel26 pci ext2 ext3 serial"
 MACHINE_EXTRA_RRECOMMENDS = "u-boot udev-extraconf"
 
-# FIXME
-# Workaround for ICE of gcc-4.8.x when passing -mfloat-gprs=double -# Bug: http://gcc.gnu.org/bugzilla/show_bug.cgi?id=57717#c1
-# Replace double-float with single-float, revert the commit when above bug is fixed -TUNE_CCARGS := "${@d.getVar('TUNE_CCARGS', True).replace('-mfloat-gprs=double', '-mfloat-gprs=single')}"
-
 PREFERRED_PROVIDER_virtual/kernel ?= "linux-qoriq-sdk"
 PREFERRED_VERSION_virtual/kernel ?= "3.0.34"
 PREFERRED_PROVIDER_linux-libc-headers ?= "linux-qoriq-sdk-headers"
diff --git a/recipes-core/coreutils/coreutils_8.21.bbappend b/recipes-core/coreutils/coreutils_8.21.bbappend
deleted file mode 100644
index 37d3ce2..0000000
--- a/recipes-core/coreutils/coreutils_8.21.bbappend
+++ /dev/null
@@ -1,4 +0,0 @@ 
-FILESEXTRAPATHS_prepend := "${THISDIR}/files:"
-
-SRC_URI_append_e500v2 = " file://coreutils-fix-dd-segfault-for-e500v2.patch \ -"
diff --git a/recipes-core/coreutils/files/coreutils-fix-dd-segfault-for-e500v2.patch b/recipes-core/coreutils/files/coreutils-fix-dd-segfault-for-e500v2.patch
deleted file mode 100644
index dbeb9b2..0000000
--- a/recipes-core/coreutils/files/coreutils-fix-dd-segfault-for-e500v2.patch
+++ /dev/null
@@ -1,37 +0,0 @@ 
-Upstream-Status: Pending
-
-Fix the segmentation fault issue when using dd on e500v2 targets, -use %f instead of $Lf, since single-float is used for e500v2 due to -an ICE of gcc-4.8.1.
-
-Signed-off-by: Zhenhua Luo <zhenhua.luo@freescale.com>
-
---- coreutils-8.21/lib/human.c.orig	2013-08-16 05:17:24.466225177 -0500
-+++ coreutils-8.21/lib/human.c	2013-08-16 05:17:52.007225123 -0500
-@@ -228,7 +228,7 @@
- 
-     if (! (opts & human_autoscale))
-       {
--        sprintf (buf, "%.0Lf", adjust_value (inexact_style, damt));
-+        sprintf (buf, "%.0f", adjust_value (inexact_style, damt));
-         buflen = strlen (buf);
-         nonintegerlen = 0;
-       }
-@@ -246,7 +246,7 @@
- 
-         damt /= e;
- 
--        sprintf (buf, "%.1Lf", adjust_value (inexact_style, damt));
-+        sprintf (buf, "%.1f", adjust_value (inexact_style, damt));
-         buflen = strlen (buf);
-         nonintegerlen = decimal_pointlen + 1;
-
-@@ -254,7 +254,7 @@
-             || ((opts & human_suppress_point_zero)
-                 && buf[buflen - 1] == '0'))
-           {
--            sprintf (buf, "%.0Lf",
-+            sprintf (buf, "%.0f",
-                      adjust_value (inexact_style, damt * 10) / 10);
-             buflen = strlen (buf);
-             nonintegerlen = 0;
--
1.7.5.4