----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: http://reviews.gem5.org/r/1987/#review4663 -----------------------------------------------------------
Ship it! Ship It! - Andreas Hansson On Aug. 28, 2013, 12:12 a.m., Steve Reinhardt wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > http://reviews.gem5.org/r/1987/ > ----------------------------------------------------------- > > (Updated Aug. 28, 2013, 12:12 a.m.) > > > Review request for Default. > > > Repository: gem5 > > > Description > ------- > > Changeset 9852:d1f17f13ef26 > --------------------------- > cpu/o3: clean up rename map and free list > > Restructured rename map and free list to clean up some > extraneous code and separate out common code that can > be reused across different register classes (int and fp > at this point). Both components now consist of a set > of Simple* objects that are stand-alone rename map & > free list for each class, plus a Unified* object that > presents a unified interface across all register > classes and then redirects accesses to the appropriate > Simple* object as needed. > > Moved free list initialization to PhysRegFile to better > isolate knowledge of physical register index mappings > to that class (and remove the need to pass a number > of parameters to the free list constructor). > > Causes a small change to these stats: > cpu.rename.int_rename_lookups > cpu.rename.fp_rename_lookups > because they are now categorized on a per-operand basis > rather than a per-instruction basis. > That is, an instruction with mixed fp/int/misc operand > types will have each operand categorized independently, > where previously the lookup was categorized based on > the instruction type. > > > Diffs > ----- > > src/cpu/o3/SConscript 3f6e2f267aba6571c478fcd83c4a1b9d6564084f > src/cpu/o3/cpu.hh 3f6e2f267aba6571c478fcd83c4a1b9d6564084f > src/cpu/o3/cpu.cc 3f6e2f267aba6571c478fcd83c4a1b9d6564084f > src/cpu/o3/cpu_policy.hh 3f6e2f267aba6571c478fcd83c4a1b9d6564084f > src/cpu/o3/free_list.hh 3f6e2f267aba6571c478fcd83c4a1b9d6564084f > src/cpu/o3/free_list.cc 3f6e2f267aba6571c478fcd83c4a1b9d6564084f > src/cpu/o3/regfile.hh 3f6e2f267aba6571c478fcd83c4a1b9d6564084f > src/cpu/o3/regfile.cc PRE-CREATION > src/cpu/o3/rename_impl.hh 3f6e2f267aba6571c478fcd83c4a1b9d6564084f > src/cpu/o3/rename_map.hh 3f6e2f267aba6571c478fcd83c4a1b9d6564084f > src/cpu/o3/rename_map.cc 3f6e2f267aba6571c478fcd83c4a1b9d6564084f > > Diff: http://reviews.gem5.org/r/1987/diff/ > > > Testing > ------- > > > Thanks, > > Steve Reinhardt > > _______________________________________________ gem5-dev mailing list [email protected] http://m5sim.org/mailman/listinfo/gem5-dev
