* [Qemu-devel] Solaris amd64 patch
@ 2007-02-02 20:15 Ben Taylor
2007-02-02 20:53 ` Igor Kovalenko
0 siblings, 1 reply; 2+ messages in thread
From: Ben Taylor @ 2007-02-02 20:15 UTC (permalink / raw)
To: Qemu-devel
[-- 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"
^ permalink raw reply [flat|nested] 2+ messages in thread
* Re: [Qemu-devel] Solaris amd64 patch
2007-02-02 20:15 [Qemu-devel] Solaris amd64 patch Ben Taylor
@ 2007-02-02 20:53 ` Igor Kovalenko
0 siblings, 0 replies; 2+ messages in thread
From: Igor Kovalenko @ 2007-02-02 20:53 UTC (permalink / raw)
To: sol10x86, qemu-devel
[-- 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 --]
^ permalink raw reply [flat|nested] 2+ messages in thread
end of thread, other threads:[~2007-02-02 20:53 UTC | newest]
Thread overview: 2+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2007-02-02 20:15 [Qemu-devel] Solaris amd64 patch Ben Taylor
2007-02-02 20:53 ` Igor Kovalenko
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.