diff --git a/Source/Core/Core/PowerPC/Gekko.h b/Source/Core/Core/PowerPC/Gekko.h index a046b883c2..ca083b1219 100644 --- a/Source/Core/Core/PowerPC/Gekko.h +++ b/Source/Core/Core/PowerPC/Gekko.h @@ -476,6 +476,12 @@ union UReg_FPSCR UReg_FPSCR() = default; explicit UReg_FPSCR(u32 hex_) : Hex{hex_} {} + + void ClearFIFR() + { + FI = 0; + FR = 0; + } }; // Hardware Implementation-Dependent Register 0 diff --git a/Source/Core/Core/PowerPC/Interpreter/Interpreter_FPUtils.h b/Source/Core/Core/PowerPC/Interpreter/Interpreter_FPUtils.h index 666ca2bffb..ed15cef6da 100644 --- a/Source/Core/Core/PowerPC/Interpreter/Interpreter_FPUtils.h +++ b/Source/Core/Core/PowerPC/Interpreter/Interpreter_FPUtils.h @@ -105,8 +105,7 @@ inline double NI_mul(double a, double b) if (Common::IsSNAN(a) || Common::IsSNAN(b)) { SetFPException(FPSCR_VXSNAN); - FPSCR.FI = 0; - FPSCR.FR = 0; + FPSCR.ClearFIFR(); } if (std::isnan(a)) @@ -115,8 +114,7 @@ inline double NI_mul(double a, double b) return MakeQuiet(b); SetFPException(FPSCR_VXIMZ); - FPSCR.FI = 0; - FPSCR.FR = 0; + FPSCR.ClearFIFR(); return PPC_NAN; } return t; @@ -131,8 +129,7 @@ inline double NI_div(double a, double b) if (Common::IsSNAN(a) || Common::IsSNAN(b)) { SetFPException(FPSCR_VXSNAN); - FPSCR.FI = 0; - FPSCR.FR = 0; + FPSCR.ClearFIFR(); } if (std::isnan(a)) @@ -145,21 +142,18 @@ inline double NI_div(double a, double b) if (a == 0.0) { SetFPException(FPSCR_VXZDZ); - FPSCR.FI = 0; - FPSCR.FR = 0; + FPSCR.ClearFIFR(); } else { SetFPException(FPSCR_ZX); - FPSCR.FI = 0; - FPSCR.FR = 0; + FPSCR.ClearFIFR(); } } else if (std::isinf(a) && std::isinf(b)) { SetFPException(FPSCR_VXIDI); - FPSCR.FI = 0; - FPSCR.FR = 0; + FPSCR.ClearFIFR(); } return PPC_NAN; @@ -177,8 +171,7 @@ inline double NI_add(double a, double b) if (Common::IsSNAN(a) || Common::IsSNAN(b)) { SetFPException(FPSCR_VXSNAN); - FPSCR.FI = 0; - FPSCR.FR = 0; + FPSCR.ClearFIFR(); } if (std::isnan(a)) @@ -187,8 +180,7 @@ inline double NI_add(double a, double b) return MakeQuiet(b); SetFPException(FPSCR_VXISI); - FPSCR.FI = 0; - FPSCR.FR = 0; + FPSCR.ClearFIFR(); return PPC_NAN; } @@ -204,8 +196,7 @@ inline double NI_sub(double a, double b) if (Common::IsSNAN(a) || Common::IsSNAN(b)) { SetFPException(FPSCR_VXSNAN); - FPSCR.FI = 0; - FPSCR.FR = 0; + FPSCR.ClearFIFR(); } if (std::isnan(a)) @@ -214,8 +205,7 @@ inline double NI_sub(double a, double b) return MakeQuiet(b); SetFPException(FPSCR_VXISI); - FPSCR.FI = 0; - FPSCR.FR = 0; + FPSCR.ClearFIFR(); return PPC_NAN; } @@ -234,8 +224,7 @@ inline double NI_madd(double a, double c, double b) if (Common::IsSNAN(a) || Common::IsSNAN(b) || Common::IsSNAN(c)) { SetFPException(FPSCR_VXSNAN); - FPSCR.FI = 0; - FPSCR.FR = 0; + FPSCR.ClearFIFR(); } if (std::isnan(a)) @@ -246,8 +235,7 @@ inline double NI_madd(double a, double c, double b) return MakeQuiet(c); SetFPException(FPSCR_VXIMZ); - FPSCR.FI = 0; - FPSCR.FR = 0; + FPSCR.ClearFIFR(); return PPC_NAN; } @@ -258,16 +246,14 @@ inline double NI_madd(double a, double c, double b) if (Common::IsSNAN(b)) { SetFPException(FPSCR_VXSNAN); - FPSCR.FI = 0; - FPSCR.FR = 0; + FPSCR.ClearFIFR(); } if (std::isnan(b)) return MakeQuiet(b); SetFPException(FPSCR_VXISI); - FPSCR.FI = 0; - FPSCR.FR = 0; + FPSCR.ClearFIFR(); return PPC_NAN; } @@ -283,8 +269,7 @@ inline double NI_msub(double a, double c, double b) if (Common::IsSNAN(a) || Common::IsSNAN(b) || Common::IsSNAN(c)) { SetFPException(FPSCR_VXSNAN); - FPSCR.FI = 0; - FPSCR.FR = 0; + FPSCR.ClearFIFR(); } if (std::isnan(a)) @@ -295,8 +280,7 @@ inline double NI_msub(double a, double c, double b) return MakeQuiet(c); SetFPException(FPSCR_VXIMZ); - FPSCR.FI = 0; - FPSCR.FR = 0; + FPSCR.ClearFIFR(); return PPC_NAN; } @@ -307,16 +291,14 @@ inline double NI_msub(double a, double c, double b) if (Common::IsSNAN(b)) { SetFPException(FPSCR_VXSNAN); - FPSCR.FI = 0; - FPSCR.FR = 0; + FPSCR.ClearFIFR(); } if (std::isnan(b)) return MakeQuiet(b); SetFPException(FPSCR_VXISI); - FPSCR.FI = 0; - FPSCR.FR = 0; + FPSCR.ClearFIFR(); return PPC_NAN; } diff --git a/Source/Core/Core/PowerPC/Interpreter/Interpreter_FloatingPoint.cpp b/Source/Core/Core/PowerPC/Interpreter/Interpreter_FloatingPoint.cpp index acf7c93afb..32f78343e9 100644 --- a/Source/Core/Core/PowerPC/Interpreter/Interpreter_FloatingPoint.cpp +++ b/Source/Core/Core/PowerPC/Interpreter/Interpreter_FloatingPoint.cpp @@ -92,8 +92,7 @@ void ConvertToInteger(UGeckoInstruction inst, RoundingMode rounding_mode) const double di = i; if (di == b) { - FPSCR.FI = 0; - FPSCR.FR = 0; + FPSCR.ClearFIFR(); } else { @@ -105,8 +104,7 @@ void ConvertToInteger(UGeckoInstruction inst, RoundingMode rounding_mode) if (exception_occurred) { - FPSCR.FI = 0; - FPSCR.FR = 0; + FPSCR.ClearFIFR(); } if (!exception_occurred || FPSCR.VE == 0) @@ -285,8 +283,7 @@ void Interpreter::frspx(UGeckoInstruction inst) // round to single PowerPC::UpdateFPRF(b); } - SetFI(0); - FPSCR.FR = 0; + FPSCR.ClearFIFR(); } else { @@ -397,8 +394,7 @@ void Interpreter::fresx(UGeckoInstruction inst) if (b == 0.0) { SetFPException(FPSCR_ZX); - FPSCR.FI = 0; - FPSCR.FR = 0; + FPSCR.ClearFIFR(); if (FPSCR.ZE == 0) compute_result(b); @@ -406,8 +402,7 @@ void Interpreter::fresx(UGeckoInstruction inst) else if (Common::IsSNAN(b)) { SetFPException(FPSCR_VXSNAN); - FPSCR.FI = 0; - FPSCR.FR = 0; + FPSCR.ClearFIFR(); if (FPSCR.VE == 0) compute_result(b); @@ -434,8 +429,7 @@ void Interpreter::frsqrtex(UGeckoInstruction inst) if (b < 0.0) { SetFPException(FPSCR_VXSQRT); - FPSCR.FI = 0; - FPSCR.FR = 0; + FPSCR.ClearFIFR(); if (FPSCR.VE == 0) compute_result(b); @@ -443,8 +437,7 @@ void Interpreter::frsqrtex(UGeckoInstruction inst) else if (b == 0.0) { SetFPException(FPSCR_ZX); - FPSCR.FI = 0; - FPSCR.FR = 0; + FPSCR.ClearFIFR(); if (FPSCR.ZE == 0) compute_result(b); @@ -452,8 +445,7 @@ void Interpreter::frsqrtex(UGeckoInstruction inst) else if (Common::IsSNAN(b)) { SetFPException(FPSCR_VXSNAN); - FPSCR.FI = 0; - FPSCR.FR = 0; + FPSCR.ClearFIFR(); if (FPSCR.VE == 0) compute_result(b); diff --git a/Source/Core/Core/PowerPC/Interpreter/Interpreter_Paired.cpp b/Source/Core/Core/PowerPC/Interpreter/Interpreter_Paired.cpp index 44145195e5..380c9ac103 100644 --- a/Source/Core/Core/PowerPC/Interpreter/Interpreter_Paired.cpp +++ b/Source/Core/Core/PowerPC/Interpreter/Interpreter_Paired.cpp @@ -121,15 +121,13 @@ void Interpreter::ps_res(UGeckoInstruction inst) if (a == 0.0 || b == 0.0) { SetFPException(FPSCR_ZX); - FPSCR.FI = 0; - FPSCR.FR = 0; + FPSCR.ClearFIFR(); } if (Common::IsSNAN(a) || Common::IsSNAN(b)) { SetFPException(FPSCR_VXSNAN); - FPSCR.FI = 0; - FPSCR.FR = 0; + FPSCR.ClearFIFR(); } rPS0(inst.FD) = Common::ApproximateReciprocal(a); @@ -148,22 +146,19 @@ void Interpreter::ps_rsqrte(UGeckoInstruction inst) if (ps0 == 0.0 || ps1 == 0.0) { SetFPException(FPSCR_ZX); - FPSCR.FI = 0; - FPSCR.FR = 0; + FPSCR.ClearFIFR(); } if (ps0 < 0.0 || ps1 < 0.0) { SetFPException(FPSCR_VXSQRT); - FPSCR.FI = 0; - FPSCR.FR = 0; + FPSCR.ClearFIFR(); } if (Common::IsSNAN(ps0) || Common::IsSNAN(ps1)) { SetFPException(FPSCR_VXSNAN); - FPSCR.FI = 0; - FPSCR.FR = 0; + FPSCR.ClearFIFR(); } rPS0(inst.FD) = ForceSingle(Common::ApproximateReciprocalSquareRoot(ps0));