----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: http://reviews.gem5.org/r/982/#review1978 -----------------------------------------------------------
Ship it! Looks good to me... Korey? Steve? - Ali Saidi On Jan. 20, 2012, 11:50 a.m., Anthony Gutierrez wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > http://reviews.gem5.org/r/982/ > ----------------------------------------------------------- > > (Updated Jan. 20, 2012, 11:50 a.m.) > > > Review request for Default. > > > Description > ------- > > Changeset 8650:a01038f0f464 > --------------------------- > Stats: Add stats for instructions and ops, both globally and per cpu > > > 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 > ------- > > Ran regression test. Diffed stats files. sim_ops matches old sim_insts. New > sim_insts and per cpu model inst/ops stats are consistent with these new > sim_ops/sim_insts. > > > Thanks, > > Anthony Gutierrez > > _______________________________________________ gem5-dev mailing list [email protected] http://m5sim.org/mailman/listinfo/gem5-dev
