> On April 29, 2015, 4:59 p.m., Steve Reinhardt wrote: > > src/mem/cache/blk.hh, line 424 > > <http://reviews.gem5.org/r/2711/diff/3/?file=44635#file44635line424> > > > > I don't think this needs to be a template anymore, does it? It's only > > instantiated once with <Cache> as the parameter. I suggest getting rid of > > the template and also the WrappedBlkVisitor typedef below. The new class > > could use either name.
Not possible since the block cannot include the Cache. The template parameter is really just there to break the circular dependency. > On April 29, 2015, 4:59 p.m., Steve Reinhardt wrote: > > src/mem/cache/blk.hh, line 427 > > <http://reviews.gem5.org/r/2711/diff/3/?file=44635#file44635line427> > > > > should be VisitorPtr Will fix. - Andreas ----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: http://reviews.gem5.org/r/2711/#review6095 ----------------------------------------------------------- On April 23, 2015, 9:36 a.m., Andreas Hansson wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > http://reviews.gem5.org/r/2711/ > ----------------------------------------------------------- > > (Updated April 23, 2015, 9:36 a.m.) > > > Review request for Default. > > > Repository: gem5 > > > Description > ------- > > Changeset 10802:5f762fe388b5 > --------------------------- > mem: Remove templates in cache model > > This patch changes the cache implementation to rely on virtual methods > rather than using the replacement policy as a template argument. > > There is no impact on the simulation performance, and overall the > changes make it easier to modify (and subclass) the cache and/or > replacement policy. > > > Diffs > ----- > > src/mem/cache/tags/fa_lru.hh 74e3c7359393 > src/mem/cache/tags/fa_lru.cc 74e3c7359393 > src/mem/cache/tags/lru.hh 74e3c7359393 > src/mem/cache/tags/lru.cc 74e3c7359393 > src/mem/cache/tags/random_repl.hh 74e3c7359393 > src/mem/cache/tags/random_repl.cc 74e3c7359393 > src/mem/cache/tags/base_set_assoc.cc 74e3c7359393 > src/mem/cache/cache.cc 74e3c7359393 > src/mem/cache/cache_impl.hh 74e3c7359393 > src/mem/cache/tags/base.hh 74e3c7359393 > src/mem/cache/tags/base_set_assoc.hh 74e3c7359393 > src/mem/cache/base.cc 74e3c7359393 > src/mem/cache/blk.hh 74e3c7359393 > src/mem/cache/cache.hh 74e3c7359393 > > Diff: http://reviews.gem5.org/r/2711/diff/ > > > Testing > ------- > > > Thanks, > > Andreas Hansson > > _______________________________________________ gem5-dev mailing list [email protected] http://m5sim.org/mailman/listinfo/gem5-dev
