> On Feb. 26, 2014, 11:01 a.m., Andreas Hansson wrote: > > src/cpu/pred/bi_mode.cc, line 91 > > <http://reviews.gem5.org/r/2174/diff/1/?file=39301#file39301line91> > > > > should this be const? I guess not since there is an update at the end, > > but perhaps it could be made more explicit
That is the interface exposed by the base predictor for lookup up predictions, which is a pure virtual function. That would have to be changed first. - Anthony ----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: http://reviews.gem5.org/r/2174/#review4927 ----------------------------------------------------------- On June 24, 2014, 5:38 p.m., Anthony Gutierrez wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > http://reviews.gem5.org/r/2174/ > ----------------------------------------------------------- > > (Updated June 24, 2014, 5:38 p.m.) > > > Review request for Default. > > > Repository: gem5 > > > Description > ------- > > Changeset 10243:18b306b6fce3 > --------------------------- > cpu: implement bi-mode-style branch predictor > > > Diffs > ----- > > configs/common/O3_ARM_v7a.py cb4e86c177672fde6be7a409793c944e36353fc0 > src/cpu/o3/fetch_impl.hh cb4e86c177672fde6be7a409793c944e36353fc0 > src/cpu/pred/BranchPredictor.py cb4e86c177672fde6be7a409793c944e36353fc0 > src/cpu/pred/SConscript cb4e86c177672fde6be7a409793c944e36353fc0 > src/cpu/pred/bi_mode.hh PRE-CREATION > src/cpu/pred/bi_mode.cc PRE-CREATION > src/cpu/pred/bpred_unit.hh cb4e86c177672fde6be7a409793c944e36353fc0 > src/cpu/pred/bpred_unit.cc cb4e86c177672fde6be7a409793c944e36353fc0 > > Diff: http://reviews.gem5.org/r/2174/diff/ > > > Testing > ------- > > > Thanks, > > Anthony Gutierrez > > _______________________________________________ gem5-dev mailing list gem5-dev@gem5.org http://m5sim.org/mailman/listinfo/gem5-dev
