Openembedded Core Discussions
 help / color / mirror / Atom feed
* [PATCH] gcc: Upgrade to 4.8.1
@ 2013-06-11 21:57 Khem Raj
  2013-06-11 22:31 ` [RFC][PATCH] gcc-4.8: Backport fix for ICE triggered in pixman Martin Jansa
                   ` (2 more replies)
  0 siblings, 3 replies; 11+ messages in thread
From: Khem Raj @ 2013-06-11 21:57 UTC (permalink / raw)
  To: openembedded-core

Signed-off-by: Khem Raj <raj.khem@gmail.com>
---
 meta/recipes-devtools/gcc/gcc-4.8.inc |    8 ++++----
 1 file changed, 4 insertions(+), 4 deletions(-)

diff --git a/meta/recipes-devtools/gcc/gcc-4.8.inc b/meta/recipes-devtools/gcc/gcc-4.8.inc
index 80bd02c..6664f1c 100644
--- a/meta/recipes-devtools/gcc/gcc-4.8.inc
+++ b/meta/recipes-devtools/gcc/gcc-4.8.inc
@@ -6,7 +6,7 @@ require gcc-common.inc
 # on branch then PV should be incremented to 4.7.2+svnr${SRCPV}
 # to reflect that change
 
-PV = "4.8.0"
+PV = "4.8.1"
 
 # BINV should be incremented after updating to a revision
 # after a minor gcc release (e.g. 4.7.1 or 4.7.2) has been made
@@ -14,7 +14,7 @@ PV = "4.8.0"
 # 4.7.1 then the value below will have 2 which will mean 4.7.2
 # which will be next minor release and so on.
 
-BINV = "4.8.0"
+BINV = "4.8.1"
 
 FILESPATH = "${@base_set_filespath([ '${FILE_DIRNAME}/gcc-4.8' ], d)}"
 
@@ -68,8 +68,8 @@ SRC_URI = "${GNU_MIRROR}/gcc/gcc-${PV}/gcc-${PV}.tar.bz2 \
 	   file://gcc-4.8-PR56797.patch \
 	   file://gcc-4.8-build-args.patch \
 	  "
-SRC_URI[md5sum] = "e6040024eb9e761c3bea348d1fa5abb0"
-SRC_URI[sha256sum] = "b037fe5132b71ecad2ea7141ec92292b5d32427bf90fd90cde432b1d5abacc2c"
+SRC_URI[md5sum] = "3b2386c114cd74185aa3754b58a79304"
+SRC_URI[sha256sum] = "545b44be3ad9f2c4e90e6880f5c9d4f0a8f0e5f67e1ffb0d45da9fa01bb05813"
 
 S = "${TMPDIR}/work-shared/gcc-${PV}-${PR}/gcc-${PV}"
 B = "${WORKDIR}/gcc-${PV}/build.${HOST_SYS}.${TARGET_SYS}"
-- 
1.7.9.5



^ permalink raw reply related	[flat|nested] 11+ messages in thread

* [RFC][PATCH] gcc-4.8: Backport fix for ICE triggered in pixman
  2013-06-11 21:57 [PATCH] gcc: Upgrade to 4.8.1 Khem Raj
@ 2013-06-11 22:31 ` Martin Jansa
  2013-06-11 22:35   ` Martin Jansa
  2013-06-13 17:12 ` [PATCH] gcc: Upgrade to 4.8.1 Saul Wold
  2013-06-28 23:47 ` Jonathan Liu
  2 siblings, 1 reply; 11+ messages in thread
From: Martin Jansa @ 2013-06-11 22:31 UTC (permalink / raw)
  To: openembedded-core

* http://gcc.gnu.org/bugzilla/show_bug.cgi?id=57329

Signed-off-by: Martin Jansa <Martin.Jansa@gmail.com>
---
 meta/recipes-devtools/gcc/gcc-4.8.inc              |  1 +
 .../gcc/gcc-4.8/0036-PR-target-56102.patch         | 83 ++++++++++++++++++++++
 2 files changed, 84 insertions(+)
 create mode 100644 meta/recipes-devtools/gcc/gcc-4.8/0036-PR-target-56102.patch

diff --git a/meta/recipes-devtools/gcc/gcc-4.8.inc b/meta/recipes-devtools/gcc/gcc-4.8.inc
index 6664f1c..bfdb766 100644
--- a/meta/recipes-devtools/gcc/gcc-4.8.inc
+++ b/meta/recipes-devtools/gcc/gcc-4.8.inc
@@ -65,6 +65,7 @@ SRC_URI = "${GNU_MIRROR}/gcc/gcc-${PV}/gcc-${PV}.tar.bz2 \
 	   file://0033-gcc-armv4-pass-fix-v4bx-to-linker-to-support-EABI.patch \
 	   file://0034-Use-the-multilib-config-files-from-B-instead-of-usin.patch \
 	   file://0035-wcast-qual-PR-55383.patch \
+	   file://0036-PR-target-56102.patch \
 	   file://gcc-4.8-PR56797.patch \
 	   file://gcc-4.8-build-args.patch \
 	  "
