Skip to content

Commit

Permalink
Clarify we don't late join with only 2 threads
Browse files Browse the repository at this point in the history
Thanks to Gary for pointing this out.

No functional change.
  • Loading branch information
mcostalba committed Feb 19, 2015
1 parent 950c843 commit 667f350
Showing 1 changed file with 3 additions and 1 deletion.
4 changes: 3 additions & 1 deletion src/search.cpp
Expand Up @@ -1604,8 +1604,10 @@ void Thread::idle_loop() {
{
assert(this != Threads[i]);
assert(!(this_sp && this_sp->slavesMask.none()));
assert(Threads.size() > 2);

// Compute the recursive split points chain size
// Prefer to join to SP with few parents to reduce the probability
// that a cut-off occurs above us, and hence we waste our work.
int level = -1;
for (SplitPoint* spp = Threads[i]->activeSplitPoint; spp; spp = spp->parentSplitPoint)
level++;
Expand Down

0 comments on commit 667f350

Please sign in to comment.