Display ICData entries at call sites, with links to the targets and guarded classes.

runtime/vm/coverage.cc

 // Copyright (c) 2013, the Dart project authors.  Please see the AUTHORS file
 // for details. All rights reserved. Use of this source code is governed by a
 // BSD-style license that can be found in the LICENSE file.
 
 #include "vm/coverage.h"
 
 #include "include/dart_api.h"
 
 #include "vm/compiler.h"
 #include "vm/isolate.h"
@@ skipping 35 matching lines @@
     (*map)[tkit.CurrentPosition()] = cur_line;
     if (tkit.CurrentTokenKind() == Token::kNEWLINE) {
       cur_line++;
     }
     tkit.Advance();
   }
 }
 
 
 void CodeCoverage::CompileAndAdd(const Function& function,
-                                 const JSONArray& hits_arr,
-                                 const GrowableArray<intptr_t>& pos_to_line) {
+                                 const JSONArray& hits_or_sites,
+                                 const GrowableArray<intptr_t>& pos_to_line,
+                                 bool as_call_sites) {
   Isolate* isolate = Isolate::Current();
   if (!function.HasCode()) {
     // If the function should not be compiled or if the compilation failed,
     // then just skip this method.
     // TODO(iposva): Maybe we should skip synthesized methods in general too.
     if (function.is_abstract() || function.IsRedirectingFactory()) {
       return;
     }
     if (function.IsNonImplicitClosureFunction() &&
         (function.context_scope() == ContextScope::null())) {
@@ skipping 38 matching lines @@
 #if defined(DEBUG)
       const Script& script = Script::Handle(function.script());
       intptr_t test_line = -1;
       script.GetTokenLocation(token_pos, &test_line, NULL);
       ASSERT(test_line == line);
 #endif
       // Merge hit data where possible.
       if (last_line == line) {
         last_count += ic_data->AggregateCount();
       } else {
-        if (last_line != -1) {
-          hits_arr.AddValue(last_line);
-          hits_arr.AddValue(last_count);
+        if ((last_line != -1) && !as_call_sites) {
+          hits_or_sites.AddValue(last_line);
+          hits_or_sites.AddValue(last_count);
         }
         last_count = ic_data->AggregateCount();
         last_line = line;
       }
+      if (as_call_sites) {
+        bool is_static_call = iter.Kind() == RawPcDescriptors::kUnoptStaticCall;
+        ic_data->PrintToJSONArray(hits_or_sites, token_pos, is_static_call);
+      }
     }
   }
   // Write last hit value if needed.
-  if (last_line != -1) {
-    hits_arr.AddValue(last_line);
-    hits_arr.AddValue(last_count);
+  if ((last_line != -1) && !as_call_sites) {
+    hits_or_sites.AddValue(last_line);
+    hits_or_sites.AddValue(last_count);
   }
 }
 
 
 void CodeCoverage::PrintClass(const Library& lib,
                               const Class& cls,
                               const JSONArray& jsarr,
-                              CoverageFilter* filter) {
+                              CoverageFilter* filter,
+                              bool as_call_sites) {
   Isolate* isolate = Isolate::Current();
   if (cls.EnsureIsFinalized(isolate) != Error::null()) {
     // Only classes that have been finalized do have a meaningful list of
     // functions.
     return;
   }
   Array& functions = Array::Handle(cls.functions());
   ASSERT(!functions.IsNull());
   Function& function = Function::Handle();
   Script& script = Script::Handle();
   String& saved_url = String::Handle();
   String& url = String::Handle();
   GrowableArray<intptr_t> pos_to_line;
   int i = 0;
   while (i < functions.Length()) {
     HANDLESCOPE(isolate);
     function ^= functions.At(i);
     script = function.script();
     saved_url = script.url();
     if (!filter->ShouldOutputCoverageFor(lib, script, cls, function)) {
       i++;
       continue;
     }
     ComputeTokenPosToLineNumberMap(script, &pos_to_line);
     JSONObject jsobj(&jsarr);
     jsobj.AddProperty("source", saved_url.ToCString());
     jsobj.AddProperty("script", script);
-    JSONArray hits_arr(&jsobj, "hits");
+    JSONArray hits_or_sites(&jsobj, as_call_sites ? "callSites" : "hits");
 
     // We stay within this loop while we are seeing functions from the same
     // source URI.
     while (i < functions.Length()) {
       function ^= functions.At(i);
       script = function.script();
       url = script.url();
       if (!url.Equals(saved_url)) {
         pos_to_line.Clear();
         break;
       }
       if (!filter->ShouldOutputCoverageFor(lib, script, cls, function)) {
         i++;
         continue;
       }
-      CompileAndAdd(function, hits_arr, pos_to_line);
+      CompileAndAdd(function, hits_or_sites, pos_to_line, as_call_sites);
       if (function.HasImplicitClosureFunction()) {
         function = function.ImplicitClosureFunction();
-        CompileAndAdd(function, hits_arr, pos_to_line);
+        CompileAndAdd(function, hits_or_sites, pos_to_line, as_call_sites);
       }
       i++;
     }
   }
 
   GrowableObjectArray& closures =
       GrowableObjectArray::Handle(cls.closures());
   if (!closures.IsNull()) {
     i = 0;
     pos_to_line.Clear();
     // We need to keep rechecking the length of the closures array, as handling
     // a closure potentially adds new entries to the end.
     while (i < closures.Length()) {
       HANDLESCOPE(isolate);
       function ^= closures.At(i);
       script = function.script();
       saved_url = script.url();
       if (!filter->ShouldOutputCoverageFor(lib, script, cls, function)) {
         i++;
         continue;
       }
       ComputeTokenPosToLineNumberMap(script, &pos_to_line);
       JSONObject jsobj(&jsarr);
       jsobj.AddProperty("source", saved_url.ToCString());
       jsobj.AddProperty("script", script);
-      JSONArray hits_arr(&jsobj, "hits");
+      JSONArray hits_or_sites(&jsobj, as_call_sites ? "callSites" : "hits");
 
       // We stay within this loop while we are seeing functions from the same
       // source URI.
       while (i < closures.Length()) {
         function ^= closures.At(i);
         script = function.script();
         url = script.url();
         if (!url.Equals(saved_url)) {
           pos_to_line.Clear();
           break;
         }
-        CompileAndAdd(function, hits_arr, pos_to_line);
+        CompileAndAdd(function, hits_or_sites, pos_to_line, as_call_sites);
         i++;
       }
     }
   }
 }
 
 
 void CodeCoverage::Write(Isolate* isolate) {
   if (FLAG_coverage_dir == NULL) {
     return;
   }
 
   Dart_FileOpenCallback file_open = Isolate::file_open_callback();
   Dart_FileWriteCallback file_write = Isolate::file_write_callback();
   Dart_FileCloseCallback file_close = Isolate::file_close_callback();
   if ((file_open == NULL) || (file_write == NULL) || (file_close == NULL)) {
     return;
   }
 
   JSONStream stream;
-  PrintJSON(isolate, &stream, NULL);
+  PrintJSON(isolate, &stream, NULL, false);
 
   const char* format = "%s/dart-cov-%" Pd "-%" Pd ".json";
   intptr_t pid = OS::ProcessId();
   intptr_t len = OS::SNPrint(NULL, 0, format,
                              FLAG_coverage_dir, pid, isolate->main_port());
   char* filename = Isolate::Current()->current_zone()->Alloc<char>(len + 1);
   OS::SNPrint(filename, len + 1, format,
               FLAG_coverage_dir, pid, isolate->main_port());
   void* file = (*file_open)(filename, true);
   if (file == NULL) {
     OS::Print("Failed to write coverage file: %s\n", filename);
     return;
   }
   (*file_write)(stream.buffer()->buf(), stream.buffer()->length(), file);
   (*file_close)(file);
 }
 
 
 void CodeCoverage::PrintJSON(Isolate* isolate,
                              JSONStream* stream,
-                             CoverageFilter* filter) {
+                             CoverageFilter* filter,
+                             bool as_call_sites) {
   CoverageFilterAll default_filter;
   if (filter == NULL) {
     filter = &default_filter;
   }
   const GrowableObjectArray& libs = GrowableObjectArray::Handle(
       isolate, isolate->object_store()->libraries());
   Library& lib = Library::Handle();
   Class& cls = Class::Handle();
   JSONObject coverage(stream);
   coverage.AddProperty("type", "CodeCoverage");
   {
     JSONArray jsarr(&coverage, "coverage");
     for (int i = 0; i < libs.Length(); i++) {
       lib ^= libs.At(i);
       ClassDictionaryIterator it(lib, ClassDictionaryIterator::kIteratePrivate);
       while (it.HasNext()) {
         cls = it.GetNextClass();
         ASSERT(!cls.IsNull());
-        PrintClass(lib, cls, jsarr, filter);
+        PrintClass(lib, cls, jsarr, filter, as_call_sites);
       }
     }
   }
 }
 
 
 }  // namespace dart