[Buildroot] [PATCH] ext-toolchain: Automatically set -m64 if the architecture is x86_64

[Thomas Petazzoni <thomas.petazzoni@free-electrons.com>]

Hello Arnout,

Coming back to this patch, I had a closer look this week-end as I
wanted to integrate it, but I have some comments.

> --- a/toolchain/toolchain-external/Config.in
> +++ b/toolchain/toolchain-external/Config.in
> @@ -341,16 +341,8 @@ config
> BR2_TOOLCHAIN_EXTERNAL_CODESOURCERY_X86_201109 Sourcery CodeBench
> toolchain for the x86/x86_64 architectures, from Mentor Graphics. It
> uses gcc 4.6.1, binutils 2.21.53, glibc 2.13, gdb 7.2.50 and kernel
> headers
> -	  3.0.1. It has support for the following variants:
> -	    - Intel Pentium 4, glibc, 32 bits
> -              Default, nothing special to do.
> -            - Intel Atom, glibc, 32 bits
> -              Select an Atom core
> -            - Intel Xeon, glibc, 64 bits
> -              Set BR2_TARGET_OPTIMIZATION to -m64
> -            - Intel Core 2, glibc, 64 bits
> -              Select a Core 2 core
> -              Set BR2_TARGET_OPTIMIZATION to -m64
> +	  3.0.1. It has support for all the i386 and x86_64 variants
> +	  mentioned in the Target Architecture Variant menu.

I disagree with this change. If you select "i386" or "i486" as the
Target Architecture Variant, then this Sourcery CodeBench toolchain
will not work: it only has libc variants for Pentium 4, Atom, Xeon and
Core 2, so if you try to use it on something older than Pentium 4, it
might use nonexistent instructions.

Can you explain why you made this change, or revert it?

> +ifeq ($(call qstrip,$(BR2_ARCH_IS_64)),y)
> +TOOLCHAIN_EXTERNAL_CFLAGS += -m64
> +TOOLCHAIN_EXTERNAL_WRAPPER_ARGS += -DBR_64
> +endif

I don't agree here. -m64 is not an option that exists for all
64-bits architectures. For example, it does not exist on MIPS. -m64 is
part of the architecture-specific options of gcc, so your change should
rather be:

ifeq ($(BR2_x86_64),y)
TOOLCHAIN_EXTERNAL_CFLAGS += -m64
TOOLCHAIN_EXTERNAL_WRAPPER_ARGS += -DBR_64
endif

(The 'qstrip' is useless)

Regards,

Thomas
-- 
Thomas Petazzoni, Free Electrons
Kernel, drivers, real-time and embedded Linux
development, consulting, training and support.
http://free-electrons.com