[Qemu-devel] Solaris amd64 patch

[Qemu-devel] Solaris amd64 patch

[Ben Taylor]

[-- Attachment #1: Type: text/plain, Size: 225 bytes --]


This patch (by Martin Bochnig mb1x@gmx.com) fixes a register
allocation issue on Solaris.

Some supporting evidence and the original patch were posted at:

http://www.opensolaris.org/jive/thread.jspa?threadID=22392&tstart=0

[-- Attachment #2: sol-amd64-reg.diff --]
[-- Type: text/x-patch, Size: 391 bytes --]

diff -ruN qemu-ORIG/dyngen-exec.h qemu/dyngen-exec.h
--- qemu-ORIG/dyngen-exec.h	2006-12-22 19:49:32.000000000 -0500
+++ qemu/dyngen-exec.h	2007-02-02 14:30:56.001770000 -0500
@@ -88,8 +88,8 @@
 #define AREG3 "edi"
 #endif
 #ifdef __x86_64__
-#define AREG0 "rbp"
-#define AREG1 "rbx"
+#define AREG0 "r14"
+#define AREG1 "r15"
 #define AREG2 "r12"
 #define AREG3 "r13"
 //#define AREG4 "r14"

Re: [Qemu-devel] Solaris amd64 patch

[Igor Kovalenko]

[-- Attachment #1: Type: text/plain, Size: 474 bytes --]

On 2/2/07, Ben Taylor <sol10x86@cox.net> wrote:
>
>
> This patch (by Martin Bochnig mb1x@gmx.com) fixes a register
> allocation issue on Solaris.
>
> Some supporting evidence and the original patch were posted at:
>
> http://www.opensolaris.org/jive/thread.jspa?threadID=22392&tstart=0
>
>
This also makes debugging a bit easier (for base pointer is left untouched)
Still it is better to make AREG0 %r13 - the generated code is smaller.

-- 
Kind regards,
Igor V. Kovalenko

[-- Attachment #2: Type: text/html, Size: 901 bytes --]