diff mbox

xvisor: needs gcc >= 4.9

Message ID 1474721260-6810-1-git-send-email-eric.le.bihan.dev@free.fr
State Accepted
Headers show

Commit Message

Eric Le Bihan Sept. 24, 2016, 12:47 p.m. UTC
xvisor needs gcc >= 4.9, as previous versions trigger an ICE on
dwarf2out_frame_debug_adjust_cfa() [1].

Fixes:

 - http://autobuild.buildroot.net/results/fdb/fdb5a568fb38a9d20780cd37eecb71b60b6ca96a/

[1] https://gcc.gnu.org/bugzilla/show_bug.cgi?id=60264

Signed-off-by: Eric Le Bihan <eric.le.bihan.dev@free.fr>
---
 package/xvisor/Config.in | 7 +++++++
 1 file changed, 7 insertions(+)

Comments

Thomas Petazzoni Sept. 25, 2016, 8 p.m. UTC | #1
Hello,

On Sat, 24 Sep 2016 14:47:40 +0200, Eric Le Bihan wrote:
> xvisor needs gcc >= 4.9, as previous versions trigger an ICE on
> dwarf2out_frame_debug_adjust_cfa() [1].
> 
> Fixes:
> 
>  - http://autobuild.buildroot.net/results/fdb/fdb5a568fb38a9d20780cd37eecb71b60b6ca96a/
> 
> [1] https://gcc.gnu.org/bugzilla/show_bug.cgi?id=60264
> 
> Signed-off-by: Eric Le Bihan <eric.le.bihan.dev@free.fr>
> ---
>  package/xvisor/Config.in | 7 +++++++
>  1 file changed, 7 insertions(+)

I've applied to master, after adding before a patch that adds a
BR2_PACKAGE_XVISOR_ARCH_SUPPORTS option, in order to avoid repeating
the architecture dependencies between the main option and the comment.

Thanks a lot!

Thomas
diff mbox

Patch

diff --git a/package/xvisor/Config.in b/package/xvisor/Config.in
index f450f6f..92e8937 100644
--- a/package/xvisor/Config.in
+++ b/package/xvisor/Config.in
@@ -1,6 +1,7 @@ 
 menuconfig BR2_PACKAGE_XVISOR
 	bool "xvisor"
 	depends on BR2_USE_MMU
+	depends on BR2_TOOLCHAIN_GCC_AT_LEAST_4_9
 	depends on BR2_arm || BR2_aarch64 || BR2_x86_64
 	depends on !BR2_ARM_CPU_ARMV4 && !BR2_ARM_CPU_ARMV7M
 	help
@@ -56,3 +57,9 @@  config BR2_PACKAGE_XVISOR_BUILD_TEST_DTB
 	  Build test device-tree blobs for popular boards.
 
 endif
+
+comment "xvisor needs a toolchain w/ gcc >= 4.9"
+	depends on !BR2_TOOLCHAIN_GCC_AT_LEAST_4_9
+	depends on BR2_USE_MMU
+	depends on BR2_arm || BR2_aarch64 || BR2_x86_64
+	depends on !BR2_ARM_CPU_ARMV4 && !BR2_ARM_CPU_ARMV7M