[Valgrind-developers] valgrind: r5446 - branches/VALGRIND_3_1_BRANCH/coregrind/m_dispatch

[<sv...@va...>]

Author: sewardj
Date: 2005-12-27 14:03:47 +0000 (Tue, 27 Dec 2005)
New Revision: 5446

Log:
Merge r5262 (fix for: fsub 3,3,3 in ppc32 dispatcher doesn't clear NaNs)

Modified:
   branches/VALGRIND_3_1_BRANCH/coregrind/m_dispatch/dispatch-ppc32-linux=
.S


Modified: branches/VALGRIND_3_1_BRANCH/coregrind/m_dispatch/dispatch-ppc3=
2-linux.S
=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=
=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=
=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D
--- branches/VALGRIND_3_1_BRANCH/coregrind/m_dispatch/dispatch-ppc32-linu=
x.S	2005-12-27 02:08:03 UTC (rev 5445)
+++ branches/VALGRIND_3_1_BRANCH/coregrind/m_dispatch/dispatch-ppc32-linu=
x.S	2005-12-27 14:03:47 UTC (rev 5446)
@@ -163,8 +163,8 @@
         stw     3,28(1)       /* spill orig guest_state ptr */
=20
         /* 24(sp) used later to stop ctr reg being clobbered */
-
-	/* 8:20(sp) free */
+        /* 20(sp) used later to load fpscr with zero */
+        /* 8:16(sp) free */
 =09
         /* Linkage Area (reserved)
            4(sp)  : LR
@@ -190,7 +190,14 @@
         cmplwi  3,0
         beq     LafterFP2
=20
-        fsub    3,3,3   /* generate zero */
+       /* get zero into f3 (tedious) */
+       /* note: fsub 3,3,3 is not a reliable way to do this,=20
+          since if f3 holds a NaN or similar then we don't necessarily
+          wind up with zero. */
+        li      3,0
+        stw     3,20(1)
+        lfs     3,20(1)
+	/* load f3 to fpscr (0xFF =3D all bit fields) */
         mtfsf   0xFF,3
 LafterFP2:
=20