Reviewers: Jakob,
Message:
PTAL.
Description:
Directly set the value in the descriptors pointer to avoid checks.
This is the only place from where we are allowed to modify the value of
the descriptors pointer.
Please review this at https://chromiumcodereview.appspot.com/10905266/
SVN Base: https://v8.googlecode.com/svn/branches/bleeding_edge
Affected files:
M src/objects.cc
Index: src/objects.cc
diff --git a/src/objects.cc b/src/objects.cc
index
0bcabdf50e63ef6daa0ec451a3e0597eeca5ed8e..3c193a5271502e38bf00ec129b961d1ef662df2c
100644
--- a/src/objects.cc
+++ b/src/objects.cc
@@ -1810,7 +1810,8 @@ MaybeObject*
JSObject::ConvertTransitionToMapTransition(
// new_map and install its descriptors in the old_map. Since the
old_map
// stores the descriptors for the new_map, remove the transition array
of
// the new_map that is only in place to store the descriptors.
-
old_map->transitions()->set_descriptors(new_map->instance_descriptors());
+ old_map->transitions()->descriptors_pointer()->set_value(
+ new_map->instance_descriptors());
new_map->ClearTransitions(GetHeap());
old_map->set_owns_descriptors(false);
Map* map;
--
v8-dev mailing list
[email protected]
http://groups.google.com/group/v8-dev
