From: Aurelien Jarno <aurelien@aurel32.net>
To: qemu-devel@nongnu.org
Subject: [Qemu-devel] [4337] Qemu 32-bit i386, gcc >= 3.4 spill error fix
Date: Mon, 05 May 2008 06:00:28 +0000 [thread overview]
Message-ID: <E1JstkS-000235-Oe@cvs.savannah.gnu.org> (raw)
Revision: 4337
http://svn.sv.gnu.org/viewvc/?view=rev&root=qemu&revision=4337
Author: aurel32
Date: 2008-05-05 06:00:27 +0000 (Mon, 05 May 2008)
Log Message:
-----------
Qemu 32-bit i386, gcc >= 3.4 spill error fix
(Ben Taylor)
Modified Paths:
--------------
trunk/Makefile.target
trunk/configure
Modified: trunk/Makefile.target
===================================================================
--- trunk/Makefile.target 2008-05-04 21:42:11 UTC (rev 4336)
+++ trunk/Makefile.target 2008-05-05 06:00:27 UTC (rev 4337)
@@ -96,7 +96,13 @@
ifeq ($(ARCH),i386)
HELPER_CFLAGS+=-fomit-frame-pointer
OP_CFLAGS+=-mpreferred-stack-boundary=2 -fomit-frame-pointer
+# op.c and helper.c need this on 32-bit x86 system to avoid
+# a compiler spill error. This can probably go away
+# once the SSE ops have been converted to TCG
+ifeq ($(HAVE_GT_GCC_3_3), true)
+I386_CFLAGS=-march=i586 -mtune=i686
endif
+endif
ifeq ($(ARCH),ppc)
CPPFLAGS+= -D__powerpc__
@@ -309,7 +315,7 @@
$(DYNGEN) -g -o $@ $<
op.o: op.c
- $(CC) $(OP_CFLAGS) $(CPPFLAGS) -c -o $@ $<
+ $(CC) $(OP_CFLAGS) $(CPPFLAGS) $(I386_CFLAGS) -c -o $@ $<
machine.o: machine.c
$(CC) $(OP_CFLAGS) $(CPPFLAGS) -c -o $@ $<
@@ -319,7 +325,7 @@
ifeq ($(TARGET_BASE_ARCH), i386)
# XXX: rename helper.c to op_helper.c
helper.o: helper.c
- $(CC) $(HELPER_CFLAGS) $(CPPFLAGS) $(CFLAGS) -c -o $@ $<
+ $(CC) $(HELPER_CFLAGS) $(CPPFLAGS) $(CFLAGS) $(I386_CFLAGS) -c -o $@ $<
else
op_helper.o: op_helper.c
$(CC) $(HELPER_CFLAGS) $(CPPFLAGS) $(CFLAGS) -c -o $@ $<
Modified: trunk/configure
===================================================================
--- trunk/configure 2008-05-04 21:42:11 UTC (rev 4336)
+++ trunk/configure 2008-05-05 06:00:27 UTC (rev 4337)
@@ -1199,6 +1199,13 @@
then
echo "#define USE_KQEMU 1" >> $config_h
fi
+ gcc3minver=`$cc --version 2> /dev/null| fgrep "(GCC) 3." | awk '{ print $3 }' | cut -f2 -d.`
+ if test -n "$gcc3minver" -a $gcc3minver -gt 3
+ then
+ echo "HAVE_GT_GCC_3_3=true" >> $config_mak
+ else
+ echo "HAVE_GT_GCC_3_3=false" >> $config_mak
+ fi
;;
x86_64)
echo "TARGET_ARCH=x86_64" >> $config_mak
reply other threads:[~2008-05-05 6:00 UTC|newest]
Thread overview: [no followups] expand[flat|nested] mbox.gz Atom feed
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=E1JstkS-000235-Oe@cvs.savannah.gnu.org \
--to=aurelien@aurel32.net \
--cc=qemu-devel@nongnu.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).