----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: http://reviews.gem5.org/r/982/#review1982 -----------------------------------------------------------
src/cpu/inorder/cpu.hh <http://reviews.gem5.org/r/982/#comment2480> I'm still a little confused here... it seems like either we should not rename these stats from "committed" to "executed", or the comments are wrong and should say "executed" rather than "committed". - Steve Reinhardt On Jan. 22, 2012, 1:46 p.m., Anthony Gutierrez wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > http://reviews.gem5.org/r/982/ > ----------------------------------------------------------- > > (Updated Jan. 22, 2012, 1:46 p.m.) > > > Review request for Default. > > > Description > ------- > > Stats: Add stats for instructions and micro ops, both globally and per CPU. > > Adds an instructions_executed and operations_executed stat for each CPU > model. Also creates a new global stat sim_uOps which counts all operations. > Changes sim_insts to count instructions. > > > Diffs > ----- > > src/arch/noisa/cpu_dummy.hh UNKNOWN > src/cpu/base.hh UNKNOWN > src/cpu/base.cc UNKNOWN > src/cpu/inorder/cpu.hh UNKNOWN > src/cpu/inorder/cpu.cc UNKNOWN > src/cpu/inorder/inorder_dyn_inst.hh UNKNOWN > src/cpu/o3/commit.hh UNKNOWN > src/cpu/o3/commit_impl.hh UNKNOWN > src/cpu/o3/cpu.hh UNKNOWN > src/cpu/o3/cpu.cc UNKNOWN > src/cpu/simple/base.hh UNKNOWN > src/cpu/simple/base.cc UNKNOWN > src/cpu/thread_state.hh UNKNOWN > src/cpu/thread_state.cc UNKNOWN > src/sim/stat_control.cc UNKNOWN > > Diff: http://reviews.gem5.org/r/982/diff/diff > > > Testing > ------- > > > Thanks, > > Anthony Gutierrez > > _______________________________________________ gem5-dev mailing list [email protected] http://m5sim.org/mailman/listinfo/gem5-dev
