----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: http://reviews.gem5.org/r/2350/#review5297 -----------------------------------------------------------
src/mem/ruby/common/TypeDefines.hh <http://reviews.gem5.org/r/2350/#comment4819> It would be nice to prune these as well and rely in the established uint64_t int64_t etc. In STL all index types tends to be size_t. Is there a reason for not using that here? - Andreas Hansson On Aug. 16, 2014, 1:32 a.m., Nilay Vaish wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > http://reviews.gem5.org/r/2350/ > ----------------------------------------------------------- > > (Updated Aug. 16, 2014, 1:32 a.m.) > > > Review request for Default. > > > Repository: gem5 > > > Description > ------- > > Changeset 10294:b79de412799b > --------------------------- > ruby: remove typedef of Index as int64 > The Index type defined as typedef int64 does not really provide any help > since in most places we use primitive types instead of Index. Also, the name > Index is very generic that it does not merit being used as a typename. > > > Diffs > ----- > > src/mem/ruby/common/Address.hh 79fde1c67ed8 > src/mem/ruby/common/Address.cc 79fde1c67ed8 > src/mem/ruby/common/TypeDefines.hh 79fde1c67ed8 > src/mem/ruby/structures/AbstractReplacementPolicy.hh PRE-CREATION > src/mem/ruby/structures/BankedArray.hh PRE-CREATION > src/mem/ruby/structures/BankedArray.cc PRE-CREATION > src/mem/ruby/structures/CacheMemory.hh PRE-CREATION > src/mem/ruby/structures/CacheMemory.cc PRE-CREATION > src/mem/ruby/structures/DirectoryMemory.cc PRE-CREATION > src/mem/ruby/structures/LRUPolicy.hh PRE-CREATION > src/mem/ruby/structures/PseudoLRUPolicy.hh PRE-CREATION > > Diff: http://reviews.gem5.org/r/2350/diff/ > > > Testing > ------- > > > Thanks, > > Nilay Vaish > > _______________________________________________ gem5-dev mailing list [email protected] http://m5sim.org/mailman/listinfo/gem5-dev
