Improve property access with keys that can always be converted into a symbol string.

src/ic.cc

Index: src/ic.cc
===================================================================
--- src/ic.cc	(revision 7451)
+++ src/ic.cc	(working copy)
@@ -1143,6 +1143,15 @@
 MaybeObject* KeyedLoadIC::Load(State state,
                                Handle<Object> object,
                                Handle<Object> key) {
+  // Check for values that can be converted into a symbol.

[Mads Ager (chromium), 2011/03/31 09:41:49]

Could you file a bug report stating that this is put in because Kraken 1.1 is
not released in the web runner. Kraken 1.0 accesses undefined and NaN in arrays
a lot and this speeds it up. We have put it in so people do not make wrong
conclusions based on broken benchmarks. Once the web runner is updated to Kraken
1.1 which does correct array accesses we should get rid of this again.

+  HandleScope scope(isolate());
+  if (key->IsHeapNumber() &&
+      isnan(HeapNumber::cast(*key)->value())) {
+    key = isolate()->factory()->nan_symbol();
+  } else if (key->IsUndefined()) {
+    key = isolate()->factory()->undefined_symbol();
+  }
+
   if (key->IsSymbol()) {
     Handle<String> name = Handle<String>::cast(key);