----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: http://reviews.gem5.org/r/1987/ -----------------------------------------------------------
(Updated Aug. 27, 2013, 5:10 p.m.) Review request for Default. Repository: gem5 Description (updated) ------- Changeset 9852:b30933fc9697 --------------------------- 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 (updated) ----- 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
