[libcxx-commits] [PATCH] D150284: [libc++][PSTL] Add a simple std::thread backend

[Nikolas Klauser via Phabricator via libcxx-commits]

philnik added inline comments.


================
Comment at: libcxx/include/__algorithm/pstl_backends/cpu_backends/thread.h:32
+    auto __max_thread_count = thread::hardware_concurrency();
+    return __max_thread_count == 0 ? 1 : __max_thread_count;
+  }());
----------------
EricWF wrote:
> This calculation seems a bit weird to me.
> 
> We're basically saying that if no concurrency is reported, then we'll use 2 (not one) threads.
> Why are we not just running the algorithm in the current thread in this case?
> 
> 
> Either way, it seems like we should be using the calling thread to do some work rather than just waiting on the other threads to join.
> 
Frankly, I don't really care. As I already said, this is a shitty backend and shouldn't be used in production. This is more of a placeholder than anything else. We should implement a proper thread pool later and not instantiate threads every time an algorithm is called. 


Repository:
  rG LLVM Github Monorepo

CHANGES SINCE LAST ACTION
  https://reviews.llvm.org/D150284/new/

https://reviews.llvm.org/D150284