PPC: Fix simulation of floating point <-> integer conversion.

src/ppc/simulator-ppc.cc

 // Copyright 2014 the V8 project authors. All rights reserved.
 // Use of this source code is governed by a BSD-style license that can be
 // found in the LICENSE file.
 
 #include <stdarg.h>
 #include <stdlib.h>
 #include <cmath>
 
 #if V8_TARGET_ARCH_PPC
 
@@ skipping 2684 matching lines @@
 }
 
 
 void Simulator::ExecuteExt3(Instruction* instr) {
   int opcode = instr->Bits(10, 1) << 1;
   switch (opcode) {
     case FCFID: {
       // fcfids
       int frt = instr->RTValue();
       int frb = instr->RBValue();
-      double t_val = get_double_from_d_register(frb);
-      int64_t* frb_val_p = reinterpret_cast<int64_t*>(&t_val);
-      double frt_val = static_cast<float>(*frb_val_p);
+      int64_t frb_val = get_d_register(frb);
+      double frt_val = static_cast<float>(frb_val);
       set_d_register_from_double(frt, frt_val);
       return;
     }
   }
   UNIMPLEMENTED();  // Not used by V8.
 }
 
 
 void Simulator::ExecuteExt4(Instruction* instr) {
   switch (instr->Bits(5, 1) << 1) {
@@ skipping 161 matching lines @@
       double frt_val = static_cast<float>(frb_val);
       set_d_register_from_double(frt, frt_val);
       if (instr->Bit(0)) {  // RC bit set
                             //  UNIMPLEMENTED();
       }
       return;
     }
     case FCFID: {
       int frt = instr->RTValue();
       int frb = instr->RBValue();
-      double t_val = get_double_from_d_register(frb);
-      int64_t* frb_val_p = reinterpret_cast<int64_t*>(&t_val);
-      double frt_val = static_cast<double>(*frb_val_p);
+      int64_t frb_val = get_d_register(frb);
+      double frt_val = static_cast<double>(frb_val);
       set_d_register_from_double(frt, frt_val);
       return;
     }
     case FCFIDU: {
       int frt = instr->RTValue();
       int frb = instr->RBValue();
-      double t_val = get_double_from_d_register(frb);
-      uint64_t* frb_val_p = reinterpret_cast<uint64_t*>(&t_val);
-      double frt_val = static_cast<double>(*frb_val_p);
+      uint64_t frb_val = get_d_register(frb);
+      double frt_val = static_cast<double>(frb_val);
       set_d_register_from_double(frt, frt_val);
       return;
     }
     case FCTID: {
       int frt = instr->RTValue();
       int frb = instr->RBValue();
       double frb_val = get_double_from_d_register(frb);
       int64_t frt_val;
       int64_t one = 1;  // work-around gcc
       int64_t kMinLongLong = (one << 63);
@@ skipping 13 matching lines @@
             break;
           case kRoundToMinusInf:
             frt_val = (int64_t)std::floor(frb_val);
             break;
           default:
             frt_val = (int64_t)frb_val;
             UNIMPLEMENTED();  // Not used by V8.
             break;
         }
       }
-      double* p = reinterpret_cast<double*>(&frt_val);
-      set_d_register_from_double(frt, *p);
+      set_d_register(frt, frt_val);
       return;
     }
     case FCTIDZ: {
       int frt = instr->RTValue();
       int frb = instr->RBValue();
       double frb_val = get_double_from_d_register(frb);
       int64_t frt_val;
       int64_t one = 1;  // work-around gcc
       int64_t kMinLongLong = (one << 63);
       int64_t kMaxLongLong = kMinLongLong - 1;
 
       if (frb_val > kMaxLongLong) {
         frt_val = kMaxLongLong;
       } else if (frb_val < kMinLongLong) {
         frt_val = kMinLongLong;
       } else {
         frt_val = (int64_t)frb_val;
       }
-      double* p = reinterpret_cast<double*>(&frt_val);
-      set_d_register_from_double(frt, *p);
+      set_d_register(frt, frt_val);
       return;
     }
     case FCTIW:
     case FCTIWZ: {
       int frt = instr->RTValue();
       int frb = instr->RBValue();
       double frb_val = get_double_from_d_register(frb);
       int64_t frt_val;
       if (frb_val > kMaxInt) {
         frt_val = kMaxInt;
@@ skipping 23 matching lines @@
               }
 
               break;
             default:
               DCHECK(false);
               frt_val = (int64_t)frb_val;
               break;
           }
         }
       }
-      double* p = reinterpret_cast<double*>(&frt_val);
-      set_d_register_from_double(frt, *p);
+      set_d_register(frt, frt_val);
       return;
     }
     case FNEG: {
       int frt = instr->RTValue();
       int frb = instr->RBValue();
       double frb_val = get_double_from_d_register(frb);
       double frt_val = -frb_val;
       set_d_register_from_double(frt, frt_val);
       return;
     }
@@ skipping 943 matching lines @@
   uintptr_t* stack_slot = reinterpret_cast<uintptr_t*>(current_sp);
   uintptr_t address = *stack_slot;
   set_register(sp, current_sp + sizeof(uintptr_t));
   return address;
 }
 }  // namespace internal
 }  // namespace v8
 
 #endif  // USE_SIMULATOR
 #endif  // V8_TARGET_ARCH_PPC