[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [Qemu-devel] [PATCH v1 03/24] configure: move i386_cc to cross_cc_i3
From: |
Philippe Mathieu-Daudé |
Subject: |
Re: [Qemu-devel] [PATCH v1 03/24] configure: move i386_cc to cross_cc_i386 |
Date: |
Sun, 15 Apr 2018 21:01:44 -0300 |
User-agent: |
Mozilla/5.0 (X11; Linux x86_64; rv:52.0) Gecko/20100101 Thunderbird/52.7.0 |
Hi Alex,
On 04/10/2018 04:38 PM, Alex Bennée wrote:
> We should still be able to use the system cross compiler with the
> appropriate flags on x86_64 hosts.
>
> Signed-off-by: Alex Bennée <address@hidden>
> ---
> configure | 23 +++++++++++++++--------
> 1 file changed, 15 insertions(+), 8 deletions(-)
>
> diff --git a/configure b/configure
> index b5f3b3fe29..add87ff4d4 100755
> --- a/configure
> +++ b/configure
> @@ -284,7 +284,6 @@ libs_softmmu=""
> libs_tools=""
> audio_pt_int=""
> audio_win_int=""
> -cc_i386=i386-pc-linux-gnu-gcc
> libs_qga=""
> debug_info="yes"
> stack_protector=""
> @@ -457,6 +456,8 @@ docker="no"
> cross_cc_aarch64="aarch64-linux-gnu-gcc"
> cross_cc_arm="arm-linux-gnueabihf-gcc"
> cross_cc_powerpc="powerpc-linux-gnu-gcc"
> +cross_cc_i386="i386-pc-linux-gnu-gcc"
> +cross_cc_i386_cflags=""
Can you keep this sorted? (arm < i386 < powerpc)
Thanks :)
>
> enabled_cross_compilers=""
>
> @@ -687,12 +688,10 @@ case "$cpu" in
What do you think about renaming $cpu -> host_cpu/target_cpu?
> i386|i486|i586|i686|i86pc|BePC)
> cpu="i386"
> supported_cpu="yes"
> - cross_cc_i386=gcc
> ;;
> x86_64|amd64)
> cpu="x86_64"
> supported_cpu="yes"
> - cross_cc_x86_64=gcc
> ;;
> armv*b|armv*l|arm)
> cpu="arm"
> @@ -1435,7 +1434,6 @@ case "$cpu" in
> i386)
> CPU_CFLAGS="-m32"
> LDFLAGS="-m32 $LDFLAGS"
> - cc_i386='$(CC) -m32'
> ;;
> x86_64)
> # ??? Only extremely old AMD cpus do not have cmpxchg16b.
> @@ -1443,12 +1441,14 @@ case "$cpu" in
> # runtime and generate the fallback to serial emulation.
> CPU_CFLAGS="-m64 -mcx16"
> LDFLAGS="-m64 $LDFLAGS"
> - cc_i386='$(CC) -m32'
> + cross_cc_i386=$cc
> + cross_cc_i386_cflags="-m32"
> ;;
> x32)
> CPU_CFLAGS="-mx32"
> LDFLAGS="-mx32 $LDFLAGS"
> - cc_i386='$(CC) -m32'
> + cross_cc_i386=$cc
> + cross_cc_i386_cflags="-m32"
> ;;
> # No special flags required for other host CPUs
> esac
> @@ -6664,7 +6664,6 @@ echo "CC=$cc" >> $config_host_mak
> if $iasl -h > /dev/null 2>&1; then
> echo "IASL=$iasl" >> $config_host_mak
> fi
> -echo "CC_I386=$cc_i386" >> $config_host_mak
> echo "HOST_CC=$host_cc" >> $config_host_mak
> echo "CXX=$cxx" >> $config_host_mak
> echo "OBJCC=$objcc" >> $config_host_mak
> @@ -6783,6 +6782,7 @@ case "$target" in
> esac
>
> target_compiler=""
> +target_compiler_cflags=""
>
> mkdir -p $target_dir
> echo "# Automatically generated by configure - do not modify" >
> $config_target_mak
> @@ -6799,10 +6799,13 @@ TARGET_ABI_DIR=""
> case "$target_name" in
> i386)
> gdb_xml_files="i386-32bit.xml i386-32bit-core.xml i386-32bit-sse.xml"
> + target_compiler=$cross_cc_i386
> + target_compiler_cflags=$cross_cc_i386_cflags
> ;;
> x86_64)
> TARGET_BASE_ARCH=i386
> gdb_xml_files="i386-64bit.xml i386-64bit-core.xml i386-64bit-sse.xml"
> + target_compiler=$cross_cc_x86_64
> ;;
> alpha)
> mttcg="yes"
> @@ -6947,7 +6950,7 @@ int main(void) {
> }
> EOF
>
> - if ! do_compiler $target_compiler -o $TMPE $TMPC -static ; then
> + if ! do_compiler $target_compiler $target_compiler_cflags -o $TMPE $TMPC
> -static ; then
> target_compiler=""
> else
> enabled_cross_compilers="${enabled_cross_compilers}
> ${target_compiler}"
> @@ -7033,6 +7036,10 @@ if test -n "$target_compiler"; then
> echo "CROSS_CC_GUEST=$target_compiler" >> $config_target_mak
> fi
>
> +if test -n "$target_compiler_cflags"; then
> + echo "CROSS_CC_GUEST_CFLAGS=$target_compiler_cflags" >> $config_target_mak
> +fi
> +
> # generate QEMU_CFLAGS/LDFLAGS for targets
>
> cflags=""
>
- [Qemu-devel] [PATCH v1 04/24] Makefile: Rename TARGET_DIRS to TARGET_LIST, (continued)
- [Qemu-devel] [PATCH v1 04/24] Makefile: Rename TARGET_DIRS to TARGET_LIST, Alex Bennée, 2018/04/10
- [Qemu-devel] [PATCH v1 02/24] configure: add support for --cross-cc-FOO, Alex Bennée, 2018/04/10
- [Qemu-devel] [PATCH v1 05/24] docker: Add "cc" subcommand, Alex Bennée, 2018/04/10
- [Qemu-devel] [PATCH v1 08/24] docker: Makefile.include introduce DOCKER_SCRIPT, Alex Bennée, 2018/04/10
- [Qemu-devel] [PATCH v1 03/24] configure: move i386_cc to cross_cc_i386, Alex Bennée, 2018/04/10
- Re: [Qemu-devel] [PATCH v1 03/24] configure: move i386_cc to cross_cc_i386,
Philippe Mathieu-Daudé <=
- [Qemu-devel] [PATCH v1 06/24] docker: extend "cc" command to accept compiler, Alex Bennée, 2018/04/10
- [Qemu-devel] [PATCH v1 07/24] docker: allow "cc" command to run in user context, Alex Bennée, 2018/04/10
- [Qemu-devel] [PATCH v1 09/24] tests/tcg: move architecture independent tests into subdir, Alex Bennée, 2018/04/10