[FAST486]

The number of bit places in a shift operation is ANDed by the width of the type
when it is in a register, so (1 << c), when c is 32, is actually 1, and not 0.
Fix the calculation of MaxValue by using SignFlag | (SignFlag - 1) instead.

svn path=/branches/ntvdm/; revision=60912

diff --git a/lib/fast486/opgroups.c b/lib/fast486/opgroups.c
index b786d39..f29bf51 100644 (file)
--- a/lib/fast486/opgroups.c
+++ b/lib/fast486/opgroups.c
@@ -43,7 +43,7 @@ Fast486ArithmeticOperation(PFAST486_STATE State,
 {
     ULONG Result;
     ULONG SignFlag = 1 << (Bits - 1);
 {
     ULONG Result;
     ULONG SignFlag = 1 << (Bits - 1);

-    ULONG MaxValue = (1 << Bits) - 1;
+    ULONG MaxValue = (SignFlag - 1) | SignFlag;

 
     /* Make sure the values don't exceed the maximum for their size */
     FirstValue &= MaxValue;
 
     /* Make sure the values don't exceed the maximum for their size */
     FirstValue &= MaxValue;