Cleanup native, collected, and stub code handling in profiler

runtime/vm/object.cc

 // Copyright (c) 2012, 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/object.h"
 
 #include "include/dart_api.h"
 #include "platform/assert.h"
 #include "vm/assembler.h"
 #include "vm/cpu.h"
@@ skipping 4510 matching lines @@
 
 bool Function::HasBreakpoint() const {
   return Isolate::Current()->debugger()->HasBreakpoint(*this);
 }
 
 
 void Function::SetCode(const Code& value) const {
   StorePointer(&raw_ptr()->code_, value.raw());
   ASSERT(Function::Handle(value.function()).IsNull() ||
     (value.function() == this->raw()));
-  value.set_function(*this);
+  value.set_owner_function(*this);

[Ivan Posva, 2014/02/28 21:46:32]

How about just value.set_owner(*this)? The set_owner function knows what to do
with this anyway.

 }
 
 
 void Function::SwitchToUnoptimizedCode() const {
   ASSERT(HasOptimizedCode());
 
   const Code& current_code = Code::Handle(CurrentCode());
 
   // Optimized code object might have been actually fully produced by the
   // intrinsifier in this case nothing has to be done. In fact an attempt to
@@ skipping 308 matching lines @@
   Function& outer_function = Function::Handle(parent_function());
   while (outer_function.IsLocalFunction()) {
     outer_function = outer_function.parent_function();
   }
   return outer_function.IsFactory();
 }
 
 
 void Function::set_name(const String& value) const {
   ASSERT(value.IsSymbol());
+  if (value.IsNull()) {

[Ivan Posva, 2014/02/28 21:46:32]

Debugging code...

+    printf("Function given null name.\n");

[turnidge, 2014/02/28 19:23:37]

Debug printf?

[Cutch, 2014/02/28 19:37:08]

Done.

+  }
+  if (value.IsSymbol() == false) {
+    printf("Function name is not a symbol.\n");

[turnidge, 2014/02/28 19:23:37]

Debug printf?

[Cutch, 2014/02/28 19:37:08]

Done.

+  }
   StorePointer(&raw_ptr()->name_, value.raw());
 }
 
 
 void Function::set_owner(const Object& value) const {
   ASSERT(!value.IsNull());
   StorePointer(&raw_ptr()->owner_, value.raw());
 }
 
 
@@ skipping 5381 matching lines @@
   }
 }
 
 
 // Check if object matches find condition.
 bool Code::FindRawCodeVisitor::FindObject(RawObject* obj) const {
   return RawInstructions::ContainsPC(obj, pc_);
 }
 
 
