Revision: 17922
Author:   [email protected]
Date:     Wed Nov 20 13:25:40 2013 UTC
Log: Use SimpleLoadField for JSObjectFieldAccessors, rather than compiling a new stub.

[email protected]

Review URL: https://chromiumcodereview.appspot.com/76403002
http://code.google.com/p/v8/source/detail?r=17922

Modified:
 /branches/bleeding_edge/src/ic.cc

=======================================
--- /branches/bleeding_edge/src/ic.cc   Mon Nov 18 17:18:14 2013 UTC
+++ /branches/bleeding_edge/src/ic.cc   Wed Nov 20 13:25:40 2013 UTC
@@ -1237,15 +1237,12 @@
       return isolate()->builtins()->LoadIC_Normal();
     case CALLBACKS: {
       // Use simple field loads for some well-known callback properties.
-      int object_offset;
       if (object->IsJSObject()) {
         Handle<JSObject> receiver = Handle<JSObject>::cast(object);
         Handle<Map> map(receiver->map());
+        int object_offset;
if (Accessors::IsJSObjectFieldAccessor(map, name, &object_offset)) {
-          PropertyIndex index =
-              PropertyIndex::NewHeaderIndex(object_offset / kPointerSize);
-          return compiler.CompileLoadField(
-              receiver, receiver, name, index, Representation::Tagged());
+          return SimpleFieldLoad(object_offset / kPointerSize);
         }
       }

--
--
v8-dev mailing list
[email protected]
http://groups.google.com/group/v8-dev
--- You received this message because you are subscribed to the Google Groups "v8-dev" group.
To unsubscribe from this group and stop receiving emails from it, send an email 
to [email protected].
For more options, visit https://groups.google.com/groups/opt_out.

Reply via email to