kate/BadFish
1
0
Fork 0
mirror of https://github.com/sockspls/badfish synced 2025-04-29 16:23:09 +00:00
Code Issues Releases Wiki Activity

Fix some icc's "statement is unreachable" warnings

Browse source 
No functional change.

Signed-off-by: Marco Costalba <mcostalba@gmail.com>
This commit is contained in:
Marco Costalba 2010-10-30 18:44:34 +01:00
parent 2991ff0dc2
commit 49a6fee4fa
1 changed files with 22 additions and 20 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

30
src/search.cpp
View file

@ -968,6 +968,7 @@ namespace {
Key posKey;
Move ttMove, move, excludedMove, threatMove;
Depth ext, newDepth;
ValueType vt;
Value bestValue, value, oldAlpha;
Value refinedValue, nullValue, futilityBase, futilityValueScaled; // Non-PV specific
bool isCheck, singleEvasion, singularExtensionNode, moveIsCheck, captureOrPromotion, dangerous;
@ -987,7 +988,7 @@ namespace {
threatMove = sp->threatMove;
mateThreat = sp->mateThreat;
goto split_point_start;
}
} else {} // Hack to fix icc's "statement is unreachable" warning
// Step 1. Initialize node and poll. Polling can abort search
ThreadsMgr.incrementNodeCounter(threadID);
@ -1394,29 +1395,22 @@ split_point_start: // At split points actual search starts from here
threatMove, mateThreat, moveCount, &mp, PvNode);
}
if (SpNode)
{
// Here we have the lock still grabbed
sp->slaves[threadID] = 0;
lock_release(&(sp->lock));
return bestValue;
}
// Step 19. Check for mate and stalemate
// All legal moves have been searched and if there are
// no legal moves, it must be mate or stalemate.
// If one move was excluded return fail low score.
if (!moveCount)
if (!SpNode && !moveCount)
return excludedMove ? oldAlpha : isCheck ? value_mated_in(ply) : VALUE_DRAW;
// Step 20. Update tables
// If the search is not aborted, update the transposition table,
// history counters, and killer moves.
if (AbortSearch || ThreadsMgr.thread_should_stop(threadID))
return bestValue;
if (!SpNode && !AbortSearch && !ThreadsMgr.thread_should_stop(threadID))
{
move = bestValue <= oldAlpha ? MOVE_NONE : ss->bestMove;
vt = bestValue <= oldAlpha ? VALUE_TYPE_UPPER
: bestValue >= beta ? VALUE_TYPE_LOWER : VALUE_TYPE_EXACT;
ValueType vt = (bestValue <= oldAlpha ? VALUE_TYPE_UPPER : bestValue >= beta ? VALUE_TYPE_LOWER : VALUE_TYPE_EXACT);
move = (bestValue <= oldAlpha ? MOVE_NONE : ss->bestMove);
TT.store(posKey, value_to_tt(bestValue, ply), vt, depth, move, ss->eval, ss->evalMargin);
// Update killers and history only for non capture moves that fails high
@ -1426,6 +1420,14 @@ split_point_start: // At split points actual search starts from here
update_history(pos, move, depth, movesSearched, moveCount);
update_killers(move, ss);
}
}
if (SpNode)
{
// Here we have the lock still grabbed
sp->slaves[threadID] = 0;
lock_release(&(sp->lock));
}
assert(bestValue > -VALUE_INFINITE && bestValue < VALUE_INFINITE);