Unify handling of identical(NaN, NaN) between generated code and Dart API.

runtime/vm/object_test.cc

 // Copyright (c) 2012, the Dart project authors.  Please see the AUTHORS file
 // for details. All rights reserved. Use of this source code is governed by a
 // BSD-style license that can be found in the LICENSE file.
 
 // TODO(zra): Remove when tests are ready to enable.
 #include "platform/globals.h"
 
 #include "vm/assembler.h"
 #include "vm/bigint_operations.h"
 #include "vm/class_finalizer.h"
@@ skipping 506 matching lines @@
     const Double& dbl1 = Double::Handle(Double::New(dbl_const));
     const Double& dbl2 = Double::Handle(Double::New(dbl_const));
     EXPECT(dbl1.OperatorEquals(dbl2));
     EXPECT(dbl1.IsIdenticalTo(dbl2));
     EXPECT(dbl1.CanonicalizeEquals(dbl2));
     const Double& dbl3 = Double::Handle(Double::New(3.3));
     EXPECT(!dbl1.OperatorEquals(dbl3));
     EXPECT(!dbl1.OperatorEquals(Smi::Handle(Smi::New(3))));
     EXPECT(!dbl1.OperatorEquals(Double::Handle()));
     const Double& nan0 = Double::Handle(Double::New(NAN));
+    EXPECT(isnan(nan0.value()));
     EXPECT(nan0.IsIdenticalTo(nan0));
     EXPECT(nan0.CanonicalizeEquals(nan0));
     EXPECT(!nan0.OperatorEquals(nan0));
-    // TODO(18738): Test bitwise different NaNs after agreement on spec.
+    const Double& nan1 = Double::Handle(Double::New(0.0/0.0));
+    const Double& nan2 = Double::Handle(Double::New(1.0/0.0));

[koda, 2014/06/06 23:54:28]

Please rename the variable to something more appropriate, like 'inf'.

[regis, 2014/06/09 16:37:21]

Done.

+    EXPECT(isnan(nan1.value()));
+    EXPECT(!isnan(nan2.value()));
+    EXPECT(!nan1.IsIdenticalTo(nan2));
+    EXPECT(!nan1.CanonicalizeEquals(nan2));
+    EXPECT(!nan1.OperatorEquals(nan2));
+    const Double& nan3 = Double::Handle(
+        Double::New(bit_cast<double>(kMaxUint64 - 0)));
+    const Double& nan4 = Double::Handle(
+        Double::New(bit_cast<double>(kMaxUint64 - 1)));
+    EXPECT(isnan(nan3.value()));
+    EXPECT(isnan(nan4.value()));
+    EXPECT(!nan3.IsIdenticalTo(nan4));
+    EXPECT(!nan3.CanonicalizeEquals(nan4));
+    EXPECT(!nan3.OperatorEquals(nan4));
   }
   {
     const String& dbl_str0 = String::Handle(String::New("bla"));
     const Double& dbl0 = Double::Handle(Double::New(dbl_str0));
     EXPECT(dbl0.IsNull());
 
     const String& dbl_str1 = String::Handle(String::New("2.0"));
     const Double& dbl1 = Double::Handle(Double::New(dbl_str1));
     EXPECT_EQ(2.0, dbl1.value());
 
@@ skipping 3503 matching lines @@
   const Instance& a0 = Instance::Handle(Instance::New(clazz));
   const Instance& a1 = Instance::Handle(Instance::New(clazz));
   EXPECT(a0.raw() != a1.raw());
   EXPECT(a0.OperatorEquals(a0));
   EXPECT(a0.OperatorEquals(a1));
   EXPECT(a0.IsIdenticalTo(a0));
   EXPECT(!a0.IsIdenticalTo(a1));
 }
 
 }  // namespace dart