Changeset 38080 in webkit for trunk/JavaScriptCore/runtime/StructureID.cpp

Timestamp:

Nov 3, 2008, 2:43:00 PM (17 years ago)

Author:

[email protected]

Message:

2008-11-03 Sam Weinig <​[email protected]>

Reviewed by Mark Rowe.

Move #define to turn on dumping StructureID statistics to StructureID.cpp so that
turning it on does not require a full rebuild.

• runtime/StructureID.cpp: (JSC::StructureID::dumpStatistics):
• runtime/StructureID.h:

File:

• trunk/JavaScriptCore/runtime/StructureID.cpp (modified) (4 diffs)

trunk/JavaScriptCore/runtime/StructureID.cpp

@@ -39 +39 @@
 #endif
 
+#define DUMP_STRUCTURE_ID_STATISTICS 0
+
 #ifndef NDEBUG
 #define DO_PROPERTYMAP_CONSTENCY_CHECK 0
@@ -71 +73 @@
 #if DUMP_STRUCTURE_ID_STATISTICS
 static HashSet<StructureID*> liveStructureIDSet;
+#endif
 
 void StructureID::dumpStatistics()
 {
+#if DUMP_STRUCTURE_ID_STATISTICS
     unsigned numberLeaf = 0;
     unsigned numberUsingSingleSlot = 0;
@@ -93 +97 @@
 
         if (structureID->m_propertyTable)
-            totalPropertyMapsSize += PropertyMapHashTable::allocationSize(m_propertyTable->size);;
+            totalPropertyMapsSize += PropertyMapHashTable::allocationSize(structureID->m_propertyTable->size);
     }
 
@@ -104 +108 @@
     printf("Size of sum of all property maps: %d\n", totalPropertyMapsSize);
     printf("Size of average of all property maps: %f\n", static_cast<double>(totalPropertyMapsSize) / static_cast<double>(liveStructureIDSet.size()));
-}
-#endif
+#else
+    printf("Dumping StructureID statistics is not enabled.\n");
+#endif
+}
 
 StructureID::StructureID(JSValue* prototype, const TypeInfo& typeInfo)