[Lldb-commits] [PATCH] D13727: Add task pool to LLDB

Greg Clayton via lldb-commits lldb-commits at lists.llvm.org
Thu Oct 15 10:43:38 PDT 2015


clayborg added a comment.

In http://reviews.llvm.org/D13727#268091, @tberghammer wrote:

> I agree that parking the threads in the kernel should be very cheap (I am not sure about Windows) but on high end multi core machines having 40+ threads around just to wait for work is a bit strange for me too and it definitely makes debugging of LLDB more difficult.


Ok, maybe we can pick a larger timeout then like say 2 seconds?

> In the near future I expect that we will use the TaskPool only for very heavy computations while we filling up our internal caches (such as debug info parsing or symtab parsing) and then don't use it for anything later.


I want to used it in many ModuleList functions like:

ModuleList::FindFunctionSymbols()
ModuleList::FindFunctions()
ModuleList::FindCompileUnits()
ModuleList::FindGlobalVariables()
ModuleList::FindSymbolsWithNameAndType()
ModuleList::FindSymbolsMatchingRegExAndType()

and many more.


http://reviews.llvm.org/D13727





More information about the lldb-commits mailing list