----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: http://reviews.m5sim.org/r/830/ -----------------------------------------------------------
(Updated 2011-09-13 19:16:20.828084) Review request for Default, Ali Saidi, Gabe Black, Steve Reinhardt, and Nathan Binkert. Summary ------- BP: Fix several Branch Predictor issues. 1. Updates the Branch Predictor correctly to the state just after a mispredicted branch, if a squash occurs. 2. If a BTB does not find an entry, the branch is predicted not taken. The global history is modified to correctly reflect this prediction. 3. Local history is now updated at the fetch stage instead of execute stage. 4. In the Update stage of the branch predictor the local predictors are now correctly updated according to the state of local history during fetch stage. This patch also improves performance by as much as 17% on some benchmarks Diffs (updated) ----- src/cpu/inorder/resources/bpred_unit.hh 5272eb9ee1c9 src/cpu/inorder/resources/bpred_unit.cc 5272eb9ee1c9 src/cpu/o3/bpred_unit.hh 5272eb9ee1c9 src/cpu/o3/bpred_unit_impl.hh 5272eb9ee1c9 src/cpu/o3/commit_impl.hh 5272eb9ee1c9 src/cpu/o3/decode_impl.hh 5272eb9ee1c9 src/cpu/pred/2bit_local.hh 5272eb9ee1c9 src/cpu/pred/2bit_local.cc 5272eb9ee1c9 src/cpu/pred/tournament.hh 5272eb9ee1c9 src/cpu/pred/tournament.cc 5272eb9ee1c9 Diff: http://reviews.m5sim.org/r/830/diff Testing ------- Thanks, Ali _______________________________________________ gem5-dev mailing list [email protected] http://m5sim.org/mailman/listinfo/gem5-dev
