----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: http://reviews.gem5.org/r/1827/#review4312 -----------------------------------------------------------
Are they anymore concerns/objections to this? - Anthony Gutierrez On April 19, 2013, 8:26 a.m., Anthony Gutierrez wrote: > > ----------------------------------------------------------- > 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. > > > Description > ------- > > 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 > ----- > > 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
