[PATCH] D36607: [Support/Parallel] - Do not spawn thread for single/last task.

George Rimar via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Mon Aug 14 05:56:20 PDT 2017


grimar abandoned this revision.
grimar added a comment.

I was mistaken. Did not notice that `TaskGroup::spawn()` has different implementation for windows/non-windows case.
Linux implementation actually based on a `ThreadPoolExecutor` which implemented via thread pool, 
so l was mistaken when said that it creates 65k threads, issue happens because of something else.

This patch still looks as correct optimization for me and solves the problem too, avoiding using of unnecessary thread. 
Looks it helps because avoids using implementation based on thread pool, which looks have some narrow place. 
I am going to find real issue now.


https://reviews.llvm.org/D36607





More information about the llvm-commits mailing list