> On Oct. 26, 2016, 7:22 p.m., Andreas Hansson wrote: > > Somehow the patch is not displaying. Did you use hg postreview? > > Christian Menard wrote: > Hi Andreas! Thanks for your reviews, I will give them a look soon. I > wanted to do some cleaning up anyway as there are still some inconsistencies > in master and slave port. > > I uploaded the patches manually since I don't have the postreview > extension. Where can I get it? The link on the wiki is broken.
No worries. The link is updated. - Andreas ----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: http://reviews.gem5.org/r/3686/#review8990 ----------------------------------------------------------- On Oct. 26, 2016, 4:26 p.m., Christian Menard wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > http://reviews.gem5.org/r/3686/ > ----------------------------------------------------------- > > (Updated Oct. 26, 2016, 4:26 p.m.) > > > Review request for Default. > > > Repository: gem5 > > > Description > ------- > > This patch simplifies the implementation of the SystemC TLM slave port. > > > Diffs > ----- > > util/tlm/sc_slave_port.hh c6e92c923a9e1c8cfd06ce9da1f61476ae4cb007 > util/tlm/sc_slave_port.cc c6e92c923a9e1c8cfd06ce9da1f61476ae4cb007 > > Diff: http://reviews.gem5.org/r/3686/diff/ > > > Testing > ------- > > Example applications are still running. > > > Thanks, > > Christian Menard > > _______________________________________________ gem5-dev mailing list [email protected] http://m5sim.org/mailman/listinfo/gem5-dev
