Reviewers: Dmitry Lomov (chromium),
Message:
PTAL
Description:
Use SimpleLoadField for JSObjectFieldAccessors, rather than compiling a new
stub.
Please review this at https://chromiumcodereview.appspot.com/76403002/
SVN Base: https://v8.googlecode.com/svn/branches/bleeding_edge
Affected files (+4, -3 lines):
M src/ic.cc
Index: src/ic.cc
diff --git a/src/ic.cc b/src/ic.cc
index
9b304057ade037d471539297de6c3c4762d9bc73..82f598b0cd227b74dbb2aedec13b0058d30a2111
100644
--- a/src/ic.cc
+++ b/src/ic.cc
@@ -1242,10 +1242,11 @@ Handle<Code> LoadIC::CompileHandler(LookupResult*
lookup,
Handle<JSObject> receiver = Handle<JSObject>::cast(object);
Handle<Map> map(receiver->map());
if (Accessors::IsJSObjectFieldAccessor(map, name, &object_offset))
{
- PropertyIndex index =
+ PropertyIndex field =
PropertyIndex::NewHeaderIndex(object_offset / kPointerSize);
- return compiler.CompileLoadField(
- receiver, receiver, name, index, Representation::Tagged());
+ return SimpleFieldLoad(field.translate(receiver),
+ field.is_inobject(receiver),
+ Representation::Tagged());
}
}
--
--
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.
