From: Paul Brook <paul@codesourcery.com>
To: qemu-devel@nongnu.org
Cc: Michael Matz <matz@suse.de>, Alexander Graf <agraf@suse.de>
Subject: Re: [Qemu-devel] [patch] make qemu work with GCC 4
Date: Wed, 29 Aug 2007 16:06:10 +0100 [thread overview]
Message-ID: <200708291606.14173.paul@codesourcery.com> (raw)
In-Reply-To: <Pine.LNX.4.64.0708282124590.23011@wotan.suse.de>
> I solved that by placing one of the T[012] operands into memory
> for HOST_I386, thereby freeing one reg. Here's some justification
> of why that doesn't really cost performance: with three free regs
> GCC is already spilling like mad in the snippets, we just trade one
> of those memory accesses (to stack) with one other mem access to
> the cpu_state structure, which will be in cache.
Do you have any evidence to support this claim? Last time I did this it caused
a significant performance hit. I'd guess that most common ops are simple
enough that we don't need more than 3 registers.
> --- qemu-0.9.0.cvs.orig/softmmu_header.h
> - : "%eax", "%ecx", "%edx", "memory", "cc");
> + : "%eax", "%edx", "memory", "cc");
This change is wrong. The inline asm calls C code which clobbers %ecx.
Paul
next prev parent reply other threads:[~2007-08-29 15:06 UTC|newest]
Thread overview: 26+ messages / expand[flat|nested] mbox.gz Atom feed top
2007-08-28 19:57 [Qemu-devel] [patch] make qemu work with GCC 4 Michael Matz
2007-08-29 8:41 ` Andreas Färber
2007-08-29 11:40 ` Michael Matz
2007-08-29 13:14 ` Andreas Färber
2007-08-29 13:30 ` Michael Matz
2007-08-29 13:59 ` Mulyadi Santosa
2007-08-29 14:11 ` Johannes Schindelin
2007-08-29 16:40 ` Michael Matz
2007-08-29 16:55 ` Johannes Schindelin
2007-08-29 18:09 ` Blue Swirl
2007-08-30 12:46 ` Carlo Marcelo Arenas Belon
2007-08-29 13:59 ` Johannes Schindelin
2007-08-29 14:13 ` Ronald
2007-08-29 14:19 ` Johannes Schindelin
2007-08-29 14:38 ` Andreas Färber
2007-08-29 14:27 ` Andreas Färber
2007-08-29 11:08 ` Johannes Schindelin
2007-08-29 11:46 ` Michael Matz
2007-08-29 12:40 ` Johannes Schindelin
2007-08-29 15:06 ` Paul Brook [this message]
2007-08-29 17:29 ` Michael Matz
2007-08-30 16:52 ` Michael Matz
2007-08-29 18:08 ` Anthony Liguori
2007-08-30 20:28 ` Thiemo Seufer
2007-08-31 13:31 ` Michael Matz
2007-08-31 14:17 ` Thiemo Seufer
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=200708291606.14173.paul@codesourcery.com \
--to=paul@codesourcery.com \
--cc=agraf@suse.de \
--cc=matz@suse.de \
--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 an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.