qemu-devel.nongnu.org archive mirror
 help / color / mirror / Atom feed
From: "Alex Bennée" <alex.bennee@linaro.org>
To: peter.maydell@linaro.org
Cc: qemu-devel@nongnu.org, cota@braap.org, famz@redhat.com,
	berrange@redhat.com, f4bug@amsat.org,
	richard.henderson@linaro.org, balrogg@gmail.com,
	aurelien@aurel32.net, agraf@suse.de,
	"Alex Bennée" <alex.bennee@linaro.org>
Subject: [Qemu-devel] [PULL 02/56] configure: move i386_cc to cross_cc_i386
Date: Tue, 19 Jun 2018 16:43:41 +0100	[thread overview]
Message-ID: <20180619154435.18898-3-alex.bennee@linaro.org> (raw)
In-Reply-To: <20180619154435.18898-1-alex.bennee@linaro.org>

Also dont assume x86_64 compiler can build i386 binaries.

Signed-off-by: Alex Bennée <alex.bennee@linaro.org>
Tested-by: Philippe Mathieu-Daudé <f4bug@amsat.org>
Reviewed-by: Richard Henderson <richard.henderson@linaro.org>
---
v2
  - drop using system x86_64 compiler for i386 compiles
v3
  - fix cross_cc_i386 when on i386
v4
  - fix merge conflicts
  - sort cross_cc defines

diff --git a/configure b/configure
index 089de03918..6e33dbbe10 100755
--- a/configure
+++ b/configure
@@ -289,7 +289,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=""
@@ -461,6 +460,8 @@ docker="no"
 # cross compilers defaults, can be overridden with --cross-cc-ARCH
 cross_cc_aarch64="aarch64-linux-gnu-gcc"
 cross_cc_arm="arm-linux-gnueabihf-gcc"
+cross_cc_i386="i386-pc-linux-gnu-gcc"
+cross_cc_cflags_i386=""
 cross_cc_powerpc="powerpc-linux-gnu-gcc"
 
 enabled_cross_compilers=""
@@ -1447,7 +1448,8 @@ case "$cpu" in
     i386)
            CPU_CFLAGS="-m32"
            LDFLAGS="-m32 $LDFLAGS"
-           cc_i386='$(CC) -m32'
+           cross_cc_i386=$cc
+           cross_cc_cflags_i386=$CPU_CFLAGS
            ;;
     x86_64)
            # ??? Only extremely old AMD cpus do not have cmpxchg16b.
@@ -1455,12 +1457,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_x86_64=$cc
+           cross_cc_cflags_x86_64=$CPU_CFLAGS
            ;;
     x32)
            CPU_CFLAGS="-mx32"
            LDFLAGS="-mx32 $LDFLAGS"
-           cc_i386='$(CC) -m32'
+           cross_cc_i386=$cc
+           cross_cc_cflags_i386="-m32"
            ;;
     # No special flags required for other host CPUs
 esac
@@ -6723,7 +6727,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
@@ -6844,6 +6847,7 @@ esac
 
 target_compiler=""
 target_compiler_static=""
+target_compiler_cflags=""
 
 mkdir -p $target_dir
 echo "# Automatically generated by configure - do not modify" > $config_target_mak
@@ -6861,6 +6865,7 @@ 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_ccflags_i386
   ;;
   x86_64)
     TARGET_BASE_ARCH=i386
@@ -7033,9 +7038,9 @@ if has $target_compiler; then
 
     write_c_skeleton
 
-    if ! do_compiler $target_compiler -o $TMPE $TMPC -static ; then
+    if ! do_compiler $target_compiler $target_compiler_cflags -o $TMPE $TMPC -static ; then
         # For host systems we might get away with building without -static
-        if ! do_compiler $target_compiler -o $TMPE $TMPC ; then
+        if ! do_compiler $target_compiler $target_compiler_cflags -o $TMPE $TMPC ; then
             target_compiler=""
         else
             enabled_cross_compilers="${enabled_cross_compilers} ${target_compiler}"
@@ -7128,8 +7133,13 @@ if test -n "$target_compiler"; then
   if test -n "$target_compiler_static"; then
       echo "CROSS_CC_GUEST_STATIC=$target_compiler_static" >> $config_target_mak
   fi
+
+  if test -n "$target_compiler_cflags"; then
+      echo "CROSS_CC_GUEST_CFLAGS=$target_compiler_cflags" >> $config_target_mak
+  fi
 fi
 
+
 # generate QEMU_CFLAGS/LDFLAGS for targets
 
 cflags=""
-- 
2.17.1

  parent reply	other threads:[~2018-06-19 15:44 UTC|newest]

