Fix assert in PagedSpace::SizeOfObject for concurrent sweeping.

src/spaces.cc

Index: src/spaces.cc
diff --git a/src/spaces.cc b/src/spaces.cc
index 91f8b81ed8c27be4702ee51b9a14d2dca1f6f8fe..1d6a4ac36e0ed2bb0268977e771b31100f3a1448 100644
--- a/src/spaces.cc
+++ b/src/spaces.cc
@@ -2476,6 +2476,13 @@ bool PagedSpace::ReserveSpace(int size_in_bytes) {
 }
 
 
+intptr_t PagedSpace::SizeOfObjects() {
+  ASSERT(!IsLazySweepingComplete() || (unswept_free_bytes_ == 0) ||
+         heap()->mark_compact_collector()->IsConcurrentSweepingInProgress());

[Michael Starzinger, 2013/03/04 15:26:29]

Better use the following assertion ...

ASSERT(heap()->IsSweepingComplete() || (unswept_free_bytes_ == 0));

... and move it back to the header so that it can be inlined.

[Hannes Payer (out of office), 2013/03/04 15:47:41]

Done.

I changed the ASSERT as suggested, but now it is weaker.

+  return Size() - unswept_free_bytes_ - (limit() - top());
+}
+
+
 // After we have booted, we have created a map which represents free space
 // on the heap.  If there was already a free list then the elements on it
 // were created with the wrong FreeSpaceMap (normally NULL), so we need to