----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: http://reviews.gem5.org/r/1827/#review4253 -----------------------------------------------------------
Ship it! Ship It! - Andreas Hansson On April 18, 2013, 7:26 a.m., Anthony Gutierrez wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > http://reviews.gem5.org/r/1827/ > ----------------------------------------------------------- > > (Updated April 18, 2013, 7:26 a.m.) > > > Review request for Default. > > > Description > ------- > > Changeset 9638:213065dd02d9 > --------------------------- > 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 c22628fa25646077ed4f18a811f348b47a64bd5d > src/cpu/pred/BranchPredictor.py c22628fa25646077ed4f18a811f348b47a64bd5d > src/cpu/pred/tournament.cc c22628fa25646077ed4f18a811f348b47a64bd5d > > 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
