Estimate a (close) upper bound on the size of black-marked objects on each page.

src/ia32/stub-cache-ia32.cc

Index: src/ia32/stub-cache-ia32.cc
diff --git a/src/ia32/stub-cache-ia32.cc b/src/ia32/stub-cache-ia32.cc
index 2e0d2857c86ac79cf82ea126093032b3d448f7d0..9702aa67744caa39eae49f77983cd9021bb0de32 100644
--- a/src/ia32/stub-cache-ia32.cc
+++ b/src/ia32/stub-cache-ia32.cc
@@ -1545,6 +1545,11 @@ MaybeObject* CallStubCompiler::CompileArrayPushCall(Object* object,
       // Increment element's and array's sizes.
       __ add(FieldOperand(ebx, FixedArray::kLengthOffset),
              Immediate(Smi::FromInt(kAllocationDelta)));
+
+      // TODO(gc): This only happen in new-space, where we don't

[Erik Corry, 2011/06/24 09:46:17]

I would prefer just a Note: rather than a TODO(gc): since there is no reason to
expect that this will ever get fixed.

[Lasse Reichstein, 2011/06/24 11:09:08]

Done.

+      // care about the black-byte-count. Otherwise we should
+      // update that too if the object is black.
+
       __ mov(FieldOperand(edx, JSArray::kLengthOffset), eax);
 
       __ ret((argc + 1) * kPointerSize);