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

[Greg Clayton via lldb-commits]

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