[wasm] Fix I32ReinterpretF32 and I64ReinterpretF64 on ia32.

src/compiler/ia32/code-generator-ia32.cc

Index: src/compiler/ia32/code-generator-ia32.cc
diff --git a/src/compiler/ia32/code-generator-ia32.cc b/src/compiler/ia32/code-generator-ia32.cc
index 4e159efd239acad9ed501aa7eea0ccbe43df0f90..0d49efba757566d385fcb7304de34f1e4fbd4906 100644
--- a/src/compiler/ia32/code-generator-ia32.cc
+++ b/src/compiler/ia32/code-generator-ia32.cc
@@ -2106,7 +2106,7 @@ void CodeGenerator::AssembleMove(InstructionOperand* source,
       __ Move(dst, g.ToImmediate(source));
     } else if (src_constant.type() == Constant::kFloat32) {
       // TODO(turbofan): Can we do better here?
-      uint32_t src = bit_cast<uint32_t>(src_constant.ToFloat32());
+      uint32_t src = bit_cast<uint32_t>(src_constant.ToFloat32AsInt());

[titzer, 2017/01/19 13:13:01]

I think you can remove the bit_cast now that this method returns an
int32_t...but you probably want it to return a uint32_t.

[ahaas, 2017/01/19 15:46:19]

Done.

       if (destination->IsFPRegister()) {
         XMMRegister dst = g.ToDoubleRegister(destination);
         __ Move(dst, src);
@@ -2117,7 +2117,7 @@ void CodeGenerator::AssembleMove(InstructionOperand* source,
       }
     } else {
       DCHECK_EQ(Constant::kFloat64, src_constant.type());
-      uint64_t src = bit_cast<uint64_t>(src_constant.ToFloat64());
+      uint64_t src = bit_cast<uint64_t>(src_constant.ToFloat64AsInt());

[titzer, 2017/01/19 13:13:01]

Same here.

[ahaas, 2017/01/19 15:46:19]

Done.

       uint32_t lower = static_cast<uint32_t>(src);
       uint32_t upper = static_cast<uint32_t>(src >> 32);
       if (destination->IsFPRegister()) {