Revision: 13077
Author:   [email protected]
Date:     Wed Nov 28 06:02:30 2012
Log: Relax restrictions on CONSTANT_FUNCTION descriptors in TransformPropertiesToFastFor.

Since r10174 they are not required to be in new space.

[email protected]
BUG=

Review URL: https://codereview.chromium.org/11418181
http://code.google.com/p/v8/source/detail?r=13077

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

=======================================
--- /branches/bleeding_edge/src/objects.cc      Wed Nov 28 00:35:46 2012
+++ /branches/bleeding_edge/src/objects.cc      Wed Nov 28 06:02:30 2012
@@ -13269,8 +13269,7 @@
       PropertyType type = DetailsAt(i).type();
       ASSERT(type != FIELD);
       instance_descriptor_length++;
-      if (type == NORMAL &&
-          (!value->IsJSFunction() || heap->InNewSpace(value))) {
+      if (type == NORMAL && !value->IsJSFunction()) {
         number_of_fields += 1;
       }
     }
@@ -13335,7 +13334,7 @@
       int enumeration_index = details.descriptor_index();
       PropertyType type = details.type();

-      if (value->IsJSFunction() && !heap->InNewSpace(value)) {
+      if (value->IsJSFunction()) {
         ConstantFunctionDescriptor d(key,
                                      JSFunction::cast(value),
                                      details.attributes(),

--
v8-dev mailing list
[email protected]
http://groups.google.com/group/v8-dev

Reply via email to