From mboxrd@z Thu Jan 1 00:00:00 1970 Received: from mailman by lists.gnu.org with tmda-scanned (Exim 4.43) id 1IQRya-0004h9-B3 for qemu-devel@nongnu.org; Wed, 29 Aug 2007 14:09:12 -0400 Received: from exim by lists.gnu.org with spam-scanned (Exim 4.43) id 1IQRyX-0004dF-MN for qemu-devel@nongnu.org; Wed, 29 Aug 2007 14:09:11 -0400 Received: from [199.232.76.173] (helo=monty-python.gnu.org) by lists.gnu.org with esmtp (Exim 4.43) id 1IQRyX-0004cx-Du for qemu-devel@nongnu.org; Wed, 29 Aug 2007 14:09:09 -0400 Received: from nf-out-0910.google.com ([64.233.182.184]) by monty-python.gnu.org with esmtp (Exim 4.60) (envelope-from ) id 1IQRyW-0008Vy-R0 for qemu-devel@nongnu.org; Wed, 29 Aug 2007 14:09:09 -0400 Received: by nf-out-0910.google.com with SMTP id 30so242226nfu for ; Wed, 29 Aug 2007 11:09:07 -0700 (PDT) Message-ID: Date: Wed, 29 Aug 2007 21:09:06 +0300 From: "Blue Swirl" Subject: Re: [Qemu-devel] [patch] make qemu work with GCC 4 In-Reply-To: MIME-Version: 1.0 Content-Type: multipart/mixed; boundary="----=_Part_2460_12228405.1188410946632" References: <8AEA1D4C-F488-43F2-B350-D62C3C4C1FD9@web.de> <49F27907-6C55-40D8-A6FF-7769156F8E74@web.de> <46D57BB5.5000302@softhome.net> Reply-To: qemu-devel@nongnu.org List-Id: qemu-devel.nongnu.org List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , To: Michael Matz , Johannes.Schindelin@gmx.de Cc: qemu-devel@nongnu.org ------=_Part_2460_12228405.1188410946632 Content-Type: text/plain; charset=UTF-8 Content-Transfer-Encoding: 7bit Content-Disposition: inline On 8/29/07, Johannes Schindelin wrote: > Hi, > > On Wed, 29 Aug 2007, Michael Matz wrote: > > > On Wed, 29 Aug 2007, Johannes Schindelin wrote: > > > > > > Thanks for your effor Michael! Now, I only hope, one of the patches > > > > that makes qemu gcc4 compliant are soon merged. > > > > > > Well, to throw a spanner in the works: this patch is the 4th patch along > > > the lines that I came about. None of them (AFAICT) was tested well > > > enough to be included in CVS. Indeed, the biggest problem seems to be > > > to make a patch that not only works on the machine of the poster, but on > > > other machines, too. > > > > Please? qemu with that patch builds on four architectures, and tests > > fine on two of them (including cross targets) and only because I haven't > > run it on the other two architectures yet. > > Hey, I am not criticising you! Instead, I am thankful enough that I went > so far as to expose it with git. > > > It might or might not need more patches than just mine (as I started > > from our package), but in that case they are independend of making qemu > > work with gcc 4, which is the only thing my patch is concerned about. > > I already reported the non-compiling state of sparc-linux-user... Any > idea what I could do about it? With the attached patch I can run Sparc32 and Sparc64 emulators on x86_64 host. The performance feels slightly worse. Maybe the patch should be conditional on GCC >= 4? I think that the decision of using host registers vs. env fields should be done in one file at top level. ------=_Part_2460_12228405.1188410946632 Content-Type: text/x-diff; name="sparc-tadd.diff" Content-Transfer-Encoding: base64 Content-Disposition: attachment; filename="sparc-tadd.diff"; filename="sparc-tadd.diff"; filename="sparc-tadd.diff"; filename="sparc-tadd.diff"; filename="sparc-tadd.diff" X-Attachment-Id: f_f5y4zufm SW5kZXg6IHFlbXUvdGFyZ2V0LXNwYXJjL29wLmMKPT09PT09PT09PT09PT09PT09PT09PT09PT09 PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PQotLS0gcWVtdS5vcmlnL3Rh cmdldC1zcGFyYy9vcC5jCTIwMDctMDgtMjkgMTc6NTc6MjEuMDAwMDAwMDAwICswMDAwCisrKyBx ZW11L3RhcmdldC1zcGFyYy9vcC5jCTIwMDctMDgtMjkgMTc6NTg6MDAuMDAwMDAwMDAwICswMDAw CkBAIC01MjAsOCArNTIwLDExIEBACiB7CiAgICAgdGFyZ2V0X3Vsb25nIHNyYzE7CiAKLSAgICBp ZiAoKFQwICYgMHgwMykgfHwgKFQxICYgMHgwMykpCisgICAgaWYgKChUMCAmIDB4MDMpIHx8IChU MSAmIDB4MDMpKSB7CiAgICAgICAgIHJhaXNlX2V4Y2VwdGlvbihUVF9UT1ZGKTsKKyAgICAgICAg Rk9SQ0VfUkVUKCk7CisgICAgICAgIHJldHVybjsKKyAgICB9CiAKICAgICBzcmMxID0gVDA7CiAg ICAgVDAgKz0gVDE7Cg== ------=_Part_2460_12228405.1188410946632--