[runtime] Record runtime call stats for Map::TransitionTo*Property, JSObject::OptimizeAsPrototype t…

src/lookup.cc

Index: src/lookup.cc
diff --git a/src/lookup.cc b/src/lookup.cc
index 6c73244843908b6396f58afd3175d9c1b8763a13..f455a2903b19fd9447c86ee9991bb229af5d9bf6 100644
--- a/src/lookup.cc
+++ b/src/lookup.cc
@@ -352,9 +352,14 @@ void LookupIterator::Delete() {
     ElementsAccessor* accessor = object->GetElementsAccessor();
     accessor->Delete(object, number_);
   } else {
-    PropertyNormalizationMode mode = holder->map()->is_prototype_map()
-                                         ? KEEP_INOBJECT_PROPERTIES
-                                         : CLEAR_INOBJECT_PROPERTIES;
+    bool is_prototype_map = holder->map()->is_prototype_map();
+    RuntimeCallTimerScope stats_scope(
+        isolate_, is_prototype_map
+                      ? &RuntimeCallStats::PrototypeObject_DeleteProperty
+                      : &RuntimeCallStats::Object_DeleteProperty);

[Camillo Bruni, 2016/05/11 13:01:46]

I hope this case doesn't happen too much, I've seen quite some impact by the
counter when using them for the builtins. That's why I added the separate branch
that only jumps there if we enable the flag. Otherwise I could still see the
overhead, since clang inlined the timer scope nevertheless and probably reserved
stack in either case.

+
+    PropertyNormalizationMode mode =
+        is_prototype_map ? KEEP_INOBJECT_PROPERTIES : CLEAR_INOBJECT_PROPERTIES;
 
     if (holder->HasFastProperties()) {
       JSObject::NormalizeProperties(Handle<JSObject>::cast(holder), mode, 0,