summary |
shortlog |
log |
commit | commitdiff |
tree
raw |
patch |
inline | side by side (from parent 1:
77547a4)
Most of the time we cut-off earlier, at captures, so this
results in useless work.
There is a small functionality change becuase 'ss' can change
from MovePicker c'tor to when killers are tried due, for
instance, to singular search.
bench:
4603795
captureThreshold = 0;
cur = end = moves;
endBadCaptures = moves + MAX_MOVES - 1;
captureThreshold = 0;
cur = end = moves;
endBadCaptures = moves + MAX_MOVES - 1;
ss = s;
if (p.checkers())
ss = s;
if (p.checkers())
- killers[0].move = ss->killers[0];
- killers[1].move = ss->killers[1];
- killers[2].move = cm[0];
- killers[3].move = cm[1];
-
// Consider sligtly negative captures as good if at low depth and far from beta
// Consider sligtly negative captures as good if at low depth and far from beta
- if (ss && ss->staticEval < beta - PawnValueMg && d < 3 * ONE_PLY)
+ if (ss->staticEval < beta - PawnValueMg && d < 3 * ONE_PLY)
captureThreshold = -PawnValueMg;
// Consider negative captures as good if still enough to reach beta
captureThreshold = -PawnValueMg;
// Consider negative captures as good if still enough to reach beta
- else if (ss && ss->staticEval > beta)
+ else if (ss->staticEval > beta)
captureThreshold = beta - ss->staticEval;
}
captureThreshold = beta - ss->staticEval;
}
cur = killers;
end = cur + 2;
cur = killers;
end = cur + 2;
- if ((cur+3)->move && (cur+3)->move == (cur+2)->move) // Due to a SMP race
- (cur+3)->move = MOVE_NONE;
+ killers[0].move = ss->killers[0];
+ killers[1].move = ss->killers[1];
+ killers[2].move = killers[3].move = MOVE_NONE;
// Be sure countermoves are different from killers
// Be sure countermoves are different from killers
- if ((cur+2)->move != cur->move && (cur+2)->move != (cur+1)->move)
- end++;
+ for (int i = 0; i < 2; i++)
+ if (countermoves[i] != cur->move && countermoves[i] != (cur+1)->move)
+ (end++)->move = countermoves[i];
+
+ if (countermoves[1] && countermoves[1] == countermoves[0]) // Due to SMP races
+ killers[3].move = MOVE_NONE;
- if ((cur+3)->move != cur->move && (cur+3)->move != (cur+1)->move)
- (end++)->move = (cur+3)->move;
return;
case QUIETS_1_S1:
return;
case QUIETS_1_S1:
const Position& pos;
const HistoryStats& history;
Search::Stack* ss;
const Position& pos;
const HistoryStats& history;
Search::Stack* ss;
Depth depth;
Move ttMove;
MoveStack killers[4];
Depth depth;
Move ttMove;
MoveStack killers[4];