Thread overview: 63+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2018-06-19 15:43 [Qemu-devel] [PULL 00/56] add check-tcg and associated machinery Alex Bennée
2018-06-19 15:43 ` [Qemu-devel] [PULL 01/56] configure: add support for --cross-cc-FOO Alex Bennée
2018-06-20 15:22   ` Peter Maydell
2018-06-19 15:43 ` Alex Bennée [this message]
2018-06-19 15:43 ` [Qemu-devel] [PULL 03/56] configure: allow user to specify --cross-cc-cflags-foo= Alex Bennée
2018-06-19 15:43 ` [Qemu-devel] [PULL 04/56] configure: set cross_cc_FOO for host compiler Alex Bennée
2018-06-19 15:43 ` [Qemu-devel] [PULL 05/56] docker: Add "cc" subcommand Alex Bennée
2018-06-19 15:43 ` [Qemu-devel] [PULL 06/56] docker: extend "cc" command to accept compiler Alex Bennée
2018-06-19 15:43 ` [Qemu-devel] [PULL 07/56] docker: allow "cc" command to run in user context Alex Bennée
2018-06-19 15:43 ` [Qemu-devel] [PULL 08/56] docker: Makefile.include introduce DOCKER_SCRIPT Alex Bennée
2018-06-19 15:43 ` [Qemu-devel] [PULL 09/56] tests/tcg: move architecture independent tests into subdir Alex Bennée
2018-06-19 15:43 ` [Qemu-devel] [PULL 10/56] tests/tcg/multiarch: Build fix for linux-test Alex Bennée
2018-06-19 15:43 ` [Qemu-devel] [PULL 11/56] tests/tcg/multiarch: enable additional linux-test tests Alex Bennée
2018-06-19 15:43 ` [Qemu-devel] [PULL 12/56] tests/tcg/multiarch: don't hard code paths/ports for linux-test Alex Bennée
2018-06-19 15:43 ` [Qemu-devel] [PULL 13/56] tests/tcg/multiarch: move most output to stdout Alex Bennée
2018-06-19 15:43 ` [Qemu-devel] [PULL 14/56] tests/tcg: move i386 specific tests into subdir Alex Bennée
2018-06-19 15:43 ` [Qemu-devel] [PULL 15/56] tests/tcg: enable building for i386 Alex Bennée
2018-06-19 15:43 ` [Qemu-devel] [PULL 16/56] tests/tcg/i386: Build fix for hello-i386 Alex Bennée
2018-06-19 15:43 ` [Qemu-devel] [PULL 17/56] tests/tcg/i386: fix test-i386 Alex Bennée
2018-06-19 15:43 ` [Qemu-devel] [PULL 18/56] tests/tcg/i386: add runner for test-i386-fprem Alex Bennée
2018-06-19 15:43 ` [Qemu-devel] [PULL 19/56] tests/tcg/x86_64: add Makefile.target Alex Bennée
2018-06-19 15:43 ` [Qemu-devel] [PULL 20/56] tests/tcg/i386/test-i386: use modern vector_size attributes Alex Bennée
2018-06-19 15:44 ` [Qemu-devel] [PULL 21/56] tests/tcg/i386/test-i386: fix printf format Alex Bennée
2018-06-19 15:44 ` [Qemu-devel] [PULL 22/56] tests/tcg: move ARM specific tests into subdir Alex Bennée
2018-06-19 15:44 ` [Qemu-devel] [PULL 23/56] tests/tcg: enable building for ARM Alex Bennée
2018-06-19 15:44 ` [Qemu-devel] [PULL 24/56] tests/tcg/arm: fix up test-arm-iwmmxt test Alex Bennée
2018-06-19 15:44 ` [Qemu-devel] [PULL 25/56] tests/tcg: enable building for AArch64 Alex Bennée
2018-06-19 15:44 ` [Qemu-devel] [PULL 26/56] tests/tcg/arm: add fcvt test cases for AArch32/64 Alex Bennée
2018-06-19 15:44 ` [Qemu-devel] [PULL 27/56] tests/tcg: move MIPS specific tests into subdir Alex Bennée
2018-06-19 15:44 ` [Qemu-devel] [PULL 28/56] tests/tcg: enable building for MIPS Alex Bennée
2018-06-19 15:44 ` [Qemu-devel] [PULL 29/56] tests/docker/Makefile.include: fix mipsel-cross dependancy Alex Bennée
2018-06-19 15:44 ` [Qemu-devel] [PULL 30/56] tests/tcg/mips: include common mips hello-mips Alex Bennée
2018-06-19 15:44 ` [Qemu-devel] [PULL 31/56] tests/tcg: enable building for s390x Alex Bennée
2018-06-19 15:44 ` [Qemu-devel] [PULL 32/56] tests/tcg: enable building for ppc64 Alex Bennée
2018-06-19 15:44 ` [Qemu-devel] [PULL 33/56] tests/tcg: enable building for Alpha Alex Bennée
2018-06-19 15:44 ` [Qemu-devel] [PULL 34/56] tests/tcg/alpha: add Alpha specific tests Alex Bennée
2018-06-19 15:44 ` [Qemu-devel] [PULL 35/56] tests/tcg: enable building for HPPA Alex Bennée
2018-06-19 15:44 ` [Qemu-devel] [PULL 36/56] tests/tcg: enable building for m68k Alex Bennée
2018-06-19 15:44 ` [Qemu-devel] [PULL 37/56] tests/tcg: enable building for sh4 Alex Bennée
2018-06-19 15:44 ` [Qemu-devel] [PULL 38/56] tests/tcg: enable building for sparc64 Alex Bennée
2018-06-19 15:44 ` [Qemu-devel] [PULL 39/56] tests/tcg: enable building for mips64 Alex Bennée
2018-06-19 15:44 ` [Qemu-devel] [PULL 40/56] tests/tcg: enable building for RISCV64 Alex Bennée
2018-06-19 15:44 ` [Qemu-devel] [PULL 41/56] docker: move debian-powerpc-cross to sid based build Alex Bennée
2018-06-19 15:44 ` [Qemu-devel] [PULL 42/56] tests/tcg: enable building for PowerPC Alex Bennée
2018-06-19 15:44 ` [Qemu-devel] [PULL 43/56] tests/tcg/Makefile: update to be called from Makefile.target Alex Bennée
2018-06-19 15:44 ` [Qemu-devel] [PULL 44/56] Makefile.target: add (clean-/build-)guest-tests targets Alex Bennée
2018-06-19 15:44 ` [Qemu-devel] [PULL 45/56] tests/Makefile.include: add [build|clean|check]-tcg targets Alex Bennée
2018-06-19 15:44 ` [Qemu-devel] [PULL 46/56] tests/tcg: add run, diff, and skip helper macros Alex Bennée
2018-06-19 15:44 ` [Qemu-devel] [PULL 47/56] tests/tcg: override runners for broken tests Alex Bennée
2018-06-19 15:44 ` [Qemu-devel] [PULL 48/56] tests/tcg/i386: extend timeout for runcom test Alex Bennée
2018-06-19 15:44 ` [Qemu-devel] [PULL 49/56] tests: add top-level make dependency for docker builds Alex Bennée
2018-06-19 15:44 ` [Qemu-devel] [PULL 50/56] docker: docker.py use "version" to probe usage Alex Bennée
2018-06-19 15:44 ` [Qemu-devel] [PULL 51/56] docker: docker.py don't conflate checksums for extra_files Alex Bennée
2018-06-19 15:44 ` [Qemu-devel] [PULL 52/56] docker: docker.py add check sub-command Alex Bennée
2018-06-19 15:44 ` [Qemu-devel] [PULL 53/56] tests/Makefile: call sub-makes with SKIP_DOCKER_BUILD=1 Alex Bennée
2018-06-19 15:44 ` [Qemu-devel] [PULL 54/56] docker: docker.py adding age check command Alex Bennée
2018-06-19 15:44 ` [Qemu-devel] [PULL 55/56] tests/docker/Makefile.include: only force SID to NOCACHE if old Alex Bennée
2018-06-19 15:44 ` [Qemu-devel] [PULL 56/56] .travis.yml: add check-tcg test Alex Bennée
2018-06-20 10:43 ` [Qemu-devel] [PULL 00/56] add check-tcg and associated machinery Peter Maydell
2018-06-20 14:09   ` Alex Bennée
2018-06-20 14:13   ` Daniel P. Berrangé
2018-06-20 14:21 ` Peter Maydell
2018-06-20 14:34   ` Alex Bennée

Reply instructions:

You may reply publicly to this message via plain-text email
using any one of the following methods:

* Save the following mbox file, import it into your mail client,
  and reply-to-all from there: mbox

  Avoid top-posting and favor interleaved quoting:
  https://en.wikipedia.org/wiki/Posting_style#Interleaved_style

* Reply using the --to, --cc, and --in-reply-to
  switches of git-send-email(1):

  git send-email \
    --in-reply-to=20180619154435.18898-3-alex.bennee@linaro.org \
    --to=alex.bennee@linaro.org \
    --cc=agraf@suse.de \
    --cc=aurelien@aurel32.net \
    --cc=balrogg@gmail.com \
    --cc=berrange@redhat.com \
    --cc=cota@braap.org \
    --cc=f4bug@amsat.org \
    --cc=famz@redhat.com \
    --cc=peter.maydell@linaro.org \
    --cc=qemu-devel@nongnu.org \
    --cc=richard.henderson@linaro.org \
    /path/to/YOUR_REPLY

  https://kernel.org/pub/software/scm/git/docs/git-send-email.html

* If your mail client supports setting the In-Reply-To header
  via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).