> On April 5, 2012, 9:16 a.m., Nilay Vaish wrote: > > src/cpu/testers/directedtest/RubyDirectedTester.hh, line 50 > > <http://reviews.gem5.org/r/1124/diff/1/?file=25447#file25447line50> > > > > Is this class still required?
Yes. The MasterPort has abstract members and the CpuPort is still needed to implement the accessors. - Andreas ----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: http://reviews.gem5.org/r/1124/#review2446 ----------------------------------------------------------- On April 4, 2012, 10:23 a.m., Andreas Hansson wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > http://reviews.gem5.org/r/1124/ > ----------------------------------------------------------- > > (Updated April 4, 2012, 10:23 a.m.) > > > Review request for Default. > > > Description > ------- > > Ruby: Use MasterPort base-class pointers where possible > > This patch simplifies future patches by changing the pointer type used > in a number of the Ruby testers to use MasterPort instead of using a > derived CpuPort class. There is no reason for using the more > specialised pointers, and there is no longer a need to do any casting. > > > Diffs > ----- > > src/cpu/testers/directedtest/InvalidateGenerator.cc 570b44fe6e04 > src/cpu/testers/directedtest/RubyDirectedTester.hh 570b44fe6e04 > src/cpu/testers/directedtest/SeriesRequestGenerator.cc 570b44fe6e04 > src/cpu/testers/rubytest/Check.cc 570b44fe6e04 > src/cpu/testers/rubytest/RubyTester.hh 570b44fe6e04 > > Diff: http://reviews.gem5.org/r/1124/diff/ > > > Testing > ------- > > util/regress all passing (disregarding t1000 and eio) > > > Thanks, > > Andreas Hansson > > _______________________________________________ gem5-dev mailing list [email protected] http://m5sim.org/mailman/listinfo/gem5-dev
