On Feb. 26, 2014, 11:01 a.m., Anthony Gutierrez wrote: > > As Nilay pointed out, the level of comments is not overwhelming. It would > > be great to see a bit more of an explanation around the computations done > > (and the constants used).
I added more comments about the intricacies of the bi-mode predictor. Hopefully they suffice. - 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
