# HG changeset patch
# User Maximilien Breughe <[email protected]>
# Date 1272957849 -7200
# Node ID fdb25b77b535a8e4ba5b32f38a89b548275ec25c
# Parent  4b2774bb954ac740d63379209a70763c02a0e1e4
Fixed the treshold-bug in the tournament predictor.

Suppose the saturating counters of a branch predictor contain n bits.
When the counter is between 0 and (2^(n-1) - 1), boundaries included, the branch
is predicted as not taken.
When the counter is between 2^(n-1) and (2^n - 1), boundaries included, the 
branch
is predicted as taken.

diff -r 4b2774bb954a -r fdb25b77b535 src/cpu/pred/tournament.cc
--- a/src/cpu/pred/tournament.cc        Tue May 04 09:12:41 2010 +0200
+++ b/src/cpu/pred/tournament.cc        Tue May 04 09:24:09 2010 +0200
@@ -106,7 +106,6 @@
 
     // @todo: Allow for different thresholds between the predictors.
     threshold = (1 << (localCtrBits - 1)) - 1;
-    threshold = threshold / 2;
 }
 
 inline
_______________________________________________
m5-dev mailing list
[email protected]
http://m5sim.org/mailman/listinfo/m5-dev

Reply via email to