From: Aleksandar Andrejevic <aandrejevic@reactos.org>
Date: Thu, 14 Nov 2013 02:39:54 +0000 (+0000)
Subject: [FAST486]
X-Git-Tag: backups/0.3.17@66124~1365^2~235
X-Git-Url: https://git.reactos.org/?p=reactos.git;a=commitdiff_plain;h=cdf876ead954bb6095f30261aa5809a22580c71d

[FAST486]
Fix the BSF and BSR instructions. Partially fix SHLD and SHRD.


svn path=/branches/ntvdm/; revision=60987
---

diff --git a/lib/fast486/extraops.c b/lib/fast486/extraops.c
index 273c4fe5ba2..d7db59c8c5b 100644
--- a/lib/fast486/extraops.c
+++ b/lib/fast486/extraops.c
@@ -669,6 +669,7 @@ FAST486_OPCODE_HANDLER(Fast486ExtOpcodeShld)
     else
     {
         USHORT Source, Destination, Result;
+        ULONG DoubleSource;
 
         if (!Fast486ReadModrmWordOperands(State, &ModRegRm, &Source, &Destination))
         {
@@ -676,8 +677,10 @@ FAST486_OPCODE_HANDLER(Fast486ExtOpcodeShld)
             return FALSE;
         }
 
+        DoubleSource = Source | (Source << 16);
+
         /* Calculate the result */
-        Result = (Destination << Count) | (Source >> (16 - Count));
+        Result = (Destination << Count) | (DoubleSource >> (32 - Count));
 
         /* Update flags */
         State->Flags.Cf = (Destination >> (16 - Count)) & 1;
@@ -881,6 +884,8 @@ FAST486_OPCODE_HANDLER(Fast486ExtOpcodeShrd)
         /* Calculate the result */
         Result = (Destination >> Count) | (Source << (16 - Count));
 
+        if (Count >= 16) Result |= (ULONG)(Source | (Source << 16)) >> (Count - 16);
+
         /* Update flags */
         State->Flags.Cf = (Result >> (Count - 1)) & 1;
         if (Count == 1) State->Flags.Of = (Result & SIGN_FLAG_WORD)
@@ -1535,16 +1540,14 @@ FAST486_OPCODE_HANDLER(Fast486ExtOpcodeBsf)
         }
     }
 
-    /* Clear ZF */
-    State->Flags.Zf = FALSE;
+    /* Set ZF */
+    State->Flags.Zf = (Value == 0);
+    if (State->Flags.Zf) return TRUE;
 
     for (i = 0; i < DataSize; i++)
     {
         if(Value & (1 << i))
         {
-            /* Set ZF */
-            State->Flags.Zf = TRUE;
-
             /* Save the bit number */
             BitNumber = i;
 
@@ -1553,24 +1556,21 @@ FAST486_OPCODE_HANDLER(Fast486ExtOpcodeBsf)
         }
     }
 
-    if (State->Flags.Zf)
+    /* Write back the result */
+    if (OperandSize)
     {
-        /* Write back the result */
-        if (OperandSize)
+        if (!Fast486WriteModrmDwordOperands(State, &ModRegRm, TRUE, BitNumber))
         {
-            if (!Fast486WriteModrmDwordOperands(State, &ModRegRm, TRUE, BitNumber))
-            {
-                /* Exception occurred */
-                return FALSE;
-            }
+            /* Exception occurred */
+            return FALSE;
         }
-        else
+    }
+    else
+    {
+        if (!Fast486WriteModrmWordOperands(State, &ModRegRm, TRUE, LOWORD(BitNumber)))
         {
-            if (!Fast486WriteModrmWordOperands(State, &ModRegRm, TRUE, LOWORD(BitNumber)))
-            {
-                /* Exception occurred */
-                return FALSE;
-            }
+            /* Exception occurred */
+            return FALSE;
         }
     }
 
@@ -1625,16 +1625,14 @@ FAST486_OPCODE_HANDLER(Fast486ExtOpcodeBsr)
         }
     }
 
-    /* Clear ZF */
-    State->Flags.Zf = FALSE;
+    /* Set ZF according to the value */
+    State->Flags.Zf = (Value == 0);
+    if (State->Flags.Zf) return TRUE;
 
     for (i = DataSize - 1; i >= 0; i--)
     {
         if(Value & (1 << i))
         {
-            /* Set ZF */
-            State->Flags.Zf = TRUE;
-
             /* Save the bit number */
             BitNumber = i;
 
@@ -1643,24 +1641,21 @@ FAST486_OPCODE_HANDLER(Fast486ExtOpcodeBsr)
         }
     }
 
-    if (State->Flags.Zf)
+    /* Write back the result */
+    if (OperandSize)
     {
-        /* Write back the result */
-        if (OperandSize)
+        if (!Fast486WriteModrmDwordOperands(State, &ModRegRm, TRUE, BitNumber))
         {
-            if (!Fast486WriteModrmDwordOperands(State, &ModRegRm, TRUE, BitNumber))
-            {
-                /* Exception occurred */
-                return FALSE;
-            }
+            /* Exception occurred */
+            return FALSE;
         }
-        else
+    }
+    else
+    {
+        if (!Fast486WriteModrmWordOperands(State, &ModRegRm, TRUE, LOWORD(BitNumber)))
         {
-            if (!Fast486WriteModrmWordOperands(State, &ModRegRm, TRUE, LOWORD(BitNumber)))
-            {
-                /* Exception occurred */
-                return FALSE;
-            }
+            /* Exception occurred */
+            return FALSE;
         }
     }