Minor changes based on the profile while running dart2js compile all.

vm/object.h

 // 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.
 
 #ifndef VM_OBJECT_H_
 #define VM_OBJECT_H_
 
 #include "include/dart_api.h"
 #include "platform/assert.h"
 #include "platform/utils.h"
@@ skipping 3170 matching lines @@
   static intptr_t hash_offset() { return OFFSET_OF(RawString, hash_); }
   static intptr_t Hash(const String& str, intptr_t begin_index, intptr_t len);
   static intptr_t Hash(const uint8_t* characters, intptr_t len);
   static intptr_t Hash(const uint16_t* characters, intptr_t len);
   static intptr_t Hash(const uint32_t* characters, intptr_t len);
 
   virtual int32_t CharAt(intptr_t index) const;
 
   virtual intptr_t CharSize() const;
 
-  bool Equals(const String& str) const {
-    if (raw() == str.raw()) {
-      return true;  // Both handles point to the same raw instance.
-    }
-    if (str.IsNull()) {
-      return false;
-    }
-    return Equals(str, 0, str.Length());
-  }
-  bool Equals(const String& str, intptr_t begin_index, intptr_t len) const;
+  inline bool Equals(const String& str) const;
+  inline bool Equals(const String& str,
+                     intptr_t begin_index,
+                     intptr_t len) const;
   bool Equals(const char* str) const;
   bool Equals(const uint8_t* characters, intptr_t len) const;
   bool Equals(const uint16_t* characters, intptr_t len) const;
   bool Equals(const uint32_t* characters, intptr_t len) const;
 
   virtual bool Equals(const Instance& other) const;
 
   intptr_t CompareTo(const String& other) const;
 
   bool StartsWith(const String& other) const;
@@ skipping 1920 matching lines @@
 
 void Context::SetAt(intptr_t index, const Instance& value) const {
   StorePointer(InstanceAddr(index), value.raw());
 }
 
 
 intptr_t Stackmap::SizeInBits() const {
   return (Smi::Value(raw_ptr()->bitmap_size_in_bytes_) * kBitsPerByte);
 }
 
+
+bool String::Equals(const String& str) const {
+  if (raw() == str.raw()) {
+    return true;  // Both handles point to the same raw instance.
+  }
+  if (str.IsNull()) {
+    return false;
+  }
+  return Equals(str, 0, str.Length());
+}
+
+
+bool String::Equals(const String& str,
+                    intptr_t begin_index,
+                    intptr_t len) const {
+  ASSERT(begin_index >= 0);

[srdjan, 2012/07/13 23:22:23]

Compare raws as well.

[siva, 2012/07/16 18:29:23]

I don't think we can do a raw comparision and return here as we are trying to
see equality of a substring (begin_index, len).

On 2012/07/13 23:22:23, srdjan wrote:

+  ASSERT(begin_index == 0 || begin_index < str.Length());

[srdjan, 2012/07/13 23:22:23]

Add parenthesis

[siva, 2012/07/16 18:29:23]

Done.

+  ASSERT(len >= 0);
+  ASSERT(len <= str.Length());
+  if (len != this->Length()) {
+    return false;  // Lengths don't match.
+  }
+  for (intptr_t i = 0; i < len; i++) {
+    if (this->CharAt(i) != str.CharAt(begin_index + i)) {
+      return false;
+    }
+  }
+  return true;
+}
+
 }  // namespace dart
 
 #endif  // VM_OBJECT_H_