Improve load IC so it can call a native accessor even if the holder is

src/ic.cc

Index: src/ic.cc
===================================================================
--- src/ic.cc	(revision 12254)
+++ src/ic.cc	(working copy)
@@ -989,7 +989,6 @@
         if (callback->IsAccessorInfo()) {
           Handle<AccessorInfo> info = Handle<AccessorInfo>::cast(callback);
           if (v8::ToCData<Address>(info->getter()) == 0) return;
-          if (!holder->HasFastProperties()) return;

[Michael Starzinger, 2012/08/13 17:47:16]

What about the KeyedLoadIC?

           if (!info->IsCompatibleReceiver(*receiver)) return;
           code = isolate()->stub_cache()->ComputeLoadCallback(
               name, receiver, holder, info);
@@ -997,7 +996,6 @@
           Handle<Object> getter(Handle<AccessorPair>::cast(callback)->getter());
           if (!getter->IsJSFunction()) return;
           if (holder->IsGlobalObject()) return;
-          if (!holder->HasFastProperties()) return;

[Michael Starzinger, 2012/08/13 17:47:16]

Why can we generate a JavaScript getter call for a slow holder now? We are not
doing any positive dictionary lookup there.

           code = isolate()->stub_cache()->ComputeLoadViaGetter(
               name, receiver, holder, Handle<JSFunction>::cast(getter));
         } else {