[Lldb-commits] [PATCH] D37930: Use ThreadLauncher to launch TaskPool threads

Tamas Berghammer via Phabricator via lldb-commits lldb-commits at lists.llvm.org
Mon Sep 18 03:10:48 PDT 2017


tberghammer accepted this revision.
tberghammer added a comment.

Looks good



================
Comment at: source/Utility/TaskPool.cpp:61
+    lldb_private::ThreadLauncher::LaunchThread("task-pool.worker", WorkerPtr,
+                                               this, nullptr, 8 * 1024 * 1024)
+        .Release();
----------------
(nit): Can you create a named constant for the min stack size as just based on this line I would have no idea what does that number do?


https://reviews.llvm.org/D37930





More information about the lldb-commits mailing list