[Buildroot] [git commit] package/oprofile: fix configure test with gcc 4.9.x

Thomas Petazzoni thomas.petazzoni at free-electrons.com
Sun Aug 17 18:58:05 UTC 2014


commit: http://git.buildroot.net/buildroot/commit/?id=702a461c4dd7ab6d0bf104d3dbad3944af5c294a
branch: http://git.buildroot.net/buildroot/commit/?id=refs/heads/master

Backport the upstream patch that fix a configure test breaked with gcc 4.9.x.

This patch must be removed with the next version bump.

Fixes:
http://autobuild.buildroot.net/results/f80/f8083f038d0efb7b5becf7810b6fcaf440fb8294/build-end.log

Signed-off-by: Romain Naour <romain.naour at openwide.fr>
Signed-off-by: Thomas Petazzoni <thomas.petazzoni at free-electrons.com>
---
 ...e-fix-test-for-synth-check-with-GCC-4.9.0.patch |   45 ++++++++++++++++++++
 1 files changed, 45 insertions(+), 0 deletions(-)

diff --git a/package/oprofile/oprofile-003-configure-fix-test-for-synth-check-with-GCC-4.9.0.patch b/package/oprofile/oprofile-003-configure-fix-test-for-synth-check-with-GCC-4.9.0.patch
new file mode 100644
index 0000000..24bed98
--- /dev/null
+++ b/package/oprofile/oprofile-003-configure-fix-test-for-synth-check-with-GCC-4.9.0.patch
@@ -0,0 +1,45 @@
+From cc08b52f8376867121f22e166636779e2a1a6e48 Mon Sep 17 00:00:00 2001
+From: Aaro Koskinen <aaro.koskinen at iki.fi>
+Date: Mon, 12 May 2014 08:28:46 -0500
+Subject: [PATCH 1/1] configure: fix test-for-synth check with GCC 4.9.0
+
+With GCC 4.9.0 oprofile 0.9.9 build fails on non-PPC platfroms because
+the "test-for-synth" configure check result is incorrect: There is a NULL
+pointer dereference in the test program, so the compiler seems to optimize
+the rest of the code away, and the test will always succeed regardless
+whether powerpc_elf64_vec/bfd_elf64_powerpc_vec are present or not.
+Fix by allocating the referred struct statically.
+
+While at it, also include stdio.h to avoid a compiler warning.
+
+[Romain: backport the patch to 0.9.9]
+
+Signed-off-by: Aaro Koskinen <aaro.koskinen at iki.fi>
+Signed-off-by: Romain Naour <romain.naour at openwide.fr>
+
+Conflicts:
+	m4/binutils.m4
+---
+ m4/binutils.m4 | 6 ++++--
+ 1 file changed, 4 insertions(+), 2 deletions(-)
+
+diff --git a/m4/binutils.m4 b/m4/binutils.m4
+index 25fb15a..3486488 100644
+--- a/m4/binutils.m4
++++ b/m4/binutils.m4
+@@ -27,8 +27,10 @@ if test "$OS" = "Linux"; then
+ 	AC_MSG_CHECKING([whether bfd_get_synthetic_symtab() exists in BFD library])
+ 	rm -f test-for-synth
+ 	AC_LANG_CONFTEST(
+-		[AC_LANG_PROGRAM([[#include <bfd.h>]],
+-			[[asymbol * synthsyms;	bfd * ibfd = 0; 
++		[AC_LANG_PROGRAM([[#include <bfd.h>]
++			[#include <stdio.h>]
++			[static bfd _ibfd;]],
++			[[asymbol * synthsyms;	bfd * ibfd = &_ibfd;
+ 			long synth_count = bfd_get_synthetic_symtab(ibfd, 0, 0, 0, 0, &synthsyms);
+ 			extern const bfd_target bfd_elf64_powerpc_vec;
+ 			extern const bfd_target bfd_elf64_powerpcle_vec;
+-- 
+1.9.3
+


More information about the buildroot mailing list