Initial design for background compilation

runtime/vm/compiler.h

 // 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.
 
 #ifndef VM_COMPILER_H_
 #define VM_COMPILER_H_
 
 #include "vm/allocation.h"
 #include "vm/growable_array.h"
 #include "vm/runtime_entry.h"
+#include "vm/thread_pool.h"
 
 namespace dart {
 
 // Forward declarations.
 class Class;
 class Function;
 class Library;
 class ParsedFunction;
 class RawInstance;
 class Script;
@@ skipping 71 matching lines @@
   static bool allow_recompilation() { return allow_recompilation_; }
   static void set_allow_recompilation(bool value) {
     allow_recompilation_ = value;
   }
 
  private:
   static bool always_optimize_;
   static bool allow_recompilation_;
 };
 
+
+class CompileOptimizedTask : public ThreadPool::Task {

[koda, 2015/10/02 16:53:46]

The name "task" is slightly confusing to me, since it contains a work queue
itself. Did you consider something a bit more general like BackgroundCompiler or
ConcurrentOptimizer? It's up to you. (If you do decide to change it, then please
also change the isolate field name.)

[srdjan, 2015/10/02 18:19:20]

Renamed to BackgroundCompiler. Why should "Isolate* isolate_" name be changed?

[koda, 2015/10/02 18:58:05]

Not that field; I meant the added fields/accessors in the class Isolate.

+ public:
+  explicit CompileOptimizedTask(Isolate* isolate);
+
+  virtual void Run();
+
+  void CompileOptimized(const Function& function);
+
+  void Add(const Function& f);
+  RawFunction* RemoveOrNull();
+
+  static void EnsureInit(Isolate* isolate);
+
+  static void Stop(CompileOptimizedTask* task);
+
+ private:
+  Isolate* isolate_;
+  bool running_;
+  bool* done_;
+  Monitor* monitor_;
+  Monitor* done_monitor_;
+
+  DISALLOW_IMPLICIT_CONSTRUCTORS(CompileOptimizedTask);
+};
+
 }  // namespace dart
 
 #endif  // VM_COMPILER_H_