1
0
Fork 0
mirror of https://github.com/sockspls/badfish synced 2025-05-02 09:39:36 +00:00

Use sp->master instead of bestThread

Verified with:

dbg_hit_on(th != sp->master);

It is 100% equivalent on more than 200K hits.

No functional change.
This commit is contained in:
Marco Costalba 2015-02-21 10:40:59 +01:00
parent 7ff965eebf
commit e74c2df907

View file

@ -1583,7 +1583,6 @@ void Thread::idle_loop() {
// Try to late join to another split point if none of its slaves has // Try to late join to another split point if none of its slaves has
// already finished. // already finished.
SplitPoint* bestSp = NULL; SplitPoint* bestSp = NULL;
Thread* bestThread = NULL;
int bestScore = INT_MAX; int bestScore = INT_MAX;
for (Thread* th : Threads) for (Thread* th : Threads)
@ -1611,7 +1610,6 @@ void Thread::idle_loop() {
if (score < bestScore) if (score < bestScore)
{ {
bestSp = sp; bestSp = sp;
bestThread = th;
bestScore = score; bestScore = score;
} }
} }
@ -1627,7 +1625,7 @@ void Thread::idle_loop() {
if ( sp->allSlavesSearching if ( sp->allSlavesSearching
&& sp->slavesMask.count() < MAX_SLAVES_PER_SPLITPOINT && sp->slavesMask.count() < MAX_SLAVES_PER_SPLITPOINT
&& available_to(bestThread)) && available_to(sp->master))
{ {
sp->slavesMask.set(idx); sp->slavesMask.set(idx);
activeSplitPoint = sp; activeSplitPoint = sp;