mirror of
https://git.yoctoproject.org/poky
synced 2026-05-08 05:09:24 +00:00
gcc-4.9.inc: fix parallel building failure
In subdir 'gcc', Most C source files included config.h which was generated by a rule. But no related prerequisites was added to the C source compiling rule. There was potential building failure while makefile enabled parallel. The C source compiling rule used suffix rule '.c.o', but the suffix rule doesn't support prerequisites. https://www.gnu.org/software/make/manual/html_node/Suffix-Rules.html We used the pattern rule '%.o : %.c' to instead, and add the config.h as its prerequisite We also moved the '%.o : %.c' rule down to the 'build/%.o :' rule, which makes '%.o : %.c' rule doesn't override 'build/%.o :'. [YOCTO #6568] (From OE-Core rev: 86c2483f0fe05fb763d280ae22d70e54cb4bb0bc) Signed-off-by: Hongxu Jia <hongxu.jia@windriver.com> Signed-off-by: Saul Wold <sgw@linux.intel.com> Signed-off-by: Richard Purdie <richard.purdie@linuxfoundation.org>
This commit is contained in:
committed by
Richard Purdie
parent
30ef7b2c05
commit
f6507d311a
@@ -67,6 +67,7 @@ SRC_URI = "${GNU_MIRROR}/gcc/gcc-${PV}/gcc-${PV}.tar.bz2 \
|
||||
file://0051-eabispe.patch \
|
||||
file://0052-Fix-GCC-targeting-E500-SPE-errors-with-the-_Decimal64-type.patch \
|
||||
file://0053-gcc-fix-segfault-from-calling-free-on-non-malloc-d-a.patch \
|
||||
file://0054-gcc-Makefile.in-fix-parallel-building-failure.patch \
|
||||
"
|
||||
SRC_URI[md5sum] = "9709b49ae0e904cbb0a6a1b62853b556"
|
||||
SRC_URI[sha256sum] = "b9b047a97bade9c1c89970bc8e211ff57b7b8998a1730a80a653d329f8ed1257"
|
||||
|
||||
+56
@@ -0,0 +1,56 @@
|
||||
gcc/Makefile.in: fix parallel building failure
|
||||
|
||||
Most C source files included config.h which was generated by a rule.
|
||||
But no related prerequisites was added to the C source compiling rule.
|
||||
There was potential building failure while makefile enabled parallel.
|
||||
|
||||
The C source compiling rule used suffix rule '.c.o', but the suffix
|
||||
rule doesn't support prerequisites.
|
||||
https://www.gnu.org/software/make/manual/html_node/Suffix-Rules.html
|
||||
|
||||
We used the pattern rule '%.o : %.c' to instead, and add the config.h
|
||||
as its prerequisite
|
||||
|
||||
We also moved the '%.o : %.c' rule down to the 'build/%.o :' rule, which
|
||||
makes '%.o : %.c' rule doesn't override 'build/%.o :'.
|
||||
|
||||
Upstream-Status: Pending
|
||||
|
||||
Signed-off-by: Hongxu Jia <hongxu.jia@windriver.com>
|
||||
---
|
||||
gcc/Makefile.in | 12 ++++++++----
|
||||
1 file changed, 8 insertions(+), 4 deletions(-)
|
||||
|
||||
diff --git a/gcc/Makefile.in b/gcc/Makefile.in
|
||||
index 6475cba..04889fe 100644
|
||||
--- a/gcc/Makefile.in
|
||||
+++ b/gcc/Makefile.in
|
||||
@@ -1054,10 +1054,6 @@ COMPILE = source='$<' object='$@' libtool=no \
|
||||
POSTCOMPILE =
|
||||
endif
|
||||
|
||||
-.cc.o .c.o:
|
||||
- $(COMPILE) $<
|
||||
- $(POSTCOMPILE)
|
||||
-
|
||||
#
|
||||
# Support for additional languages (other than C).
|
||||
# C can be supported this way too (leave for later).
|
||||
@@ -2342,6 +2338,14 @@ build/%.o : # dependencies provided by explicit rule later
|
||||
$(COMPILER_FOR_BUILD) -c $(BUILD_COMPILERFLAGS) $(BUILD_CPPFLAGS) \
|
||||
-o $@ $<
|
||||
|
||||
+%.o: %.c $(CONFIG_H)
|
||||
+ $(COMPILE) $<
|
||||
+ $(POSTCOMPILE)
|
||||
+
|
||||
+%.o: %.cc $(CONFIG_H)
|
||||
+ $(COMPILE) $<
|
||||
+ $(POSTCOMPILE)
|
||||
+
|
||||
## build/version.o is compiled by the $(COMPILER_FOR_BUILD) but needs
|
||||
## several C macro definitions, just like version.o
|
||||
build/version.o: version.c version.h \
|
||||
--
|
||||
1.8.1.2
|
||||
|
||||
Reference in New Issue
Block a user