HCha/Stockfish
1
0
Fork 0
mirror of https://github.com/HChaZZY/Stockfish.git synced 2025-12-25 03:26:24 +08:00
Code Issues Packages Projects Releases Wiki Activity

Fix the counter in for_each_index_with_workers going out of scope before workers finish.

Browse Source
This commit is contained in:
Tomasz Sobczyk
2020-10-30 10:36:39 +01:00
committed by nodchip
parent a56d8124d8
commit 8c81bbd3db
1 changed files with 7 additions and 2 deletions
Download Patch File Download Diff File Expand all files Collapse all files

9
src/thread.h
View File

@@ -135,10 +135,15 @@ struct ThreadPool : public std::vector<Thread*> {
typename Detail::TypeIdentity<IndexT>::Type end,
FuncT func)
{
std::atomic<IndexT> i_atomic = begin;
// This value must outlive the function call.
// It's fairly safe if we make it static
// because for_each_index_with_workers
// is not reentrant nor thread safe.
static std::atomic<IndexT> i_atomic;
i_atomic.store(begin);
execute_with_workers(
[&i_atomic, end, func](Thread& th) mutable {
[end, func](Thread& th) mutable {
for(;;) {
const auto i = i_atomic.fetch_add(1);
if (i >= end)