VM: Enable collection of unoptimized code for optimized functions.

runtime/vm/raw_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/raw_object.h"
 
 #include "vm/class_table.h"
 #include "vm/dart.h"
 #include "vm/freelist.h"
 #include "vm/isolate.h"
@@ skipping 381 matching lines @@
 }
 
 
 intptr_t RawRedirectionData::VisitRedirectionDataPointers(
     RawRedirectionData* raw_obj, ObjectPointerVisitor* visitor) {
   visitor->VisitPointers(raw_obj->from(), raw_obj->to());
   return RedirectionData::InstanceSize();
 }
 
 
-bool RawFunction::SkipCode(RawFunction* raw_fun) {
+bool RawFunction::CheckUsageCounter(RawFunction* raw_fun) {
   // NOTE: This code runs while GC is in progress and runs within
   // a NoHandleScope block. Hence it is not okay to use regular Zone or
   // Scope handles. We use direct stack handles, and so the raw pointers in
   // these handles are not traversed. The use of handles is mainly to
   // be able to reuse the handle based code and avoid having to add
   // helper functions to the raw object interface.
   Function fn;
   fn = raw_fun;
 
+  // The function may not have code.
+  if (!fn.HasCode()) return false;
+  // These may not increment the usage counter.
+  if (fn.is_intrinsic()) return false;
+
+  if (fn.usage_counter() >= 0) {
+    fn.set_usage_counter(fn.usage_counter() / 2);
+  }
+  return FLAG_always_drop_code || (fn.usage_counter() == 0);
+}
+
+
+bool RawFunction::ShouldSkipCode(RawCode* raw_code) {

[Vyacheslav Egorov (Google), 2015/04/09 15:03:46]

I would call it ShouldVisitCode instead and invert results.

[Florian Schneider, 2015/04/09 15:07:50]

Done.

+  // NOTE: This code runs while GC is in progress and runs within
+  // a NoHandleScope block. Hence it is not okay to use regular Zone or
+  // Scope handles. We use direct stack handles, and so the raw pointers in
+  // these handles are not traversed. The use of handles is mainly to
+  // be able to reuse the handle based code and avoid having to add
+  // helper functions to the raw object interface.
   Code code;
-  code = fn.CurrentCode();
-
-  if (fn.HasCode() &&  // The function may not have code.
-      !fn.is_intrinsic() &&  // These may not increment the usage counter.
-      !code.is_optimized() &&
-      (fn.CurrentCode() == fn.unoptimized_code()) &&
-      !code.HasBreakpoint() &&
-      (fn.usage_counter() >= 0)) {
-    fn.set_usage_counter(fn.usage_counter() / 2);
-    if (FLAG_always_drop_code || (fn.usage_counter() == 0)) {
-      return true;
-    }
-  }
-  return false;
+  code = raw_code;
+  if (code.IsNull()) return false;
+  if (code.is_optimized()) return false;
+  if (code.HasBreakpoint()) return false;
+  return true;
 }
 
 
 intptr_t RawFunction::VisitFunctionPointers(RawFunction* raw_obj,
                                             ObjectPointerVisitor* visitor) {
-  if (visitor->visit_function_code() ||
-      !RawFunction::SkipCode(raw_obj)) {
+  if (visitor->visit_function_code() || !CheckUsageCounter(raw_obj)) {
     visitor->VisitPointers(raw_obj->from(), raw_obj->to());
+    return Function::InstanceSize();
+  }
+  visitor->VisitPointers(raw_obj->from(), raw_obj->to_no_code());
+
+  if (!ShouldSkipCode(raw_obj->ptr()->instructions_->ptr()->code_)) {
+    visitor->VisitPointer(
+        reinterpret_cast<RawObject**>(&raw_obj->ptr()->instructions_));
   } else {
-    GrowableArray<RawFunction*>* sfga = visitor->skipped_code_functions();
-    ASSERT(sfga != NULL);
-    sfga->Add(raw_obj);
-    visitor->VisitPointers(raw_obj->from(), raw_obj->to_no_code());
+    visitor->skipped_functions()->Add(raw_obj);
+  }
+
+  if (!ShouldSkipCode(raw_obj->ptr()->unoptimized_code_)) {
+    visitor->VisitPointer(
+        reinterpret_cast<RawObject**>(&raw_obj->ptr()->unoptimized_code_));
+  } else {
+    visitor->skipped_functions()->Add(raw_obj);
   }
   return Function::InstanceSize();
 }
 
 
 intptr_t RawField::VisitFieldPointers(RawField* raw_obj,
                                       ObjectPointerVisitor* visitor) {
   visitor->VisitPointers(raw_obj->from(), raw_obj->to());
   return Field::InstanceSize();
 }
@@ skipping 457 matching lines @@
 intptr_t RawUserTag::VisitUserTagPointers(
     RawUserTag* raw_obj, ObjectPointerVisitor* visitor) {
   // Make sure that we got here with the tagged pointer as this.
   ASSERT(raw_obj->IsHeapObject());
   visitor->VisitPointers(raw_obj->from(), raw_obj->to());
   return UserTag::InstanceSize();
 }
 
 
 }  // namespace dart