Revert "Remove exceptions parameter from normaliseround VFP functions"

This reverts commit edf30d84cc.

src/core/arm/skyeye_common/vfp/vfp_helper.h

@@ -304,7 +304,7 @@ inline s32 vfp_single_pack(const vfp_single* s)
 }
 
 
-u32 vfp_single_normaliseround(ARMul_State* state, int sd, vfp_single* vs, u32 fpscr, const char* func);
+u32 vfp_single_normaliseround(ARMul_State* state, int sd, vfp_single* vs, u32 fpscr, u32 exceptions, const char* func);
 
 // Double-precision
 struct vfp_double {
@@ -451,4 +451,4 @@ inline u32 fls(u32 x)
 
 u32 vfp_double_multiply(vfp_double* vdd, vfp_double* vdn, vfp_double* vdm, u32 fpscr);
 u32 vfp_double_add(vfp_double* vdd, vfp_double* vdn, vfp_double *vdm, u32 fpscr);
-u32 vfp_double_normaliseround(ARMul_State* state, int dd, vfp_double* vd, u32 fpscr, const char* func);
+u32 vfp_double_normaliseround(ARMul_State* state, int dd, vfp_double* vd, u32 fpscr, u32 exceptions, const char* func);

src/core/arm/skyeye_common/vfp/vfpdouble.cpp

@@ -85,12 +85,11 @@ static void vfp_double_normalise_denormal(struct vfp_double *vd)
     vfp_double_dump("normalise_denormal: out", vd);
 }
 
-u32 vfp_double_normaliseround(ARMul_State* state, int dd, struct vfp_double *vd, u32 fpscr, const char *func)
+u32 vfp_double_normaliseround(ARMul_State* state, int dd, struct vfp_double *vd, u32 fpscr, u32 exceptions, const char *func)
 {
     u64 significand, incr;
     int exponent, shift, underflow;
     u32 rmode;
-    u32 exceptions = 0;
 
     vfp_double_dump("pack: in", vd);
 
@@ -371,7 +370,8 @@ sqrt_invalid:
     }
     vdd.significand = vfp_shiftright64jamming(vdd.significand, 1);
 
-    exceptions |= vfp_double_normaliseround(state, dd, &vdd, fpscr, "fsqrt");
+    exceptions |= vfp_double_normaliseround(state, dd, &vdd, fpscr, 0, "fsqrt");
+
     return exceptions;
 }
 