diff --git a/meta/recipes-devtools/gcc/gcc-4.8/0036-PR-target-56102.patch b/meta/recipes-devtools/gcc/gcc-4.8/0036-PR-target-56102.patch
new file mode 100644
index 0000000..a9498fc
--- /dev/null
+++ b/meta/recipes-devtools/gcc/gcc-4.8/0036-PR-target-56102.patch
@@ -0,0 +1,83 @@
+From fa049b3584a1cc36c250205e3d5841e6a40ff677 Mon Sep 17 00:00:00 2001
+From: amker <amker@138bc75d-0d04-0410-961f-82ee72b054a4>
+Date: Wed, 27 Mar 2013 08:16:54 +0000
+Subject: [PATCH] PR target/56102
+
+* config/arm/arm.c (thumb1_rtx_costs, thumb1_size_rtx_costs): Fix rtx costs for SET/ASHIFT/ASHIFTRT/LSHIFTRT/ROTATERT patterns with mult-word mode.
+
+git-svn-id: svn+ssh://gcc.gnu.org/svn/gcc/trunk@197155 138bc75d-0d04-0410-961f-82ee72b054a4
+
+Upstream-Status: Backport
+http://gcc.gnu.org/bugzilla/show_bug.cgi?id=57329
+
+Signed-off-by: Martin Jansa <Martin.Jansa@gmail.com>
+---
+ gcc/config/arm/arm.c | 17 ++++++++++++++---
+ 1 file changed, 14 insertions(+), 3 deletions(-)
+
+diff --git a/gcc/config/arm/arm.c b/gcc/config/arm/arm.c
+index 460d333..edf850d 100644
+--- a/gcc/config/arm/arm.c
++++ b/gcc/config/arm/arm.c
+@@ -7094,7 +7094,7 @@ static inline int
+ thumb1_rtx_costs (rtx x, enum rtx_code code, enum rtx_code outer)
+ {
+   enum machine_mode mode = GET_MODE (x);
+-  int total;
++  int total, words;
+ 
+   switch (code)
+     {
+@@ -7102,6 +7102,8 @@ thumb1_rtx_costs (rtx x, enum rtx_code code, enum rtx_code outer)
+     case ASHIFTRT:
+     case LSHIFTRT:
+     case ROTATERT:
++      return (mode == SImode) ? COSTS_N_INSNS (1) : COSTS_N_INSNS (2);
++
+     case PLUS:
+     case MINUS:
+     case COMPARE:
+@@ -7125,7 +7127,10 @@ thumb1_rtx_costs (rtx x, enum rtx_code code, enum rtx_code outer)
+       return COSTS_N_INSNS (1) + 16;
+ 
+     case SET:
+-      return (COSTS_N_INSNS (1)
++      /* A SET doesn't have a mode, so let's look at the SET_DEST to get
++	 the mode.  */
++      words = ARM_NUM_INTS (GET_MODE_SIZE (GET_MODE (SET_DEST (x))));
++      return (COSTS_N_INSNS (words)
+ 	      + 4 * ((MEM_P (SET_SRC (x)))
+ 		     + MEM_P (SET_DEST (x))));
+ 
+@@ -7822,6 +7827,7 @@ static inline int
+ thumb1_size_rtx_costs (rtx x, enum rtx_code code, enum rtx_code outer)
+ {
+   enum machine_mode mode = GET_MODE (x);
++  int words;
+ 
+   switch (code)
+     {
+@@ -7829,6 +7835,8 @@ thumb1_size_rtx_costs (rtx x, enum rtx_code code, enum rtx_code outer)
+     case ASHIFTRT:
+     case LSHIFTRT:
+     case ROTATERT:
++      return (mode == SImode) ? COSTS_N_INSNS (1) : COSTS_N_INSNS (2);
++
+     case PLUS:
+     case MINUS:
+     case COMPARE:
+@@ -7847,7 +7855,10 @@ thumb1_size_rtx_costs (rtx x, enum rtx_code code, enum rtx_code outer)
+       return COSTS_N_INSNS (1);
+ 
+     case SET:
+-      return (COSTS_N_INSNS (1)
++      /* A SET doesn't have a mode, so let's look at the SET_DEST to get
++	 the mode.  */
++      words = ARM_NUM_INTS (GET_MODE_SIZE (GET_MODE (SET_DEST (x))));
++      return (COSTS_N_INSNS (words)
+               + 4 * ((MEM_P (SET_SRC (x)))
+                      + MEM_P (SET_DEST (x))));
+ 
+-- 
+1.8.2.1
+
-- 
1.8.2.1



^ permalink raw reply related	[flat|nested] 11+ messages in thread

* Re: [RFC][PATCH] gcc-4.8: Backport fix for ICE triggered in pixman
  2013-06-11 22:31 ` [RFC][PATCH] gcc-4.8: Backport fix for ICE triggered in pixman Martin Jansa
@ 2013-06-11 22:35   ` Martin Jansa
  2013-06-11 22:36     ` Khem Raj
  2013-06-11 22:39     ` Khem Raj
  0 siblings, 2 replies; 11+ messages in thread
From: Martin Jansa @ 2013-06-11 22:35 UTC (permalink / raw)
  To: openembedded-core

[-- Attachment #1: Type: text/plain, Size: 4674 bytes --]

On Wed, Jun 12, 2013 at 12:31:16AM +0200, Martin Jansa wrote:
> * http://gcc.gnu.org/bugzilla/show_bug.cgi?id=57329

Only RFC, because world build is still running to confirm that it
doesn't introduce new issues and that it indeed fixes pixman build for
xscalete.

> 
> Signed-off-by: Martin Jansa <Martin.Jansa@gmail.com>
> ---
>  meta/recipes-devtools/gcc/gcc-4.8.inc              |  1 +
>  .../gcc/gcc-4.8/0036-PR-target-56102.patch         | 83 ++++++++++++++++++++++
>  2 files changed, 84 insertions(+)
>  create mode 100644 meta/recipes-devtools/gcc/gcc-4.8/0036-PR-target-56102.patch
> 
> diff --git a/meta/recipes-devtools/gcc/gcc-4.8.inc b/meta/recipes-devtools/gcc/gcc-4.8.inc
> index 6664f1c..bfdb766 100644
> --- a/meta/recipes-devtools/gcc/gcc-4.8.inc
> +++ b/meta/recipes-devtools/gcc/gcc-4.8.inc
> @@ -65,6 +65,7 @@ SRC_URI = "${GNU_MIRROR}/gcc/gcc-${PV}/gcc-${PV}.tar.bz2 \
>  	   file://0033-gcc-armv4-pass-fix-v4bx-to-linker-to-support-EABI.patch \
>  	   file://0034-Use-the-multilib-config-files-from-B-instead-of-usin.patch \
>  	   file://0035-wcast-qual-PR-55383.patch \
> +	   file://0036-PR-target-56102.patch \
>  	   file://gcc-4.8-PR56797.patch \
>  	   file://gcc-4.8-build-args.patch \
>  	  "
> diff --git a/meta/recipes-devtools/gcc/gcc-4.8/0036-PR-target-56102.patch b/meta/recipes-devtools/gcc/gcc-4.8/0036-PR-target-56102.patch
> new file mode 100644
> index 0000000..a9498fc
> --- /dev/null
> +++ b/meta/recipes-devtools/gcc/gcc-4.8/0036-PR-target-56102.patch
> @@ -0,0 +1,83 @@
> +From fa049b3584a1cc36c250205e3d5841e6a40ff677 Mon Sep 17 00:00:00 2001
> +From: amker <amker@138bc75d-0d04-0410-961f-82ee72b054a4>
> +Date: Wed, 27 Mar 2013 08:16:54 +0000
> +Subject: [PATCH] PR target/56102
> +
> +* config/arm/arm.c (thumb1_rtx_costs, thumb1_size_rtx_costs): Fix rtx costs for SET/ASHIFT/ASHIFTRT/LSHIFTRT/ROTATERT patterns with mult-word mode.
> +
> +git-svn-id: svn+ssh://gcc.gnu.org/svn/gcc/trunk@197155 138bc75d-0d04-0410-961f-82ee72b054a4
> +
> +Upstream-Status: Backport
> +http://gcc.gnu.org/bugzilla/show_bug.cgi?id=57329
> +
> +Signed-off-by: Martin Jansa <Martin.Jansa@gmail.com>
> +---
> + gcc/config/arm/arm.c | 17 ++++++++++++++---
> + 1 file changed, 14 insertions(+), 3 deletions(-)
> +
> +diff --git a/gcc/config/arm/arm.c b/gcc/config/arm/arm.c
> +index 460d333..edf850d 100644
> +--- a/gcc/config/arm/arm.c
> ++++ b/gcc/config/arm/arm.c
> +@@ -7094,7 +7094,7 @@ static inline int
> + thumb1_rtx_costs (rtx x, enum rtx_code code, enum rtx_code outer)
> + {
> +   enum machine_mode mode = GET_MODE (x);
> +-  int total;
> ++  int total, words;
> + 
> +   switch (code)
> +     {
> +@@ -7102,6 +7102,8 @@ thumb1_rtx_costs (rtx x, enum rtx_code code, enum rtx_code outer)
> +     case ASHIFTRT:
> +     case LSHIFTRT:
> +     case ROTATERT:
> ++      return (mode == SImode) ? COSTS_N_INSNS (1) : COSTS_N_INSNS (2);
> ++
> +     case PLUS:
> +     case MINUS:
> +     case COMPARE:
> +@@ -7125,7 +7127,10 @@ thumb1_rtx_costs (rtx x, enum rtx_code code, enum rtx_code outer)
> +       return COSTS_N_INSNS (1) + 16;
> + 
> +     case SET:
> +-      return (COSTS_N_INSNS (1)
> ++      /* A SET doesn't have a mode, so let's look at the SET_DEST to get
> ++	 the mode.  */
> ++      words = ARM_NUM_INTS (GET_MODE_SIZE (GET_MODE (SET_DEST (x))));
> ++      return (COSTS_N_INSNS (words)
> + 	      + 4 * ((MEM_P (SET_SRC (x)))
> + 		     + MEM_P (SET_DEST (x))));
> + 
> +@@ -7822,6 +7827,7 @@ static inline int
> + thumb1_size_rtx_costs (rtx x, enum rtx_code code, enum rtx_code outer)
> + {
> +   enum machine_mode mode = GET_MODE (x);
> ++  int words;
> + 
> +   switch (code)
> +     {
> +@@ -7829,6 +7835,8 @@ thumb1_size_rtx_costs (rtx x, enum rtx_code code, enum rtx_code outer)
> +     case ASHIFTRT:
> +     case LSHIFTRT:
> +     case ROTATERT:
> ++      return (mode == SImode) ? COSTS_N_INSNS (1) : COSTS_N_INSNS (2);
> ++
> +     case PLUS:
> +     case MINUS:
> +     case COMPARE:
> +@@ -7847,7 +7855,10 @@ thumb1_size_rtx_costs (rtx x, enum rtx_code code, enum rtx_code outer)
> +       return COSTS_N_INSNS (1);
> + 
> +     case SET:
> +-      return (COSTS_N_INSNS (1)
> ++      /* A SET doesn't have a mode, so let's look at the SET_DEST to get
> ++	 the mode.  */
> ++      words = ARM_NUM_INTS (GET_MODE_SIZE (GET_MODE (SET_DEST (x))));
> ++      return (COSTS_N_INSNS (words)
> +               + 4 * ((MEM_P (SET_SRC (x)))
> +                      + MEM_P (SET_DEST (x))));
> + 
> +-- 
> +1.8.2.1
> +
> -- 
> 1.8.2.1
> 

-- 
Martin 'JaMa' Jansa     jabber: Martin.Jansa@gmail.com

[-- Attachment #2: Digital signature --]
[-- Type: application/pgp-signature, Size: 205 bytes --]

^ permalink raw reply	[flat|nested] 11+ messages in thread

* Re: [RFC][PATCH] gcc-4.8: Backport fix for ICE triggered in pixman
  2013-06-11 22:35   ` Martin Jansa
@ 2013-06-11 22:36     ` Khem Raj
  2013-06-11 22:39     ` Khem Raj
  1 sibling, 0 replies; 11+ messages in thread
From: Khem Raj @ 2013-06-11 22:36 UTC (permalink / raw)
  To: Martin Jansa; +Cc: openembedded-core


On Jun 11, 2013, at 3:35 PM, Martin Jansa <martin.jansa@gmail.com> wrote:

> On Wed, Jun 12, 2013 at 12:31:16AM +0200, Martin Jansa wrote:
>> * http://gcc.gnu.org/bugzilla/show_bug.cgi?id=57329
> 
> Only RFC, because world build is still running to confirm that it
> doesn't introduce new issues and that it indeed fixes pixman build for
> xscalete.
> 

yes I would love more testing. For my own testing on all qemus it did not make anything worse than it was with 4.8.0


>> 
>> Signed-off-by: Martin Jansa <Martin.Jansa@gmail.com>
>> ---
>> meta/recipes-devtools/gcc/gcc-4.8.inc              |  1 +
>> .../gcc/gcc-4.8/0036-PR-target-56102.patch         | 83 ++++++++++++++++++++++
>> 2 files changed, 84 insertions(+)
>> create mode 100644 meta/recipes-devtools/gcc/gcc-4.8/0036-PR-target-56102.patch
>> 
>> diff --git a/meta/recipes-devtools/gcc/gcc-4.8.inc b/meta/recipes-devtools/gcc/gcc-4.8.inc
>> index 6664f1c..bfdb766 100644
>> --- a/meta/recipes-devtools/gcc/gcc-4.8.inc
>> +++ b/meta/recipes-devtools/gcc/gcc-4.8.inc
>> @@ -65,6 +65,7 @@ SRC_URI = "${GNU_MIRROR}/gcc/gcc-${PV}/gcc-${PV}.tar.bz2 \
>> 	   file://0033-gcc-armv4-pass-fix-v4bx-to-linker-to-support-EABI.patch \
>> 	   file://0034-Use-the-multilib-config-files-from-B-instead-of-usin.patch \
>> 	   file://0035-wcast-qual-PR-55383.patch \
>> +	   file://0036-PR-target-56102.patch \
>> 	   file://gcc-4.8-PR56797.patch \
>> 	   file://gcc-4.8-build-args.patch \
>> 	  "
>> diff --git a/meta/recipes-devtools/gcc/gcc-4.8/0036-PR-target-56102.patch b/meta/recipes-devtools/gcc/gcc-4.8/0036-PR-target-56102.patch
>> new file mode 100644
>> index 0000000..a9498fc
>> --- /dev/null
>> +++ b/meta/recipes-devtools/gcc/gcc-4.8/0036-PR-target-56102.patch
>> @@ -0,0 +1,83 @@
>> +From fa049b3584a1cc36c250205e3d5841e6a40ff677 Mon Sep 17 00:00:00 2001
>> +From: amker <amker@138bc75d-0d04-0410-961f-82ee72b054a4>
>> +Date: Wed, 27 Mar 2013 08:16:54 +0000
>> +Subject: [PATCH] PR target/56102
>> +
>> +* config/arm/arm.c (thumb1_rtx_costs, thumb1_size_rtx_costs): Fix rtx costs for SET/ASHIFT/ASHIFTRT/LSHIFTRT/ROTATERT patterns with mult-word mode.
>> +
>> +git-svn-id: svn+ssh://gcc.gnu.org/svn/gcc/trunk@197155 138bc75d-0d04-0410-961f-82ee72b054a4
>> +
>> +Upstream-Status: Backport
>> +http://gcc.gnu.org/bugzilla/show_bug.cgi?id=57329
>> +
>> +Signed-off-by: Martin Jansa <Martin.Jansa@gmail.com>
>> +---
>> + gcc/config/arm/arm.c | 17 ++++++++++++++---
>> + 1 file changed, 14 insertions(+), 3 deletions(-)
>> +
>> +diff --git a/gcc/config/arm/arm.c b/gcc/config/arm/arm.c
>> +index 460d333..edf850d 100644
>> +--- a/gcc/config/arm/arm.c
>> ++++ b/gcc/config/arm/arm.c
>> +@@ -7094,7 +7094,7 @@ static inline int
>> + thumb1_rtx_costs (rtx x, enum rtx_code code, enum rtx_code outer)
>> + {
>> +   enum machine_mode mode = GET_MODE (x);
>> +-  int total;
>> ++  int total, words;
>> + 
>> +   switch (code)
>> +     {
>> +@@ -7102,6 +7102,8 @@ thumb1_rtx_costs (rtx x, enum rtx_code code, enum rtx_code outer)
>> +     case ASHIFTRT:
>> +     case LSHIFTRT:
>> +     case ROTATERT:
>> ++      return (mode == SImode) ? COSTS_N_INSNS (1) : COSTS_N_INSNS (2);
>> ++
>> +     case PLUS:
>> +     case MINUS:
>> +     case COMPARE:
>> +@@ -7125,7 +7127,10 @@ thumb1_rtx_costs (rtx x, enum rtx_code code, enum rtx_code outer)
>> +       return COSTS_N_INSNS (1) + 16;
>> + 
>> +     case SET:
>> +-      return (COSTS_N_INSNS (1)
>> ++      /* A SET doesn't have a mode, so let's look at the SET_DEST to get
>> ++	 the mode.  */
>> ++      words = ARM_NUM_INTS (GET_MODE_SIZE (GET_MODE (SET_DEST (x))));
>> ++      return (COSTS_N_INSNS (words)
>> + 	      + 4 * ((MEM_P (SET_SRC (x)))
>> + 		     + MEM_P (SET_DEST (x))));
>> + 
>> +@@ -7822,6 +7827,7 @@ static inline int
>> + thumb1_size_rtx_costs (rtx x, enum rtx_code code, enum rtx_code outer)
>> + {
>> +   enum machine_mode mode = GET_MODE (x);
>> ++  int words;
>> + 
>> +   switch (code)
>> +     {
>> +@@ -7829,6 +7835,8 @@ thumb1_size_rtx_costs (rtx x, enum rtx_code code, enum rtx_code outer)
>> +     case ASHIFTRT:
>> +     case LSHIFTRT:
>> +     case ROTATERT:
>> ++      return (mode == SImode) ? COSTS_N_INSNS (1) : COSTS_N_INSNS (2);
>> ++
>> +     case PLUS:
>> +     case MINUS:
>> +     case COMPARE:
>> +@@ -7847,7 +7855,10 @@ thumb1_size_rtx_costs (rtx x, enum rtx_code code, enum rtx_code outer)
>> +       return COSTS_N_INSNS (1);
>> + 
>> +     case SET:
>> +-      return (COSTS_N_INSNS (1)
>> ++      /* A SET doesn't have a mode, so let's look at the SET_DEST to get
>> ++	 the mode.  */
>> ++      words = ARM_NUM_INTS (GET_MODE_SIZE (GET_MODE (SET_DEST (x))));
>> ++      return (COSTS_N_INSNS (words)
>> +               + 4 * ((MEM_P (SET_SRC (x)))
>> +                      + MEM_P (SET_DEST (x))));
>> + 
>> +-- 
>> +1.8.2.1
>> +
>> -- 
>> 1.8.2.1
>> 
> 
> -- 
> Martin 'JaMa' Jansa     jabber: Martin.Jansa@gmail.com
> _______________________________________________
> Openembedded-core mailing list
> Openembedded-core@lists.openembedded.org
> http://lists.openembedded.org/mailman/listinfo/openembedded-core



^ permalink raw reply	[flat|nested] 11+ messages in thread

* Re: [RFC][PATCH] gcc-4.8: Backport fix for ICE triggered in pixman
  2013-06-11 22:35   ` Martin Jansa
  2013-06-11 22:36     ` Khem Raj
@ 2013-06-11 22:39     ` Khem Raj
  2013-06-11 23:00       ` Martin Jansa
  1 sibling, 1 reply; 11+ messages in thread
From: Khem Raj @ 2013-06-11 22:39 UTC (permalink / raw)
  To: Martin Jansa; +Cc: openembedded-core


On Jun 11, 2013, at 3:35 PM, Martin Jansa <martin.jansa@gmail.com> wrote:

> On Wed, Jun 12, 2013 at 12:31:16AM +0200, Martin Jansa wrote:
>> * http://gcc.gnu.org/bugzilla/show_bug.cgi?id=57329
> 
> Only RFC, because world build is still running to confirm that it
> doesn't introduce new issues and that it indeed fixes pixman build for
> xscalete.

can you also test it with rebased on top of gcc 4.8.1 upgrade patch I sent earlier today

> 
>> 
>> Signed-off-by: Martin Jansa <Martin.Jansa@gmail.com>
>> ---
>> meta/recipes-devtools/gcc/gcc-4.8.inc              |  1 +
>> .../gcc/gcc-4.8/0036-PR-target-56102.patch         | 83 ++++++++++++++++++++++
>> 2 files changed, 84 insertions(+)
>> create mode 100644 meta/recipes-devtools/gcc/gcc-4.8/0036-PR-target-56102.patch
>> 
>> diff --git a/meta/recipes-devtools/gcc/gcc-4.8.inc b/meta/recipes-devtools/gcc/gcc-4.8.inc
>> index 6664f1c..bfdb766 100644
>> --- a/meta/recipes-devtools/gcc/gcc-4.8.inc
>> +++ b/meta/recipes-devtools/gcc/gcc-4.8.inc
>> @@ -65,6 +65,7 @@ SRC_URI = "${GNU_MIRROR}/gcc/gcc-${PV}/gcc-${PV}.tar.bz2 \
>> 	   file://0033-gcc-armv4-pass-fix-v4bx-to-linker-to-support-EABI.patch \
>> 	   file://0034-Use-the-multilib-config-files-from-B-instead-of-usin.patch \
>> 	   file://0035-wcast-qual-PR-55383.patch \
>> +	   file://0036-PR-target-56102.patch \
>> 	   file://gcc-4.8-PR56797.patch \
>> 	   file://gcc-4.8-build-args.patch \
>> 	  "
>> diff --git a/meta/recipes-devtools/gcc/gcc-4.8/0036-PR-target-56102.patch b/meta/recipes-devtools/gcc/gcc-4.8/0036-PR-target-56102.patch
>> new file mode 100644
>> index 0000000..a9498fc
>> --- /dev/null
>> +++ b/meta/recipes-devtools/gcc/gcc-4.8/0036-PR-target-56102.patch
>> @@ -0,0 +1,83 @@
>> +From fa049b3584a1cc36c250205e3d5841e6a40ff677 Mon Sep 17 00:00:00 2001
>> +From: amker <amker@138bc75d-0d04-0410-961f-82ee72b054a4>
>> +Date: Wed, 27 Mar 2013 08:16:54 +0000
>> +Subject: [PATCH] PR target/56102
>> +
>> +* config/arm/arm.c (thumb1_rtx_costs, thumb1_size_rtx_costs): Fix rtx costs for SET/ASHIFT/ASHIFTRT/LSHIFTRT/ROTATERT patterns with mult-word mode.
>> +
>> +git-svn-id: svn+ssh://gcc.gnu.org/svn/gcc/trunk@197155 138bc75d-0d04-0410-961f-82ee72b054a4
>> +
>> +Upstream-Status: Backport
>> +http://gcc.gnu.org/bugzilla/show_bug.cgi?id=57329
>> +
>> +Signed-off-by: Martin Jansa <Martin.Jansa@gmail.com>
>> +---
>> + gcc/config/arm/arm.c | 17 ++++++++++++++---
>> + 1 file changed, 14 insertions(+), 3 deletions(-)
>> +
>> +diff --git a/gcc/config/arm/arm.c b/gcc/config/arm/arm.c
>> +index 460d333..edf850d 100644
>> +--- a/gcc/config/arm/arm.c
>> ++++ b/gcc/config/arm/arm.c
>> +@@ -7094,7 +7094,7 @@ static inline int
>> + thumb1_rtx_costs (rtx x, enum rtx_code code, enum rtx_code outer)
>> + {
>> +   enum machine_mode mode = GET_MODE (x);
>> +-  int total;
>> ++  int total, words;
>> + 
>> +   switch (code)
>> +     {
>> +@@ -7102,6 +7102,8 @@ thumb1_rtx_costs (rtx x, enum rtx_code code, enum rtx_code outer)
>> +     case ASHIFTRT:
>> +     case LSHIFTRT:
>> +     case ROTATERT:
>> ++      return (mode == SImode) ? COSTS_N_INSNS (1) : COSTS_N_INSNS (2);
>> ++
>> +     case PLUS:
>> +     case MINUS:
>> +     case COMPARE:
>> +@@ -7125,7 +7127,10 @@ thumb1_rtx_costs (rtx x, enum rtx_code code, enum rtx_code outer)
>> +       return COSTS_N_INSNS (1) + 16;
>> + 
>> +     case SET:
>> +-      return (COSTS_N_INSNS (1)
>> ++      /* A SET doesn't have a mode, so let's look at the SET_DEST to get
>> ++	 the mode.  */
>> ++      words = ARM_NUM_INTS (GET_MODE_SIZE (GET_MODE (SET_DEST (x))));
>> ++      return (COSTS_N_INSNS (words)
>> + 	      + 4 * ((MEM_P (SET_SRC (x)))
>> + 		     + MEM_P (SET_DEST (x))));
>> + 
>> +@@ -7822,6 +7827,7 @@ static inline int
>> + thumb1_size_rtx_costs (rtx x, enum rtx_code code, enum rtx_code outer)
>> + {
>> +   enum machine_mode mode = GET_MODE (x);
>> ++  int words;
>> + 
>> +   switch (code)
>> +     {
>> +@@ -7829,6 +7835,8 @@ thumb1_size_rtx_costs (rtx x, enum rtx_code code, enum rtx_code outer)
>> +     case ASHIFTRT:
>> +     case LSHIFTRT:
>> +     case ROTATERT:
>> ++      return (mode == SImode) ? COSTS_N_INSNS (1) : COSTS_N_INSNS (2);
>> ++
>> +     case PLUS:
>> +     case MINUS:
>> +     case COMPARE:
>> +@@ -7847,7 +7855,10 @@ thumb1_size_rtx_costs (rtx x, enum rtx_code code, enum rtx_code outer)
>> +       return COSTS_N_INSNS (1);
>> + 
>> +     case SET:
>> +-      return (COSTS_N_INSNS (1)
>> ++      /* A SET doesn't have a mode, so let's look at the SET_DEST to get
>> ++	 the mode.  */
>> ++      words = ARM_NUM_INTS (GET_MODE_SIZE (GET_MODE (SET_DEST (x))));
>> ++      return (COSTS_N_INSNS (words)
>> +               + 4 * ((MEM_P (SET_SRC (x)))
>> +                      + MEM_P (SET_DEST (x))));
>> + 
>> +-- 
>> +1.8.2.1
>> +
>> -- 
>> 1.8.2.1
>> 
> 
> -- 
> Martin 'JaMa' Jansa     jabber: Martin.Jansa@gmail.com
> _______________________________________________
> Openembedded-core mailing list
> Openembedded-core@lists.openembedded.org
> http://lists.openembedded.org/mailman/listinfo/openembedded-core



^ permalink raw reply	[flat|nested] 11+ messages in thread

* Re: [RFC][PATCH] gcc-4.8: Backport fix for ICE triggered in pixman
  2013-06-11 22:39     ` Khem Raj
@ 2013-06-11 23:00       ` Martin Jansa
  2013-06-11 23:16         ` Saul Wold
  0 siblings, 1 reply; 11+ messages in thread
From: Martin Jansa @ 2013-06-11 23:00 UTC (permalink / raw)
  To: Khem Raj; +Cc: openembedded-core

[-- Attachment #1: Type: text/plain, Size: 5656 bytes --]

On Tue, Jun 11, 2013 at 03:39:35PM -0700, Khem Raj wrote:
> 
> On Jun 11, 2013, at 3:35 PM, Martin Jansa <martin.jansa@gmail.com> wrote:
> 
> > On Wed, Jun 12, 2013 at 12:31:16AM +0200, Martin Jansa wrote:
> >> * http://gcc.gnu.org/bugzilla/show_bug.cgi?id=57329
> > 
> > Only RFC, because world build is still running to confirm that it
> > doesn't introduce new issues and that it indeed fixes pixman build for
> > xscalete.
> 
> can you also test it with rebased on top of gcc 4.8.1 upgrade patch I sent earlier today

It is, that's why I've sent it as reply-to <your-patch>.

And jenkins is indeed testing your 4.8.1 + this one.

> >> Signed-off-by: Martin Jansa <Martin.Jansa@gmail.com>
> >> ---
> >> meta/recipes-devtools/gcc/gcc-4.8.inc              |  1 +
> >> .../gcc/gcc-4.8/0036-PR-target-56102.patch         | 83 ++++++++++++++++++++++
> >> 2 files changed, 84 insertions(+)
> >> create mode 100644 meta/recipes-devtools/gcc/gcc-4.8/0036-PR-target-56102.patch
> >> 
> >> diff --git a/meta/recipes-devtools/gcc/gcc-4.8.inc b/meta/recipes-devtools/gcc/gcc-4.8.inc
> >> index 6664f1c..bfdb766 100644
> >> --- a/meta/recipes-devtools/gcc/gcc-4.8.inc
> >> +++ b/meta/recipes-devtools/gcc/gcc-4.8.inc
> >> @@ -65,6 +65,7 @@ SRC_URI = "${GNU_MIRROR}/gcc/gcc-${PV}/gcc-${PV}.tar.bz2 \
> >> 	   file://0033-gcc-armv4-pass-fix-v4bx-to-linker-to-support-EABI.patch \
> >> 	   file://0034-Use-the-multilib-config-files-from-B-instead-of-usin.patch \
> >> 	   file://0035-wcast-qual-PR-55383.patch \
> >> +	   file://0036-PR-target-56102.patch \
> >> 	   file://gcc-4.8-PR56797.patch \
> >> 	   file://gcc-4.8-build-args.patch \
> >> 	  "
> >> diff --git a/meta/recipes-devtools/gcc/gcc-4.8/0036-PR-target-56102.patch b/meta/recipes-devtools/gcc/gcc-4.8/0036-PR-target-56102.patch
> >> new file mode 100644
> >> index 0000000..a9498fc
> >> --- /dev/null
> >> +++ b/meta/recipes-devtools/gcc/gcc-4.8/0036-PR-target-56102.patch
> >> @@ -0,0 +1,83 @@
> >> +From fa049b3584a1cc36c250205e3d5841e6a40ff677 Mon Sep 17 00:00:00 2001
> >> +From: amker <amker@138bc75d-0d04-0410-961f-82ee72b054a4>
> >> +Date: Wed, 27 Mar 2013 08:16:54 +0000
> >> +Subject: [PATCH] PR target/56102
> >> +
> >> +* config/arm/arm.c (thumb1_rtx_costs, thumb1_size_rtx_costs): Fix rtx costs for SET/ASHIFT/ASHIFTRT/LSHIFTRT/ROTATERT patterns with mult-word mode.
> >> +
> >> +git-svn-id: svn+ssh://gcc.gnu.org/svn/gcc/trunk@197155 138bc75d-0d04-0410-961f-82ee72b054a4
> >> +
> >> +Upstream-Status: Backport
> >> +http://gcc.gnu.org/bugzilla/show_bug.cgi?id=57329
> >> +
> >> +Signed-off-by: Martin Jansa <Martin.Jansa@gmail.com>
> >> +---
> >> + gcc/config/arm/arm.c | 17 ++++++++++++++---
> >> + 1 file changed, 14 insertions(+), 3 deletions(-)
> >> +
> >> +diff --git a/gcc/config/arm/arm.c b/gcc/config/arm/arm.c
> >> +index 460d333..edf850d 100644
> >> +--- a/gcc/config/arm/arm.c
> >> ++++ b/gcc/config/arm/arm.c
> >> +@@ -7094,7 +7094,7 @@ static inline int
> >> + thumb1_rtx_costs (rtx x, enum rtx_code code, enum rtx_code outer)
> >> + {
> >> +   enum machine_mode mode = GET_MODE (x);
> >> +-  int total;
> >> ++  int total, words;
> >> + 
> >> +   switch (code)
> >> +     {
> >> +@@ -7102,6 +7102,8 @@ thumb1_rtx_costs (rtx x, enum rtx_code code, enum rtx_code outer)
> >> +     case ASHIFTRT:
> >> +     case LSHIFTRT:
> >> +     case ROTATERT:
> >> ++      return (mode == SImode) ? COSTS_N_INSNS (1) : COSTS_N_INSNS (2);
> >> ++
> >> +     case PLUS:
> >> +     case MINUS:
> >> +     case COMPARE:
> >> +@@ -7125,7 +7127,10 @@ thumb1_rtx_costs (rtx x, enum rtx_code code, enum rtx_code outer)
> >> +       return COSTS_N_INSNS (1) + 16;
> >> + 
> >> +     case SET:
> >> +-      return (COSTS_N_INSNS (1)
> >> ++      /* A SET doesn't have a mode, so let's look at the SET_DEST to get
> >> ++	 the mode.  */
> >> ++      words = ARM_NUM_INTS (GET_MODE_SIZE (GET_MODE (SET_DEST (x))));
> >> ++      return (COSTS_N_INSNS (words)
> >> + 	      + 4 * ((MEM_P (SET_SRC (x)))
> >> + 		     + MEM_P (SET_DEST (x))));
> >> + 
> >> +@@ -7822,6 +7827,7 @@ static inline int
> >> + thumb1_size_rtx_costs (rtx x, enum rtx_code code, enum rtx_code outer)
> >> + {
> >> +   enum machine_mode mode = GET_MODE (x);
> >> ++  int words;
> >> + 
> >> +   switch (code)
> >> +     {
> >> +@@ -7829,6 +7835,8 @@ thumb1_size_rtx_costs (rtx x, enum rtx_code code, enum rtx_code outer)
> >> +     case ASHIFTRT:
> >> +     case LSHIFTRT:
> >> +     case ROTATERT:
> >> ++      return (mode == SImode) ? COSTS_N_INSNS (1) : COSTS_N_INSNS (2);
> >> ++
> >> +     case PLUS:
> >> +     case MINUS:
> >> +     case COMPARE:
> >> +@@ -7847,7 +7855,10 @@ thumb1_size_rtx_costs (rtx x, enum rtx_code code, enum rtx_code outer)
> >> +       return COSTS_N_INSNS (1);
> >> + 
> >> +     case SET:
> >> +-      return (COSTS_N_INSNS (1)
> >> ++      /* A SET doesn't have a mode, so let's look at the SET_DEST to get
> >> ++	 the mode.  */
> >> ++      words = ARM_NUM_INTS (GET_MODE_SIZE (GET_MODE (SET_DEST (x))));
> >> ++      return (COSTS_N_INSNS (words)
> >> +               + 4 * ((MEM_P (SET_SRC (x)))
> >> +                      + MEM_P (SET_DEST (x))));
> >> + 
> >> +-- 
> >> +1.8.2.1
> >> +
> >> -- 
> >> 1.8.2.1
> >> 
> > 
> > -- 
> > Martin 'JaMa' Jansa     jabber: Martin.Jansa@gmail.com
> > _______________________________________________
> > Openembedded-core mailing list
> > Openembedded-core@lists.openembedded.org
> > http://lists.openembedded.org/mailman/listinfo/openembedded-core
> 

-- 
Martin 'JaMa' Jansa     jabber: Martin.Jansa@gmail.com

[-- Attachment #2: Digital signature --]
[-- Type: application/pgp-signature, Size: 205 bytes --]

^ permalink raw reply	[flat|nested] 11+ messages in thread

* Re: [RFC][PATCH] gcc-4.8: Backport fix for ICE triggered in pixman
  2013-06-11 23:00       ` Martin Jansa
@ 2013-06-11 23:16         ` Saul Wold
  0 siblings, 0 replies; 11+ messages in thread
From: Saul Wold @ 2013-06-11 23:16 UTC (permalink / raw)
  To: Martin Jansa; +Cc: openembedded-core

On 06/11/2013 04:00 PM, Martin Jansa wrote:
> On Tue, Jun 11, 2013 at 03:39:35PM -0700, Khem Raj wrote:
>>
>> On Jun 11, 2013, at 3:35 PM, Martin Jansa <martin.jansa@gmail.com> wrote:
>>
>>> On Wed, Jun 12, 2013 at 12:31:16AM +0200, Martin Jansa wrote:
>>>> * http://gcc.gnu.org/bugzilla/show_bug.cgi?id=57329
>>>
>>> Only RFC, because world build is still running to confirm that it
>>> doesn't introduce new issues and that it indeed fixes pixman build for
>>> xscalete.
>>
>> can you also test it with rebased on top of gcc 4.8.1 upgrade patch I sent earlier today
>
> It is, that's why I've sent it as reply-to <your-patch>.
>
> And jenkins is indeed testing your 4.8.1 + this one.
>
I will throw this all against the AB later today, it's busy with 1.4.1 
currently.  I will start some local worlds here also.

Sau!
>>>> Signed-off-by: Martin Jansa <Martin.Jansa@gmail.com>
>>>> ---
>>>> meta/recipes-devtools/gcc/gcc-4.8.inc              |  1 +
>>>> .../gcc/gcc-4.8/0036-PR-target-56102.patch         | 83 ++++++++++++++++++++++
>>>> 2 files changed, 84 insertions(+)
>>>> create mode 100644 meta/recipes-devtools/gcc/gcc-4.8/0036-PR-target-56102.patch
>>>>
>>>> diff --git a/meta/recipes-devtools/gcc/gcc-4.8.inc b/meta/recipes-devtools/gcc/gcc-4.8.inc
>>>> index 6664f1c..bfdb766 100644
>>>> --- a/meta/recipes-devtools/gcc/gcc-4.8.inc
>>>> +++ b/meta/recipes-devtools/gcc/gcc-4.8.inc
>>>> @@ -65,6 +65,7 @@ SRC_URI = "${GNU_MIRROR}/gcc/gcc-${PV}/gcc-${PV}.tar.bz2 \
>>>> 	   file://0033-gcc-armv4-pass-fix-v4bx-to-linker-to-support-EABI.patch \
>>>> 	   file://0034-Use-the-multilib-config-files-from-B-instead-of-usin.patch \
>>>> 	   file://0035-wcast-qual-PR-55383.patch \
>>>> +	   file://0036-PR-target-56102.patch \
>>>> 	   file://gcc-4.8-PR56797.patch \
>>>> 	   file://gcc-4.8-build-args.patch \
>>>> 	  "
>>>> diff --git a/meta/recipes-devtools/gcc/gcc-4.8/0036-PR-target-56102.patch b/meta/recipes-devtools/gcc/gcc-4.8/0036-PR-target-56102.patch
>>>> new file mode 100644
>>>> index 0000000..a9498fc
>>>> --- /dev/null
>>>> +++ b/meta/recipes-devtools/gcc/gcc-4.8/0036-PR-target-56102.patch
>>>> @@ -0,0 +1,83 @@
>>>> +From fa049b3584a1cc36c250205e3d5841e6a40ff677 Mon Sep 17 00:00:00 2001
>>>> +From: amker <amker@138bc75d-0d04-0410-961f-82ee72b054a4>
>>>> +Date: Wed, 27 Mar 2013 08:16:54 +0000
>>>> +Subject: [PATCH] PR target/56102
>>>> +
>>>> +* config/arm/arm.c (thumb1_rtx_costs, thumb1_size_rtx_costs): Fix rtx costs for SET/ASHIFT/ASHIFTRT/LSHIFTRT/ROTATERT patterns with mult-word mode.
>>>> +
>>>> +git-svn-id: svn+ssh://gcc.gnu.org/svn/gcc/trunk@197155 138bc75d-0d04-0410-961f-82ee72b054a4
>>>> +
>>>> +Upstream-Status: Backport
>>>> +http://gcc.gnu.org/bugzilla/show_bug.cgi?id=57329
>>>> +
>>>> +Signed-off-by: Martin Jansa <Martin.Jansa@gmail.com>
>>>> +---
>>>> + gcc/config/arm/arm.c | 17 ++++++++++++++---
>>>> + 1 file changed, 14 insertions(+), 3 deletions(-)
>>>> +
>>>> +diff --git a/gcc/config/arm/arm.c b/gcc/config/arm/arm.c
>>>> +index 460d333..edf850d 100644
>>>> +--- a/gcc/config/arm/arm.c
>>>> ++++ b/gcc/config/arm/arm.c
>>>> +@@ -7094,7 +7094,7 @@ static inline int
>>>> + thumb1_rtx_costs (rtx x, enum rtx_code code, enum rtx_code outer)
>>>> + {
>>>> +   enum machine_mode mode = GET_MODE (x);
>>>> +-  int total;
>>>> ++  int total, words;
>>>> +
>>>> +   switch (code)
>>>> +     {
>>>> +@@ -7102,6 +7102,8 @@ thumb1_rtx_costs (rtx x, enum rtx_code code, enum rtx_code outer)
>>>> +     case ASHIFTRT:
>>>> +     case LSHIFTRT:
>>>> +     case ROTATERT:
>>>> ++      return (mode == SImode) ? COSTS_N_INSNS (1) : COSTS_N_INSNS (2);
>>>> ++
>>>> +     case PLUS:
>>>> +     case MINUS:
>>>> +     case COMPARE:
>>>> +@@ -7125,7 +7127,10 @@ thumb1_rtx_costs (rtx x, enum rtx_code code, enum rtx_code outer)
>>>> +       return COSTS_N_INSNS (1) + 16;
>>>> +
>>>> +     case SET:
>>>> +-      return (COSTS_N_INSNS (1)
>>>> ++      /* A SET doesn't have a mode, so let's look at the SET_DEST to get
>>>> ++	 the mode.  */
>>>> ++      words = ARM_NUM_INTS (GET_MODE_SIZE (GET_MODE (SET_DEST (x))));
>>>> ++      return (COSTS_N_INSNS (words)
>>>> + 	      + 4 * ((MEM_P (SET_SRC (x)))
>>>> + 		     + MEM_P (SET_DEST (x))));
>>>> +
>>>> +@@ -7822,6 +7827,7 @@ static inline int
>>>> + thumb1_size_rtx_costs (rtx x, enum rtx_code code, enum rtx_code outer)
>>>> + {
>>>> +   enum machine_mode mode = GET_MODE (x);
>>>> ++  int words;
>>>> +
>>>> +   switch (code)
>>>> +     {
>>>> +@@ -7829,6 +7835,8 @@ thumb1_size_rtx_costs (rtx x, enum rtx_code code, enum rtx_code outer)
>>>> +     case ASHIFTRT:
>>>> +     case LSHIFTRT:
>>>> +     case ROTATERT:
>>>> ++      return (mode == SImode) ? COSTS_N_INSNS (1) : COSTS_N_INSNS (2);
>>>> ++
>>>> +     case PLUS:
>>>> +     case MINUS:
>>>> +     case COMPARE:
>>>> +@@ -7847,7 +7855,10 @@ thumb1_size_rtx_costs (rtx x, enum rtx_code code, enum rtx_code outer)
>>>> +       return COSTS_N_INSNS (1);
>>>> +
>>>> +     case SET:
>>>> +-      return (COSTS_N_INSNS (1)
>>>> ++      /* A SET doesn't have a mode, so let's look at the SET_DEST to get
>>>> ++	 the mode.  */
>>>> ++      words = ARM_NUM_INTS (GET_MODE_SIZE (GET_MODE (SET_DEST (x))));
>>>> ++      return (COSTS_N_INSNS (words)
>>>> +               + 4 * ((MEM_P (SET_SRC (x)))
>>>> +                      + MEM_P (SET_DEST (x))));
>>>> +
>>>> +--
>>>> +1.8.2.1
>>>> +
>>>> --
>>>> 1.8.2.1
>>>>
>>>
>>> --
>>> Martin 'JaMa' Jansa     jabber: Martin.Jansa@gmail.com
>>> _______________________________________________
>>> Openembedded-core mailing list
>>> Openembedded-core@lists.openembedded.org
>>> http://lists.openembedded.org/mailman/listinfo/openembedded-core
>>
>
>
>
> _______________________________________________
> Openembedded-core mailing list
> Openembedded-core@lists.openembedded.org
> http://lists.openembedded.org/mailman/listinfo/openembedded-core
>


^ permalink raw reply	[flat|nested] 11+ messages in thread

* Re: [PATCH] gcc: Upgrade to 4.8.1
  2013-06-11 21:57 [PATCH] gcc: Upgrade to 4.8.1 Khem Raj
  2013-06-11 22:31 ` [RFC][PATCH] gcc-4.8: Backport fix for ICE triggered in pixman Martin Jansa
@ 2013-06-13 17:12 ` Saul Wold
  2013-06-13 19:43   ` Khem Raj
  2013-06-28 23:47 ` Jonathan Liu
  2 siblings, 1 reply; 11+ messages in thread
From: Saul Wold @ 2013-06-13 17:12 UTC (permalink / raw)
  To: Khem Raj; +Cc: Otavio Salvador, openembedded-core

On 06/11/2013 02:57 PM, Khem Raj wrote:
> Signed-off-by: Khem Raj <raj.khem@gmail.com>
> ---
>   meta/recipes-devtools/gcc/gcc-4.8.inc |    8 ++++----
>   1 file changed, 4 insertions(+), 4 deletions(-)
>
> diff --git a/meta/recipes-devtools/gcc/gcc-4.8.inc b/meta/recipes-devtools/gcc/gcc-4.8.inc
> index 80bd02c..6664f1c 100644
> --- a/meta/recipes-devtools/gcc/gcc-4.8.inc
> +++ b/meta/recipes-devtools/gcc/gcc-4.8.inc
> @@ -6,7 +6,7 @@ require gcc-common.inc
>   # on branch then PV should be incremented to 4.7.2+svnr${SRCPV}
>   # to reflect that change
>
> -PV = "4.8.0"
> +PV = "4.8.1"
>

Khem, Otavio:

Not sure if it was this change or 4.8 in general, but there are a number 
of failures in the FSL-ARM and FSL-PPC builds:

PPC Issues (gettext / mfpr):
http://autobuilder.yoctoproject.org:8011/builders/nightly-fsl-ppc/builds/141/steps/Building%20Images/logs/stdio
http://autobuilder.yoctoproject.org:8011/builders/nightly-fsl-ppc-lsb/builds/139/steps/Building%20Images/logs/stdio

ARM Issues (both in perf):
http://autobuilder.yoctoproject.org:8011/builders/nightly-fsl-arm/builds/140/steps/Building%20Images/logs/stdio

Can one or both of you look into this.

Thanks
	Sau!


>   # BINV should be incremented after updating to a revision
>   # after a minor gcc release (e.g. 4.7.1 or 4.7.2) has been made
> @@ -14,7 +14,7 @@ PV = "4.8.0"
>   # 4.7.1 then the value below will have 2 which will mean 4.7.2
>   # which will be next minor release and so on.
>
> -BINV = "4.8.0"
> +BINV = "4.8.1"
>
>   FILESPATH = "${@base_set_filespath([ '${FILE_DIRNAME}/gcc-4.8' ], d)}"
>
> @@ -68,8 +68,8 @@ SRC_URI = "${GNU_MIRROR}/gcc/gcc-${PV}/gcc-${PV}.tar.bz2 \
>   	   file://gcc-4.8-PR56797.patch \
>   	   file://gcc-4.8-build-args.patch \
>   	  "
> -SRC_URI[md5sum] = "e6040024eb9e761c3bea348d1fa5abb0"
> -SRC_URI[sha256sum] = "b037fe5132b71ecad2ea7141ec92292b5d32427bf90fd90cde432b1d5abacc2c"
> +SRC_URI[md5sum] = "3b2386c114cd74185aa3754b58a79304"
> +SRC_URI[sha256sum] = "545b44be3ad9f2c4e90e6880f5c9d4f0a8f0e5f67e1ffb0d45da9fa01bb05813"
>
>   S = "${TMPDIR}/work-shared/gcc-${PV}-${PR}/gcc-${PV}"
>   B = "${WORKDIR}/gcc-${PV}/build.${HOST_SYS}.${TARGET_SYS}"
>


^ permalink raw reply	[flat|nested] 11+ messages in thread

* Re: [PATCH] gcc: Upgrade to 4.8.1
  2013-06-13 17:12 ` [PATCH] gcc: Upgrade to 4.8.1 Saul Wold
@ 2013-06-13 19:43   ` Khem Raj
  0 siblings, 0 replies; 11+ messages in thread
From: Khem Raj @ 2013-06-13 19:43 UTC (permalink / raw)
  To: Saul Wold; +Cc: Otavio Salvador, openembedded-core


On Jun 13, 2013, at 10:12 AM, Saul Wold <sgw@linux.intel.com> wrote:

> On 06/11/2013 02:57 PM, Khem Raj wrote:
>> Signed-off-by: Khem Raj <raj.khem@gmail.com>
>> ---
>>  meta/recipes-devtools/gcc/gcc-4.8.inc |    8 ++++----
>>  1 file changed, 4 insertions(+), 4 deletions(-)
>> 
>> diff --git a/meta/recipes-devtools/gcc/gcc-4.8.inc b/meta/recipes-devtools/gcc/gcc-4.8.inc
>> index 80bd02c..6664f1c 100644
>> --- a/meta/recipes-devtools/gcc/gcc-4.8.inc
>> +++ b/meta/recipes-devtools/gcc/gcc-4.8.inc
>> @@ -6,7 +6,7 @@ require gcc-common.inc
>>  # on branch then PV should be incremented to 4.7.2+svnr${SRCPV}
>>  # to reflect that change
>> 
>> -PV = "4.8.0"
>> +PV = "4.8.1"
>> 
> 
> Khem, Otavio:
> 
> Not sure if it was this change or 4.8 in general, but there are a number of failures in the FSL-ARM and FSL-PPC builds:
> 
> PPC Issues (gettext / mfpr):
> http://autobuilder.yoctoproject.org:8011/builders/nightly-fsl-ppc/builds/141/steps/Building%20Images/logs/stdio
> http://autobuilder.yoctoproject.org:8011/builders/nightly-fsl-ppc-lsb/builds/139/steps/Building%20Images/logs/stdio
> 

both above seems same issue its a compiler ICE

> ARM Issues (both in perf):
> http://autobuilder.yoctoproject.org:8011/builders/nightly-fsl-arm/builds/140/steps/Building%20Images/logs/stdio
> 

its also a compiler ICE

> Can one or both of you look into this.
> 
> Thanks
> 	Sau!
> 
> 
>>  # BINV should be incremented after updating to a revision
>>  # after a minor gcc release (e.g. 4.7.1 or 4.7.2) has been made
>> @@ -14,7 +14,7 @@ PV = "4.8.0"
>>  # 4.7.1 then the value below will have 2 which will mean 4.7.2
>>  # which will be next minor release and so on.
>> 
>> -BINV = "4.8.0"
>> +BINV = "4.8.1"
>> 
>>  FILESPATH = "${@base_set_filespath([ '${FILE_DIRNAME}/gcc-4.8' ], d)}"
>> 
>> @@ -68,8 +68,8 @@ SRC_URI = "${GNU_MIRROR}/gcc/gcc-${PV}/gcc-${PV}.tar.bz2 \
>>  	   file://gcc-4.8-PR56797.patch \
>>  	   file://gcc-4.8-build-args.patch \
>>  	  "
>> -SRC_URI[md5sum] = "e6040024eb9e761c3bea348d1fa5abb0"
>> -SRC_URI[sha256sum] = "b037fe5132b71ecad2ea7141ec92292b5d32427bf90fd90cde432b1d5abacc2c"
>> +SRC_URI[md5sum] = "3b2386c114cd74185aa3754b58a79304"
>> +SRC_URI[sha256sum] = "545b44be3ad9f2c4e90e6880f5c9d4f0a8f0e5f67e1ffb0d45da9fa01bb05813"
>> 
>>  S = "${TMPDIR}/work-shared/gcc-${PV}-${PR}/gcc-${PV}"
>>  B = "${WORKDIR}/gcc-${PV}/build.${HOST_SYS}.${TARGET_SYS}"
>> 



^ permalink raw reply	[flat|nested] 11+ messages in thread

* Re: [PATCH] gcc: Upgrade to 4.8.1
  2013-06-11 21:57 [PATCH] gcc: Upgrade to 4.8.1 Khem Raj
  2013-06-11 22:31 ` [RFC][PATCH] gcc-4.8: Backport fix for ICE triggered in pixman Martin Jansa
  2013-06-13 17:12 ` [PATCH] gcc: Upgrade to 4.8.1 Saul Wold
@ 2013-06-28 23:47 ` Jonathan Liu
  2013-06-29  0:51   ` Khem Raj
  2 siblings, 1 reply; 11+ messages in thread
From: Jonathan Liu @ 2013-06-28 23:47 UTC (permalink / raw)
  To: Khem Raj; +Cc: openembedded-core

Hi Khem,

I get the following error building gcc package for atom-pc on x86_64 host:
libtool: link:  i586-poky-linux-g++  -m32 -march=core2 -msse3 
-mtune=generic -mfpmath=sse 
--sysroot=/home/build/git/poky/build/tmp/sysroots/poky -  m32 
-march=core2 -msse3 -mtune=generic -mfpmath=sse 
-isystem/home/build/git/poky/build/tmp/sysroots/poky/usr/include 
-I/home/build/git/poky/build/tmp/ 
work/core2-poky-linux/gcc/4.8.1-r0/gcc-4.8.1/build.i586-poky-linux.i586-poky-linux/gcc/include/ 
--sysroot=/home/build/git/poky/build/tmp/      sysroots/poky -shared 
-nostdlib /home/build/git/poky/build/tmp/sysroots/poky/usr/lib/crti.o 
/home/build/git/poky/build/tmp/sysroots/poky/usr/lib/i586-poky-linux/4.8.1/crtbeginS.o 
.libs/asan_allocator.o .libs/asan_allocator2.o .libs/asan_interceptors.o 
.libs/asan_mac.o .libs/asan_malloc_mac.  o .libs/asan_new_delete.o 
.libs/asan_posix.o .libs/asan_rtl.o .libs/asan_stats.o 
.libs/asan_thread_registry.o .libs/asan_fake_stack.o 
.libs/asan_globals.o . libs/asan_linux.o .libs/asan_malloc_linux.o 
.libs/asan_malloc_win.o .libs/asan_poisoning.o .libs/asan_report.o 
.libs/asan_stack.o .libs/asan_thread.o .libs/ asan_win.o  
-Wl,--whole-archive ../sanitizer_common/.libs/libsanitizer_common.a 
../interception/.libs/libinterception.a -Wl,--no-whole-archive 
-Wl,-rpath - 
Wl,/home/build/git/poky/build/tmp/work/core2-poky-linux/gcc/4.8.1-r0/gcc-4.8.1/build.i586-poky-linux.i586-poky-linux/i586-poky-linux/ 
libstdc++-v3/src/.libs -lpthread -ldl 
../../libstdc++-v3/src/.libs/libstdc++.so 
-L/home/build/git/poky/build/tmp/sysroots/x86_64-linux/usr/lib/core2- 
poky-linux/gcc/i586-poky-linux/4.8.1 
-L/home/build/git/poky/build/tmp/sysroots/poky/lib 
-L/home/build/git/poky/build/tmp/sysroots/poky/usr/lib/i586-poky-linux/4.8.1 
-L/home/build/git/poky/build/tmp/sysroots/poky/usr/lib 
/usr/lib/libstdc++.so -lm -lc -lgcc_s /home/build/git/poky/build/ 
tmp/sysroots/poky/usr/lib/i586-poky-linux/4.8.1/crtendS.o 
/home/build/git/poky/build/tmp/sysroots/poky/usr/lib/crtn.o  -m32 
-march=core2 -      msse3 -mtune=generic -mfpmath=sse -m32 -march=core2 
-msse3 -mtune=generic -mfpmath=sse -Wl,-O1 -Wl,--hash-style=gnu 
-Wl,--as-needed   -Wl,-soname -Wl, libasan.so.0 -o .libs/libasan.so.0.0.0
/usr/lib/libstdc++.so: could not read symbols: File in wrong format
collect2: error: ld returned 1 exit status
make[3]: *** [libasan.la] Error 1
make[3]: Leaving directory 
`/home/build/git/poky/build/tmp/work/core2-poky-linux/gcc/4.8.1-r0/gcc-4.8.1/build.i586-poky-linux.i586-poky-linux/ 
i586-poky-linux/libsanitizer/asan'

Regards,
Jonathan


^ permalink raw reply	[flat|nested] 11+ messages in thread

* Re: [PATCH] gcc: Upgrade to 4.8.1
  2013-06-28 23:47 ` Jonathan Liu
@ 2013-06-29  0:51   ` Khem Raj
  0 siblings, 0 replies; 11+ messages in thread
From: Khem Raj @ 2013-06-29  0:51 UTC (permalink / raw)
  To: Jonathan Liu; +Cc: openembedded-core@lists.openembedded.org

[-- Attachment #1: Type: text/plain, Size: 2747 bytes --]

On Friday, June 28, 2013, Jonathan Liu <net147@gmail.com> wrote:
> Hi Khem,
>
> I get the following error building gcc package for atom-pc on x86_64 host:

Open a bug in up bugzilla and add your build configuration info to it


> libtool: link:  i586-poky-linux-g++  -m32 -march=core2 -msse3
-mtune=generic -mfpmath=sse
--sysroot=/home/build/git/poky/build/tmp/sysroots/poky -  m32 -march=core2
-msse3 -mtune=generic -mfpmath=sse
-isystem/home/build/git/poky/build/tmp/sysroots/poky/usr/include
-I/home/build/git/poky/build/tmp/
work/core2-poky-linux/gcc/4.8.1-r0/gcc-4.8.1/build.i586-poky-linux.i586-poky-linux/gcc/include/
--sysroot=/home/build/git/poky/build/tmp/      sysroots/poky -shared
-nostdlib /home/build/git/poky/build/tmp/sysroots/poky/usr/lib/crti.o
/home/build/git/poky/build/tmp/sysroots/poky/usr/lib/i586-poky-linux/4.8.1/crtbeginS.o
.libs/asan_allocator.o .libs/asan_allocator2.o .libs/asan_interceptors.o
.libs/asan_mac.o .libs/asan_malloc_mac.  o .libs/asan_new_delete.o
.libs/asan_posix.o .libs/asan_rtl.o .libs/asan_stats.o
.libs/asan_thread_registry.o .libs/asan_fake_stack.o .libs/asan_globals.o .
libs/asan_linux.o .libs/asan_malloc_linux.o .libs/asan_malloc_win.o
.libs/asan_poisoning.o .libs/asan_report.o .libs/asan_stack.o
.libs/asan_thread.o .libs/ asan_win.o  -Wl,--whole-archive
../sanitizer_common/.libs/libsanitizer_common.a
../interception/.libs/libinterception.a -Wl,--no-whole-archive -Wl,-rpath -
Wl,/home/build/git/poky/build/tmp/work/core2-poky-linux/gcc/4.8.1-r0/gcc-4.8.1/build.i586-poky-linux.i586-poky-linux/i586-poky-linux/
libstdc++-v3/src/.libs -lpthread -ldl
../../libstdc++-v3/src/.libs/libstdc++.so
-L/home/build/git/poky/build/tmp/sysroots/x86_64-linux/usr/lib/core2-
poky-linux/gcc/i586-poky-linux/4.8.1
-L/home/build/git/poky/build/tmp/sysroots/poky/lib
-L/home/build/git/poky/build/tmp/sysroots/poky/usr/lib/i586-poky-linux/4.8.1
-L/home/build/git/poky/build/tmp/sysroots/poky/usr/lib
/usr/lib/libstdc++.so -lm -lc -lgcc_s /home/build/git/poky/build/
tmp/sysroots/poky/usr/lib/i586-poky-linux/4.8.1/crtendS.o
/home/build/git/poky/build/tmp/sysroots/poky/usr/lib/crtn.o  -m32
-march=core2 -      msse3 -mtune=generic -mfpmath=sse -m32 -march=core2
-msse3 -mtune=generic -mfpmath=sse -Wl,-O1 -Wl,--hash-style=gnu
-Wl,--as-needed   -Wl,-soname -Wl, libasan.so.0 -o .libs/libasan.so.0.0.0
> /usr/lib/libstdc++.so: could not read symbols: File in wrong format
> collect2: error: ld returned 1 exit status
> make[3]: *** [libasan.la] Error 1
> make[3]: Leaving directory
`/home/build/git/poky/build/tmp/work/core2-poky-linux/gcc/4.8.1-r0/gcc-4.8.1/build.i586-poky-linux.i586-poky-linux/
i586-poky-linux/libsanitizer/asan'
>
> Regards,
> Jonathan
>

[-- Attachment #2: Type: text/html, Size: 2888 bytes --]

^ permalink raw reply	[flat|nested] 11+ messages in thread

end of thread, other threads:[~2013-06-29  0:51 UTC | newest]

Thread overview: 11+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2013-06-11 21:57 [PATCH] gcc: Upgrade to 4.8.1 Khem Raj
2013-06-11 22:31 ` [RFC][PATCH] gcc-4.8: Backport fix for ICE triggered in pixman Martin Jansa
2013-06-11 22:35   ` Martin Jansa
2013-06-11 22:36     ` Khem Raj
2013-06-11 22:39     ` Khem Raj
2013-06-11 23:00       ` Martin Jansa
2013-06-11 23:16         ` Saul Wold
2013-06-13 17:12 ` [PATCH] gcc: Upgrade to 4.8.1 Saul Wold
2013-06-13 19:43   ` Khem Raj
2013-06-28 23:47 ` Jonathan Liu
2013-06-29  0:51   ` Khem Raj

This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox