Patchwork cc/gcc: fix problem with gcc 4.8.1 and gcc 4.8.2 and parallel build

login
register
mail settings
Submitter Daniel Zimmermann
Date Nov. 17, 2013, 11:03 a.m.
Message ID <ea83356683a857e891cb.1384686204@haus-VirtualBox>
Download mbox | patch
Permalink /patch/291819/
State Superseded
Headers show

Comments

Daniel Zimmermann - Nov. 17, 2013, 11:03 a.m.
# HG changeset patch
# User Daniel Zimmermann <netzimme@gmail.com>
# Date 1384685304 -3600
#      Sun Nov 17 11:48:24 2013 +0100
# Node ID ea83356683a857e891cbfc4f4b19ee4ca878f5af
# Parent  7e569a9cb5fd3ab591bb307328b947a5b7312cba
cc/gcc: fix problem with gcc 4.8.1 and gcc 4.8.2 and parallel build

This patch add patches from Nicolas Hannekum. It fix the problem with parallel build on arm target.
See: http://gcc.gnu.org/bugzilla/show_bug.cgi?id=57683

Signed-off-by: "Daniel Zimmermann" <netzimme@gmail.com>


--
For unsubscribe information see http://sourceware.org/lists.html#faq
Zhenqiang Chen - Nov. 21, 2013, 5:23 a.m.
Nicolas's patch can fix the issue. But it is not committed in 4.8 or
trunk. It is not a good idea to maintain a uncommitted patch.

I just backported two patches from trunk to fix PR 57683. Please try it.

Thanks!
-Zhenqiang


On 17 November 2013 19:03, Daniel Zimmermann <netzimme@gmail.com> wrote:
> # HG changeset patch
> # User Daniel Zimmermann <netzimme@gmail.com>
> # Date 1384685304 -3600
> #      Sun Nov 17 11:48:24 2013 +0100
> # Node ID ea83356683a857e891cbfc4f4b19ee4ca878f5af
> # Parent  7e569a9cb5fd3ab591bb307328b947a5b7312cba
> cc/gcc: fix problem with gcc 4.8.1 and gcc 4.8.2 and parallel build
>
> This patch add patches from Nicolas Hannekum. It fix the problem with parallel build on arm target.
> See: http://gcc.gnu.org/bugzilla/show_bug.cgi?id=57683
>
> Signed-off-by: "Daniel Zimmermann" <netzimme@gmail.com>
>
> diff -r 7e569a9cb5fd -r ea83356683a8 patches/gcc/4.8.1/001-gcc_4_8_1_parallel_build_failure_file_57683.patch
> --- /dev/null   Thu Jan 01 00:00:00 1970 +0000
> +++ b/patches/gcc/4.8.1/001-gcc_4_8_1_parallel_build_failure_file_57683.patch   Sun Nov 17 11:48:24 2013 +0100
> @@ -0,0 +1,13 @@
> +--- a/gcc/Makefile.in  2013-05-22 02:00:49.000000000 +1000
> ++++ b/gcc/Makefile.in  2013-06-23 19:00:25.000000000 +1000
> +@@ -3801,8 +3801,8 @@ s-gtype: build/gengtype$(build_exeext) $
> +       $(STAMP) s-gtype
> +
> + generated_files = config.h tm.h $(TM_P_H) $(TM_H) multilib.h \
> +-       $(simple_generated_h) specs.h \
> +-       tree-check.h genrtl.h insn-modes.h tm-preds.h tm-constrs.h \
> ++       $(simple_generated_h) specs.h tree-check.h insn-opinit.h \
> ++       genrtl.h insn-modes.h tm-preds.h tm-constrs.h \
> +        $(ALL_GTFILES_H) gtype-desc.c gtype-desc.h gcov-iov.h
> +
> + # In order for parallel make to really start compiling the expensive
> diff -r 7e569a9cb5fd -r ea83356683a8 patches/gcc/4.8.2/001-gcc_4_8_2_parallel_build_failure_file_57683.patch
> --- /dev/null   Thu Jan 01 00:00:00 1970 +0000
> +++ b/patches/gcc/4.8.2/001-gcc_4_8_2_parallel_build_failure_file_57683.patch   Sun Nov 17 11:48:24 2013 +0100
> @@ -0,0 +1,13 @@
> +--- a/gcc/Makefile.in  2013-05-22 02:00:49.000000000 +1000
> ++++ b/gcc/Makefile.in  2013-06-23 19:00:25.000000000 +1000
> +@@ -3801,8 +3801,8 @@ s-gtype: build/gengtype$(build_exeext) $
> +       $(STAMP) s-gtype
> +
> + generated_files = config.h tm.h $(TM_P_H) $(TM_H) multilib.h \
> +-       $(simple_generated_h) specs.h \
> +-       tree-check.h genrtl.h insn-modes.h tm-preds.h tm-constrs.h \
> ++       $(simple_generated_h) specs.h tree-check.h insn-opinit.h \
> ++       genrtl.h insn-modes.h tm-preds.h tm-constrs.h \
> +        $(ALL_GTFILES_H) gtype-desc.c gtype-desc.h gcov-iov.h
> +
> + # In order for parallel make to really start compiling the expensive
>
> --
> For unsubscribe information see http://sourceware.org/lists.html#faq
>

--
For unsubscribe information see http://sourceware.org/lists.html#faq
Daniel Zimmermann - Nov. 24, 2013, 11:09 a.m.
Hi

I create a new patch that include Zhenqiang backport patches and test
it. Everything works fine for me.
Please add the last patch from 24 Nov. 11:00 oclock. The other one
include a typo in the commit message.

greetings,
 Daniel



2013/11/21 Zhenqiang Chen <zhenqiang.chen@linaro.org>:
> Nicolas's patch can fix the issue. But it is not committed in 4.8 or
> trunk. It is not a good idea to maintain a uncommitted patch.
>
> I just backported two patches from trunk to fix PR 57683. Please try it.
>
> Thanks!
> -Zhenqiang
>
>
> On 17 November 2013 19:03, Daniel Zimmermann <netzimme@gmail.com> wrote:
>> # HG changeset patch
>> # User Daniel Zimmermann <netzimme@gmail.com>
>> # Date 1384685304 -3600
>> #      Sun Nov 17 11:48:24 2013 +0100
>> # Node ID ea83356683a857e891cbfc4f4b19ee4ca878f5af
>> # Parent  7e569a9cb5fd3ab591bb307328b947a5b7312cba
>> cc/gcc: fix problem with gcc 4.8.1 and gcc 4.8.2 and parallel build
>>
>> This patch add patches from Nicolas Hannekum. It fix the problem with parallel build on arm target.
>> See: http://gcc.gnu.org/bugzilla/show_bug.cgi?id=57683
>>
>> Signed-off-by: "Daniel Zimmermann" <netzimme@gmail.com>
>>
>> diff -r 7e569a9cb5fd -r ea83356683a8 patches/gcc/4.8.1/001-gcc_4_8_1_parallel_build_failure_file_57683.patch
>> --- /dev/null   Thu Jan 01 00:00:00 1970 +0000
>> +++ b/patches/gcc/4.8.1/001-gcc_4_8_1_parallel_build_failure_file_57683.patch   Sun Nov 17 11:48:24 2013 +0100
>> @@ -0,0 +1,13 @@
>> +--- a/gcc/Makefile.in  2013-05-22 02:00:49.000000000 +1000
>> ++++ b/gcc/Makefile.in  2013-06-23 19:00:25.000000000 +1000
>> +@@ -3801,8 +3801,8 @@ s-gtype: build/gengtype$(build_exeext) $
>> +       $(STAMP) s-gtype
>> +
>> + generated_files = config.h tm.h $(TM_P_H) $(TM_H) multilib.h \
>> +-       $(simple_generated_h) specs.h \
>> +-       tree-check.h genrtl.h insn-modes.h tm-preds.h tm-constrs.h \
>> ++       $(simple_generated_h) specs.h tree-check.h insn-opinit.h \
>> ++       genrtl.h insn-modes.h tm-preds.h tm-constrs.h \
>> +        $(ALL_GTFILES_H) gtype-desc.c gtype-desc.h gcov-iov.h
>> +
>> + # In order for parallel make to really start compiling the expensive
>> diff -r 7e569a9cb5fd -r ea83356683a8 patches/gcc/4.8.2/001-gcc_4_8_2_parallel_build_failure_file_57683.patch
>> --- /dev/null   Thu Jan 01 00:00:00 1970 +0000
>> +++ b/patches/gcc/4.8.2/001-gcc_4_8_2_parallel_build_failure_file_57683.patch   Sun Nov 17 11:48:24 2013 +0100
>> @@ -0,0 +1,13 @@
>> +--- a/gcc/Makefile.in  2013-05-22 02:00:49.000000000 +1000
>> ++++ b/gcc/Makefile.in  2013-06-23 19:00:25.000000000 +1000
>> +@@ -3801,8 +3801,8 @@ s-gtype: build/gengtype$(build_exeext) $
>> +       $(STAMP) s-gtype
>> +
>> + generated_files = config.h tm.h $(TM_P_H) $(TM_H) multilib.h \
>> +-       $(simple_generated_h) specs.h \
>> +-       tree-check.h genrtl.h insn-modes.h tm-preds.h tm-constrs.h \
>> ++       $(simple_generated_h) specs.h tree-check.h insn-opinit.h \
>> ++       genrtl.h insn-modes.h tm-preds.h tm-constrs.h \
>> +        $(ALL_GTFILES_H) gtype-desc.c gtype-desc.h gcov-iov.h
>> +
>> + # In order for parallel make to really start compiling the expensive
>>
>> --
>> For unsubscribe information see http://sourceware.org/lists.html#faq
>>

