qemu-devel.nongnu.org archive mirror
 help / color / mirror / Atom feed
* [Qemu-devel] [PATCH] Alpha: fix decoding of CVTST /S
@ 2008-11-11 13:23 Laurent Desnogues
  2008-11-11 13:54 ` Aurelien Jarno
  0 siblings, 1 reply; 2+ messages in thread
From: Laurent Desnogues @ 2008-11-11 13:23 UTC (permalink / raw)
  To: qemu-devel

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

Hello,

this patch fixes the decoding of CVTST /S which was
incorrectly being decoded as CVTTS.

This fixes SPEC2000 gcc with 200.i input.


Laurent

Signed-off-by: Laurent Desnogues <laurent.desnogues@gmail.com>

[-- Warning: decoded text below may be mangled, UTF-8 assumed --]
[-- Attachment #2: alpha-cvtst.patch --]
[-- Type: text/x-patch; name=alpha-cvtst.patch, Size: 474 bytes --]

Index: target-alpha/translate.c
===================================================================
--- target-alpha/translate.c	(revision 5670)
+++ target-alpha/translate.c	(working copy)
@@ -1464,7 +1464,7 @@
             break;
         case 0x2C:
             /* XXX: incorrect */
-            if (fn11 == 0x2AC) {
+            if (fn11 == 0x2AC || fn11 == 0x6AC) {
                 /* CVTST */
                 gen_farith2(&helper_cvtst, rb, rc);
             } else {

^ permalink raw reply	[flat|nested] 2+ messages in thread

* Re: [Qemu-devel] [PATCH] Alpha: fix decoding of CVTST /S
  2008-11-11 13:23 [Qemu-devel] [PATCH] Alpha: fix decoding of CVTST /S Laurent Desnogues
@ 2008-11-11 13:54 ` Aurelien Jarno
  0 siblings, 0 replies; 2+ messages in thread
From: Aurelien Jarno @ 2008-11-11 13:54 UTC (permalink / raw)
  To: qemu-devel

On Tue, Nov 11, 2008 at 02:23:44PM +0100, Laurent Desnogues wrote:
> Hello,
> 
> this patch fixes the decoding of CVTST /S which was
> incorrectly being decoded as CVTTS.
> 
> This fixes SPEC2000 gcc with 200.i input.
> 

Thanks, applied.

> Laurent
> 
> Signed-off-by: Laurent Desnogues <laurent.desnogues@gmail.com>

> Index: target-alpha/translate.c
> ===================================================================
> --- target-alpha/translate.c	(revision 5670)
> +++ target-alpha/translate.c	(working copy)
> @@ -1464,7 +1464,7 @@
>              break;
>          case 0x2C:
>              /* XXX: incorrect */
> -            if (fn11 == 0x2AC) {
> +            if (fn11 == 0x2AC || fn11 == 0x6AC) {
>                  /* CVTST */
>                  gen_farith2(&helper_cvtst, rb, rc);
>              } else {


-- 
  .''`.  Aurelien Jarno	            | GPG: 1024D/F1BCDB73
 : :' :  Debian developer           | Electrical Engineer
 `. `'   aurel32@debian.org         | aurelien@aurel32.net
   `-    people.debian.org/~aurel32 | www.aurel32.net

^ permalink raw reply	[flat|nested] 2+ messages in thread

end of thread, other threads:[~2008-11-11 13:55 UTC | newest]

Thread overview: 2+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2008-11-11 13:23 [Qemu-devel] [PATCH] Alpha: fix decoding of CVTST /S Laurent Desnogues
2008-11-11 13:54 ` Aurelien Jarno

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).