[Lldb-commits] [PATCH] D13727: Add task pool to LLDB
Zachary Turner via lldb-commits
lldb-commits at lists.llvm.org
Wed Oct 14 11:37:11 PDT 2015
zturner added a comment.
I thought about this some more and I'm fine with going with a single implementation and not using std async. It would be nice to take advantage of any deep optimizations std::async provides over a hand-rolled solution, but at the same time there's a cost to adding complexity and asymmetric implementations, even if the asymmetry is hidden behind this interface. And the gain we get from goign single threaded -> this implementation is much greater than we would theoretically get from going from this implementation -> std async.
I'll look more closely at the implementation and see if I have other comments, but the general idea is fine.
More information about the lldb-commits