--
For unsubscribe information see http://sourceware.org/lists.html#faq

Patch

diff -r 7e569a9cb5fd -r ea83356683a8 patches/gcc/4.8.1/001-gcc_4_8_1_parallel_build_failure_file_57683.patch
--- /dev/null	Thu Jan 01 00:00:00 1970 +0000
+++ b/patches/gcc/4.8.1/001-gcc_4_8_1_parallel_build_failure_file_57683.patch	Sun Nov 17 11:48:24 2013 +0100
@@ -0,0 +1,13 @@ 
+--- a/gcc/Makefile.in	2013-05-22 02:00:49.000000000 +1000
++++ b/gcc/Makefile.in	2013-06-23 19:00:25.000000000 +1000
+@@ -3801,8 +3801,8 @@ s-gtype: build/gengtype$(build_exeext) $
+ 	$(STAMP) s-gtype
+ 
+ generated_files = config.h tm.h $(TM_P_H) $(TM_H) multilib.h \
+-       $(simple_generated_h) specs.h \
+-       tree-check.h genrtl.h insn-modes.h tm-preds.h tm-constrs.h \
++       $(simple_generated_h) specs.h tree-check.h insn-opinit.h \
++       genrtl.h insn-modes.h tm-preds.h tm-constrs.h \
+        $(ALL_GTFILES_H) gtype-desc.c gtype-desc.h gcov-iov.h
+ 
+ # In order for parallel make to really start compiling the expensive
diff -r 7e569a9cb5fd -r ea83356683a8 patches/gcc/4.8.2/001-gcc_4_8_2_parallel_build_failure_file_57683.patch
--- /dev/null	Thu Jan 01 00:00:00 1970 +0000
+++ b/patches/gcc/4.8.2/001-gcc_4_8_2_parallel_build_failure_file_57683.patch	Sun Nov 17 11:48:24 2013 +0100
@@ -0,0 +1,13 @@ 
+--- a/gcc/Makefile.in	2013-05-22 02:00:49.000000000 +1000
++++ b/gcc/Makefile.in	2013-06-23 19:00:25.000000000 +1000
+@@ -3801,8 +3801,8 @@ s-gtype: build/gengtype$(build_exeext) $
+ 	$(STAMP) s-gtype
+ 
+ generated_files = config.h tm.h $(TM_P_H) $(TM_H) multilib.h \
+-       $(simple_generated_h) specs.h \
+-       tree-check.h genrtl.h insn-modes.h tm-preds.h tm-constrs.h \
++       $(simple_generated_h) specs.h tree-check.h insn-opinit.h \
++       genrtl.h insn-modes.h tm-preds.h tm-constrs.h \
+        $(ALL_GTFILES_H) gtype-desc.c gtype-desc.h gcov-iov.h
+ 
+ # In order for parallel make to really start compiling the expensive