@@ -507,8 +507,7 @@ static u32 vfp_double_fcvts(ARMul_State* state, int sd, int unused, int dm, u32
     else
         vsd.exponent = vdm.exponent - (1023 - 127);
 
-    exceptions |= vfp_single_normaliseround(state, sd, &vsd, fpscr, "fcvts");
-    return exceptions;
+    return vfp_single_normaliseround(state, sd, &vsd, fpscr, exceptions, "fcvts");
 
 pack_nan:
     vfp_put_float(state, vfp_single_pack(&vsd), sd);
@@ -518,7 +517,6 @@ pack_nan:
 static u32 vfp_double_fuito(ARMul_State* state, int dd, int unused, int dm, u32 fpscr)
 {
     struct vfp_double vdm;
-    u32 exceptions = 0;
     u32 m = vfp_get_float(state, dm);
 
     LOG_TRACE(Core_ARM11, "In %s", __FUNCTION__);
@@ -526,14 +524,12 @@ static u32 vfp_double_fuito(ARMul_State* state, int dd, int unused, int dm, u32
     vdm.exponent = 1023 + 63 - 1;
     vdm.significand = (u64)m;
 
-    exceptions |= vfp_double_normaliseround(state, dd, &vdm, fpscr, "fuito");
-    return exceptions;
+    return vfp_double_normaliseround(state, dd, &vdm, fpscr, 0, "fuito");
 }
 
 static u32 vfp_double_fsito(ARMul_State* state, int dd, int unused, int dm, u32 fpscr)
 {
     struct vfp_double vdm;
-    u32 exceptions = 0;
     u32 m = vfp_get_float(state, dm);
 
     LOG_TRACE(Core_ARM11, "In %s", __FUNCTION__);
@@ -541,8 +537,7 @@ static u32 vfp_double_fsito(ARMul_State* state, int dd, int unused, int dm, u32
     vdm.exponent = 1023 + 63 - 1;
     vdm.significand = vdm.sign ? (~m + 1) : m;
 
-    exceptions |= vfp_double_normaliseround(state, dd, &vdm, fpscr, "fsito");
-    return exceptions;
+    return vfp_double_normaliseround(state, dd, &vdm, fpscr, 0, "fsito");
 }
 
 static u32 vfp_double_ftoui(ARMul_State* state, int sd, int unused, int dm, u32 fpscr)
@@ -938,8 +933,7 @@ vfp_double_multiply_accumulate(ARMul_State* state, int dd, int dn, int dm, u32 f
 
     exceptions |= vfp_double_add(&vdd, &vdn, &vdp, fpscr);
 
-    exceptions |= vfp_double_normaliseround(state, dd, &vdd, fpscr, func);
-    return exceptions;
+    return vfp_double_normaliseround(state, dd, &vdd, fpscr, exceptions, func);
 }
 
 /*
@@ -1000,9 +994,7 @@ static u32 vfp_double_fmul(ARMul_State* state, int dd, int dn, int dm, u32 fpscr
         vfp_double_normalise_denormal(&vdm);
 
     exceptions |= vfp_double_multiply(&vdd, &vdn, &vdm, fpscr);
-
-    exceptions |= vfp_double_normaliseround(state, dd, &vdd, fpscr, "fmul");
-    return exceptions;
+    return vfp_double_normaliseround(state, dd, &vdd, fpscr, exceptions, "fmul");
 }
 
 /*
@@ -1025,8 +1017,7 @@ static u32 vfp_double_fnmul(ARMul_State* state, int dd, int dn, int dm, u32 fpsc
     exceptions |= vfp_double_multiply(&vdd, &vdn, &vdm, fpscr);
     vdd.sign = vfp_sign_negate(vdd.sign);
 
-    exceptions |= vfp_double_normaliseround(state, dd, &vdd, fpscr, "fnmul");
-    return exceptions;
+    return vfp_double_normaliseround(state, dd, &vdd, fpscr, exceptions, "fnmul");
 }
 
 /*
@@ -1048,8 +1039,7 @@ static u32 vfp_double_fadd(ARMul_State* state, int dd, int dn, int dm, u32 fpscr
 
     exceptions |= vfp_double_add(&vdd, &vdn, &vdm, fpscr);
 
-    exceptions |= vfp_double_normaliseround(state, dd, &vdd, fpscr, "fadd");
-    return exceptions;
+    return vfp_double_normaliseround(state, dd, &vdd, fpscr, exceptions, "fadd");
 }
 
 /*
@@ -1076,8 +1066,7 @@ static u32 vfp_double_fsub(ARMul_State* state, int dd, int dn, int dm, u32 fpscr
 
     exceptions |= vfp_double_add(&vdd, &vdn, &vdm, fpscr);
 
-    exceptions |= vfp_double_normaliseround(state, dd, &vdd, fpscr, "fsub");
-    return exceptions;
+    return vfp_double_normaliseround(state, dd, &vdd, fpscr, exceptions, "fsub");
 }
 
 /*
@@ -1160,9 +1149,7 @@ static u32 vfp_double_fdiv(ARMul_State* state, int dd, int dn, int dm, u32 fpscr
         }
         vdd.significand |= (reml != 0);
     }
-
-    exceptions |= vfp_double_normaliseround(state, dd, &vdd, fpscr, "fdiv");
-    return exceptions;
+    return vfp_double_normaliseround(state, dd, &vdd, fpscr, 0, "fdiv");
 
 vdn_nan:
     exceptions |= vfp_propagate_nan(&vdd, &vdn, &vdm, fpscr);
@@ -1188,8 +1175,7 @@ infinity:
 
 invalid:
     vfp_put_double(state, vfp_double_pack(&vfp_double_default_qnan), dd);
-    exceptions |= FPSCR_IOC;
-    return exceptions;
+    return FPSCR_IOC;
 }
 
 static struct op fops[] = {

src/core/arm/skyeye_common/vfp/vfpsingle.cpp

@@ -89,11 +89,10 @@ static void vfp_single_normalise_denormal(struct vfp_single *vs)
 }
 
 
-u32 vfp_single_normaliseround(ARMul_State* state, int sd, struct vfp_single *vs, u32 fpscr, const char *func)
+u32 vfp_single_normaliseround(ARMul_State* state, int sd, struct vfp_single *vs, u32 fpscr, u32 exceptions, const char *func)
 {
     u32 significand, incr, rmode;
     int exponent, shift, underflow;
-    u32 exceptions = 0;
 
     vfp_single_dump("pack: in", vs);
 
@@ -410,7 +409,8 @@ sqrt_invalid:
     }
     vsd.significand = vfp_shiftright32jamming(vsd.significand, 1);
 
-    exceptions |= vfp_single_normaliseround(state, sd, &vsd, fpscr, "fsqrt");
+    exceptions |= vfp_single_normaliseround(state, sd, &vsd, fpscr, 0, "fsqrt");
+
     return exceptions;
 }
 
@@ -535,8 +535,7 @@ static u32 vfp_single_fcvtd(ARMul_State* state, int dd, int unused, s32 m, u32 f
     else
         vdd.exponent = vsm.exponent + (1023 - 127);
 
-    exceptions |= vfp_double_normaliseround(state, dd, &vdd, fpscr, "fcvtd");
-    return exceptions;
+    return vfp_double_normaliseround(state, dd, &vdd, fpscr, exceptions, "fcvtd");
 
 pack_nan:
     vfp_put_double(state, vfp_double_pack(&vdd), dd);
@@ -546,27 +545,23 @@ pack_nan:
 static u32 vfp_single_fuito(ARMul_State* state, int sd, int unused, s32 m, u32 fpscr)
 {
     struct vfp_single vs;
-    u32 exceptions = 0;
 
     vs.sign = 0;
     vs.exponent = 127 + 31 - 1;
     vs.significand = (u32)m;
 
-    exceptions |= vfp_single_normaliseround(state, sd, &vs, fpscr, "fuito");
-    return exceptions;
+    return vfp_single_normaliseround(state, sd, &vs, fpscr, 0, "fuito");
 }
 
 static u32 vfp_single_fsito(ARMul_State* state, int sd, int unused, s32 m, u32 fpscr)
 {
     struct vfp_single vs;
-    u32 exceptions = 0;
 
     vs.sign = (m & 0x80000000) >> 16;
     vs.exponent = 127 + 31 - 1;
     vs.significand = vs.sign ? -m : m;
 
-    exceptions |= vfp_single_normaliseround(state, sd, &vs, fpscr, "fsito");
-    return exceptions;
+    return vfp_single_normaliseround(state, sd, &vs, fpscr, 0, "fsito");
 }
 
 static u32 vfp_single_ftoui(ARMul_State* state, int sd, int unused, s32 m, u32 fpscr)
@@ -974,8 +969,7 @@ vfp_single_multiply_accumulate(ARMul_State* state, int sd, int sn, s32 m, u32 fp
 
     exceptions |= vfp_single_add(&vsd, &vsn, &vsp, fpscr);
 
-    exceptions |= vfp_single_normaliseround(state, sd, &vsd, fpscr, func);
-    return exceptions;
+    return vfp_single_normaliseround(state, sd, &vsd, fpscr, exceptions, func);
 }
 
 /*
@@ -987,10 +981,8 @@ vfp_single_multiply_accumulate(ARMul_State* state, int sd, int sn, s32 m, u32 fp
  */
 static u32 vfp_single_fmac(ARMul_State* state, int sd, int sn, s32 m, u32 fpscr)
 {
-    u32 exceptions = 0;
     LOG_TRACE(Core_ARM11, "s%u = %08x", sn, sd);
-    exceptions |= vfp_single_multiply_accumulate(state, sd, sn, m, fpscr, 0, "fmac");
-    return exceptions;
+    return vfp_single_multiply_accumulate(state, sd, sn, m, fpscr, 0, "fmac");
 }
 
 /*
@@ -1041,9 +1033,7 @@ static u32 vfp_single_fmul(ARMul_State* state, int sd, int sn, s32 m, u32 fpscr)
         vfp_single_normalise_denormal(&vsm);
 
     exceptions |= vfp_single_multiply(&vsd, &vsn, &vsm, fpscr);
-
-    exceptions |= vfp_single_normaliseround(state, sd, &vsd, fpscr, "fmul");
-    return exceptions;
+    return vfp_single_normaliseround(state, sd, &vsd, fpscr, exceptions, "fmul");
 }
 
 /*
@@ -1067,9 +1057,7 @@ static u32 vfp_single_fnmul(ARMul_State* state, int sd, int sn, s32 m, u32 fpscr
 
     exceptions |= vfp_single_multiply(&vsd, &vsn, &vsm, fpscr);
     vsd.sign = vfp_sign_negate(vsd.sign);
-
-    exceptions |= vfp_single_normaliseround(state, sd, &vsd, fpscr, "fnmul");
-    return exceptions;
+    return vfp_single_normaliseround(state, sd, &vsd, fpscr, exceptions, "fnmul");
 }
 
 /*
@@ -1096,8 +1084,7 @@ static u32 vfp_single_fadd(ARMul_State* state, int sd, int sn, s32 m, u32 fpscr)
 
     exceptions |= vfp_single_add(&vsd, &vsn, &vsm, fpscr);
 
-    exceptions |= vfp_single_normaliseround(state, sd, &vsd, fpscr, "fadd");
-    return exceptions;
+    return vfp_single_normaliseround(state, sd, &vsd, fpscr, exceptions, "fadd");
 }
 
 /*
@@ -1194,8 +1181,7 @@ static u32 vfp_single_fdiv(ARMul_State* state, int sd, int sn, s32 m, u32 fpscr)
     if ((vsd.significand & 0x3f) == 0)
         vsd.significand |= ((u64)vsm.significand * vsd.significand != (u64)vsn.significand << 32);
 
-    exceptions |= vfp_single_normaliseround(state, sd, &vsd, fpscr, "fdiv");
-    return exceptions;
+    return vfp_single_normaliseround(state, sd, &vsd, fpscr, 0, "fdiv");
 
 vsn_nan:
     exceptions |= vfp_propagate_nan(&vsd, &vsn, &vsm, fpscr);
@@ -1221,8 +1207,7 @@ infinity:
 
 invalid:
     vfp_put_float(state, vfp_single_pack(&vfp_single_default_qnan), sd);
-    exceptions |= FPSCR_IOC;
-    return exceptions;
+    return FPSCR_IOC;
 }
 
 static struct op fops[] = {