-RawCode* Code::LookupCode(uword pc) {
-  Isolate* isolate = Isolate::Current();
+RawCode* Code::LookupCodeInIsolate(Isolate* isolate, uword pc) {
+  ASSERT((isolate == Isolate::Current()) || (isolate == Dart::vm_isolate()));
   NoGCScope no_gc;
   FindRawCodeVisitor visitor(pc);
   RawInstructions* instr;
   if (isolate->heap() == NULL) {
     return Code::null();
   }
   instr = isolate->heap()->FindObjectInCodeSpace(&visitor);
   if (instr != Instructions::null()) {
     return instr->ptr()->code_;
   }
   return Code::null();
 }
 
 
+RawCode* Code::LookupCode(uword pc) {
+  return LookupCodeInIsolate(Isolate::Current(), pc);
+}
+
+
+RawCode* Code::LookupCodeInVmIsolate(uword pc) {
+  return LookupCodeInIsolate(Dart::vm_isolate(), pc);
+}
+
+
 intptr_t Code::GetTokenIndexOfPC(uword pc) const {
   intptr_t token_pos = -1;
   const PcDescriptors& descriptors = PcDescriptors::Handle(pc_descriptors());
   for (intptr_t i = 0; i < descriptors.Length(); i++) {
     if (descriptors.PC(i) == pc) {
       token_pos = descriptors.TokenPos(i);
       break;
     }
   }
   return token_pos;
@@ skipping 34 matching lines @@
   return chars;
 }
 
 
 void Code::PrintToJSONStream(JSONStream* stream, bool ref) const {
   JSONObject jsobj(stream);
   jsobj.AddProperty("type", JSONType(ref));
   jsobj.AddPropertyF("id", "code/%" Px "", EntryPoint());
   jsobj.AddPropertyF("start", "%" Px "", EntryPoint());
   jsobj.AddPropertyF("end", "%" Px "", EntryPoint() + Size());
-  Function& func = Function::Handle();
-  func ^= function();
-  ASSERT(!func.IsNull());
-  String& name = String::Handle();
-  ASSERT(!func.IsNull());
-  name ^= func.name();
-  const char* internal_function_name = name.ToCString();
-  jsobj.AddPropertyF("name", "%s%s", is_optimized() ? "*" : "",
-                                     internal_function_name);
-  name ^= func.QualifiedUserVisibleName();
-  const char* function_name = name.ToCString();
-  jsobj.AddPropertyF("user_name", "%s%s", is_optimized() ? "*" : "",
-                                          function_name);
+  jsobj.AddProperty("is_optimized", is_optimized());
+  jsobj.AddProperty("is_alive", is_alive());
+  if (OwnerIsNotNullFunction()) {

[turnidge, 2014/02/28 19:23:37]

I don't care for negations in predicate names unless it is a performance issue. 
Can we get away with something like this:

if (OwnerIsFunction()) {
  ...
  if (func.IsNull() {
    // Regular stub. 
  } else {
  }
}

[Cutch, 2014/02/28 19:37:08]

Done.

+    // Dart function.
+    jsobj.AddProperty("kind", "Dart");
+    Function& func = Function::Handle();
+    String& name = String::Handle();
+    func ^= function();
+    ASSERT(!func.IsNull());
+    name ^= func.name();
+    ASSERT(!name.IsNull());
+    const char* name_c = name.ToCString();
+    jsobj.AddPropertyF("name", "%s%s", is_optimized() ? "*" : "", name_c);
+    name ^= func.QualifiedUserVisibleName();
+    ASSERT(!name.IsNull());
+    name_c = name.ToCString();
+    jsobj.AddPropertyF("user_name", "%s%s", is_optimized() ? "*" : "", name_c);
+    jsobj.AddProperty("function", Object::Handle(function()));
+  } else if (OwnerIsAllocationStub()) {
+    // Allocation stub.
+    jsobj.AddProperty("kind", "DartAllocationStub");
+    Isolate* isolate = Isolate::Current();
+    ClassTable* class_table = isolate->class_table();
+    intptr_t cid = Smi::Value(allocation_stub_class_id());
+    Class& cls = Class::Handle();
+    cls ^= class_table->At(cid);
+    ASSERT(!cls.IsNull());
+    String& cls_name = String::Handle(cls.UserVisibleName());
+    ASSERT(!cls_name.IsNull());
+    const char* name = cls_name.ToCString();
+    jsobj.AddPropertyF("name", "Allocation stub for %s\n", name);
+    jsobj.AddPropertyF("user_name", "Allocation stub for %s\n", name);
+  } else {
+    // Regular stub.
+    jsobj.AddProperty("kind", "DartStub");
+    const char* name = StubCode::NameOfStub(EntryPoint());
+    ASSERT(name != NULL);
+    jsobj.AddProperty("name", name);
+    jsobj.AddProperty("user_name", name);
+  }
   if (ref) {
     return;
   }
-  jsobj.AddProperty("is_optimized", is_optimized());
-  jsobj.AddProperty("is_alive", is_alive());
-  jsobj.AddProperty("function", Object::Handle(function()));
   JSONArray jsarr(&jsobj, "disassembly");
   if (is_alive()) {
     // Only disassemble alive code objects.
     DisassembleToJSONStream formatter(jsarr);
     Disassemble(&formatter);
   }
 }
 
 
 uword Code::GetPatchCodePc() const {
@@ skipping 7066 matching lines @@
   return "_MirrorReference";
 }
 
 
 void MirrorReference::PrintToJSONStream(JSONStream* stream, bool ref) const {
   Instance::PrintToJSONStream(stream, ref);
 }
 
 
 }  // namespace dart