> On Oct. 14, 2015, 2:02 a.m., Nilay Vaish wrote: > > Format this patch again. Read the following page: > > http://gem5.org/Submitting_Contributions
Several lines are more than 80 characters. Also make sure that the identation is correct. - Nilay ----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: http://reviews.gem5.org/r/3132/#review7369 ----------------------------------------------------------- On Sept. 25, 2015, 3:16 p.m., Abdul Mutaal Ahmad wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > http://reviews.gem5.org/r/3132/ > ----------------------------------------------------------- > > (Updated Sept. 25, 2015, 3:16 p.m.) > > > Review request for Default. > > > Repository: gem5 > > > Description > ------- > > In previous versions of systemC-gem5 coupling statistics were not updated for > the systemc-gem5 simulation. systemC-gem5 simulation only need the previously > built config.ini file and normal gem5 simulation has to be run once to > generate config.ini file. Thus stats.txt inside the m5out folder is redundant > for systemC-gem5 simulation. A new stats file is now generated with the all > the statistics for systemC-gem5 simulation. This will also resolve the stats > issue in tlm-sysmtemC simulation. > > > Diffs > ----- > > util/systemc/main.cc UNKNOWN > util/systemc/stats.hh UNKNOWN > util/systemc/stats.cc UNKNOWN > util/tlm/main.cc UNKNOWN > > Diff: http://reviews.gem5.org/r/3132/diff/ > > > Testing > ------- > > > Thanks, > > Abdul Mutaal Ahmad > > _______________________________________________ gem5-dev mailing list [email protected] http://m5sim.org/mailman/listinfo/gem5-dev
