-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
http://reviews.gem5.org/r/1827/
-----------------------------------------------------------

(Updated April 19, 2013, 8:26 a.m.)


Review request for Default.


Summary (updated)
-----------------

cpu: remove local/globalHistoryBits params from branch pred


Description (updated)
-------

Changeset 9644:decb4c9e987a
---------------------------
cpu: remove local/globalHistoryBits params from branch pred

having separate params for the local/globalHistoryBits and the
local/globalPredictorSize can lead to inconsistencies if they
are not carefully set. this patch dervies the number of bits
necessary to index into the local/global predictors based on
their size.

the value of the localHistoryTableSize for the ARM O3 CPU has been
increased to 1024 from 64, which is more accurate for an A15 based
on some correlation against A15 hardware.


Diffs (updated)
-----

  configs/common/O3_ARM_v7a.py 745e42ffcc802622d54e52d21a494eca9c9ee9c6 
  src/cpu/pred/BranchPredictor.py 745e42ffcc802622d54e52d21a494eca9c9ee9c6 
  src/cpu/pred/tournament.cc 745e42ffcc802622d54e52d21a494eca9c9ee9c6 

Diff: http://reviews.gem5.org/r/1827/diff/


Testing
-------


Thanks,

Anthony Gutierrez

_______________________________________________
gem5-dev mailing list
[email protected]
http://m5sim.org/mailman/listinfo/gem5-dev

Reply via email to