Revision: 12916
Author: [email protected]
Date: Fri Nov 9 03:30:05 2012
Log: Keep the number of descriptors below
DescriptorArray::kMaxNumberOfDescriptors even for accessors
Review URL: https://codereview.chromium.org/11362182
http://code.google.com/p/v8/source/detail?r=12916
Modified:
/branches/bleeding_edge/src/objects.cc
=======================================
--- /branches/bleeding_edge/src/objects.cc Fri Nov 9 02:57:54 2012
+++ /branches/bleeding_edge/src/objects.cc Fri Nov 9 03:30:05 2012
@@ -4712,7 +4712,9 @@
// to do a lookup, which seems to be a bit of overkill.
Heap* heap = GetHeap();
bool only_attribute_changes = getter->IsNull() && setter->IsNull();
- if (HasFastProperties() && !only_attribute_changes) {
+ if (HasFastProperties() && !only_attribute_changes &&
+ (map()->NumberOfOwnDescriptors() <
+ DescriptorArray::kMaxNumberOfDescriptors)) {
MaybeObject* getterOk = heap->undefined_value();
if (!getter->IsNull()) {
getterOk = DefineFastAccessor(name, ACCESSOR_GETTER, getter,
attributes);
--
v8-dev mailing list
[email protected]
http://groups.google.com/group/v8-dev
