Reviewers: Jakob,

Description:
Actually increment queue length for parallel recompile queue.


[email protected]
BUG=


Please review this at https://chromiumcodereview.appspot.com/12039088/

SVN Base: https://v8.googlecode.com/svn/branches/bleeding_edge

Affected files:
  M src/flag-definitions.h
  M src/optimizing-compiler-thread.cc


Index: src/flag-definitions.h
diff --git a/src/flag-definitions.h b/src/flag-definitions.h
index 3bf32a9b14588fcf17d39a154d44fb70217a5765..a7980ad20707bffe50f2350b777b0153b4263c42 100644
--- a/src/flag-definitions.h
+++ b/src/flag-definitions.h
@@ -234,7 +234,7 @@ DEFINE_bool(opt_safe_uint32_operations, true,
"allow uint32 values on optimize frames if they are used only in "
             "safe operations")

-DEFINE_bool(parallel_recompilation, false,
+DEFINE_bool(parallel_recompilation, true,
             "optimizing hot functions asynchronously on a separate thread")
DEFINE_bool(trace_parallel_recompilation, false, "track parallel recompilation")
 DEFINE_int(parallel_recompilation_queue_length, 2,
Index: src/optimizing-compiler-thread.cc
diff --git a/src/optimizing-compiler-thread.cc b/src/optimizing-compiler-thread.cc index e41c352f486624179e31d981e5d2bd6736d9ef6d..1cf0434fb768d6405e53e4dd463d1112d914f317 100644
--- a/src/optimizing-compiler-thread.cc
+++ b/src/optimizing-compiler-thread.cc
@@ -135,6 +135,8 @@ Handle<SharedFunctionInfo>

 void OptimizingCompilerThread::QueueForOptimization(
     OptimizingCompiler* optimizing_compiler) {
+  ASSERT(IsQueueAvailable());
+  Barrier_AtomicIncrement(&queue_length_, static_cast<Atomic32>(1));
   input_queue_.Enqueue(optimizing_compiler);
   input_queue_semaphore_->Signal();
 }


--
--
v8-dev mailing list
[email protected]
http://groups.google.com/group/v8-dev


Reply via email to