diff options
| author | Scott Gasch <[email protected]> | 2026-10-14 11:16:06 -0700 |
|---|---|---|
| committer | Scott Gasch <[email protected]> | 2026-10-14 11:16:06 -0700 |
| commit | 3845a082df2fd0d1bd059dd4c22c922067be826a (patch) | |
| tree | 01f7059b7d1d7e7f0495a281512a6ac1610a280b /src/root.c | |
| parent | 3d54c2cc82366040f00d5901f80dbe4e9d7bfe10 (diff) | |
Remove dead EFP references left over from its removal
EFP (extended futility pruning) was removed from Search()'s move loop
in favor of LMP's joint FH% table, but several comments, a dead
counter (u64EFPPruned, never incremented anywhere), and two unused
defines (EFP_FH_MIN_SAMPLES/EFP_FH_PRUNE_THRESHOLD_PCT_THOUSANDTHS)
were left behind claiming EFP still gates something. Clean these up:
- Delete u64EFPPruned (COUNTERS and g_Options) and its dead
reader/writer in root.c.
- Delete the two unused EFP_FH_* defines in search.c, and the stale
fAnyMoveEFPPruned comment describing a branch that no longer
skips a store.
- Fix dangling prose references to EFP in generate.c, chess.h,
searchsup.c, and testsearch.c that describe present-tense behavior
EFP no longer participates in, preserving the historical rationale
where it's still relevant (e.g. why EstimateHangingMaterialCp can
return a negative value).
Deliberately left alone (flagged for a separate decision, not a
clear delete): script.c's still-live JOINT_EXTRA_VERIFY calibration
labels ("EFP tier1/tier2"), search.c's _PostReduceDepthBucketEfpAligned
(same live mechanism, name only), searchsup.c's LMP_BASE_MOVENUM/
LMP_PER_PLY_MOVENUM dead macros (kept pending a real floor/ceiling
decision), and PLY_INFO.iPositional[2] (now write-only, but removing
it touches Eval()'s call signature everywhere).
Comment/dead-code only, no logic touched.
Co-Authored-By: Claude Sonnet 5 <[email protected]>
Diffstat (limited to 'src/root.c')
| -rw-r--r-- | src/root.c | 6 |
1 files changed, 2 insertions, 4 deletions
@@ -469,9 +469,8 @@ Return value: Trace("Null move cutoff rate: %5.3f percent.\n", ((n / d) * 100.0)); #ifdef PERF_COUNTERS - Trace("Overt move skips: LMP=%llu EFP=%llu moves.\n", - ctx->sCounters.tree.u64LMPPruned, - ctx->sCounters.tree.u64EFPPruned); + Trace("Overt move skips: LMP=%llu moves.\n", + ctx->sCounters.tree.u64LMPPruned); { ULONG uR; UINT64 u64TotalTries = 0; @@ -1594,7 +1593,6 @@ Return value: memcpy(g_Options.u64JointExtraResearch, ctx->sCounters.tree.u64JointExtraResearch, sizeof(g_Options.u64JointExtraResearch)); g_Options.u64LMPPruned = ctx->sCounters.tree.u64LMPPruned; - g_Options.u64EFPPruned = ctx->sCounters.tree.u64EFPPruned; memcpy(g_Options.u64NullMoveTriesByR, ctx->sCounters.tree.u64NullMoveTriesByR, sizeof(g_Options.u64NullMoveTriesByR)); memcpy(g_Options.u64NullMoveSuccessByR, ctx->sCounters.tree.u64NullMoveSuccessByR, |
