From ab25eb6449654c14a1c5778833e34f76019c16ae Mon Sep 17 00:00:00 2001
From: Lioncash <mathew1800@gmail.com>
Date: Wed, 11 Apr 2018 19:01:17 -0400
Subject: [PATCH] Interpreter_LoadStorePaired: Don't use a union to type-pun
 between integral and FP types

---
 .../Interpreter_LoadStorePaired.cpp           | 30 ++++++++++++-------
 1 file changed, 19 insertions(+), 11 deletions(-)

diff --git a/Source/Core/Core/PowerPC/Interpreter/Interpreter_LoadStorePaired.cpp b/Source/Core/Core/PowerPC/Interpreter/Interpreter_LoadStorePaired.cpp
index 2661804c71..5c9fbce27b 100644
--- a/Source/Core/Core/PowerPC/Interpreter/Interpreter_LoadStorePaired.cpp
+++ b/Source/Core/Core/PowerPC/Interpreter/Interpreter_LoadStorePaired.cpp
@@ -2,6 +2,7 @@
 // Licensed under GPLv2+
 // Refer to the license.txt file included.
 
+#include <cstring>
 #include <tuple>
 #include <type_traits>
 #include <utility>
@@ -172,21 +173,28 @@ void Interpreter::Helper_Quantize(u32 addr, u32 instI, u32 instRS, u32 instW)
   const EQuantizeType stType = gqr.st_type;
   const unsigned int stScale = gqr.st_scale;
 
-  double ps0 = rPS0(instRS);
-  double ps1 = rPS1(instRS);
+  const double ps0 = rPS0(instRS);
+  const double ps1 = rPS1(instRS);
+
   switch (stType)
   {
   case QUANTIZE_FLOAT:
   {
-    u32 convPS0 = ConvertToSingleFTZ(MathUtil::IntDouble(ps0).i);
+    u64 integral_ps0;
+    std::memcpy(&integral_ps0, &ps0, sizeof(u64));
+
+    const u32 conv_ps0 = ConvertToSingleFTZ(integral_ps0);
     if (instW)
     {
-      WriteUnpaired<u32>(convPS0, addr);
+      WriteUnpaired<u32>(conv_ps0, addr);
     }
     else
     {
-      u32 convPS1 = ConvertToSingleFTZ(MathUtil::IntDouble(ps1).i);
-      WritePair<u32>(convPS0, convPS1, addr);
+      u64 integral_ps1;
+      std::memcpy(&integral_ps1, &ps1, sizeof(double));
+
+      const u32 conv_ps1 = ConvertToSingleFTZ(integral_ps1);
+      WritePair<u32>(conv_ps0, conv_ps1, addr);
     }
     break;
   }
@@ -249,15 +257,15 @@ void Interpreter::Helper_Dequantize(u32 addr, u32 instI, u32 instRD, u32 instW)
   case QUANTIZE_FLOAT:
     if (instW)
     {
-      u32 value = ReadUnpaired<u32>(addr);
-      ps0 = MathUtil::IntFloat(value).f;
+      const u32 value = ReadUnpaired<u32>(addr);
+      std::memcpy(&ps0, &value, sizeof(float));
       ps1 = 1.0f;
     }
     else
     {
-      std::pair<u32, u32> value = ReadPair<u32>(addr);
-      ps0 = MathUtil::IntFloat(value.first).f;
-      ps1 = MathUtil::IntFloat(value.second).f;
+      const std::pair<u32, u32> value = ReadPair<u32>(addr);
+      std::memcpy(&ps0, &value.first, sizeof(float));
+      std::memcpy(&ps1, &value.second, sizeof(float));
     }
     break;