X87: [wasm] Use the Float64Max/Min machine operators to implement F64Max/Min.

src/crankshaft/x87/lithium-codegen-x87.cc

Index: src/crankshaft/x87/lithium-codegen-x87.cc
diff --git a/src/crankshaft/x87/lithium-codegen-x87.cc b/src/crankshaft/x87/lithium-codegen-x87.cc
index e7e99c5e739abc21c2417ea82f6bf9ce393fb390..a754856587990d7269000b37fa0ec236b91c0db9 100644
--- a/src/crankshaft/x87/lithium-codegen-x87.cc
+++ b/src/crankshaft/x87/lithium-codegen-x87.cc
@@ -4735,8 +4735,8 @@ void LCodeGen::EmitNumberUntagDNoSSE2(LNumberUntagD* instr, Register input_reg,
                    DeoptimizeReason::kNotAHeapNumberUndefined);
 
       __ bind(&convert);
-      __ push(Immediate(0xffffffff));
-      __ push(Immediate(0x7fffffff));
+      __ push(Immediate(0xfff80000));

[ahaas, 2016/08/11 08:06:11]

In the old code the low word "0xffffffff" was push first and the high word
"0x7fffffff" was pushed second. It seems to me that in the new code the low word
"0x00000000" is pushed second and the high word "0xfff80000" is pushed first.
Are you sure that you push the words in the right order here?

+      __ push(Immediate(0x00000000));
       __ fld_d(MemOperand(esp, 0));
       __ lea(esp, Operand(esp, kDoubleSize));
       __ jmp(&done, Label::kNear);