----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: http://reviews.gem5.org/r/1876/#review4355 -----------------------------------------------------------
tests/SConscript <http://reviews.gem5.org/r/1876/#comment4100> Now that you have separate diff_status variable, I think the next 7-8 lines should be combined with code above where case analysis is being carried out on status. - Nilay Vaish On May 23, 2013, 12:47 a.m., Andreas Hansson wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > http://reviews.gem5.org/r/1876/ > ----------------------------------------------------------- > > (Updated May 23, 2013, 12:47 a.m.) > > > Review request for Default. > > > Description > ------- > > Changeset 9733:4999737a9e21 > --------------------------- > scons: Identify runs that fail and runs with stats differences > > This patch changes the regression script such that it is possible to > identify the runs that fail with an exit code, and those that finish > with stats differences. The ones that truly fail are reported as > FAILED, and those that finish with changed stats as CHANGED. > > The yellow colour has been reclaimed from the skipped regressions and > is now used for the changed ones. With no obvious good option left the > skipped ones are now in cyan. > > While I was editing the script I also bumped an occurence of M5 to > gem5. > > > Diffs > ----- > > tests/SConscript 782b7284de21 > > Diff: http://reviews.gem5.org/r/1876/diff/ > > > Testing > ------- > > Ran a number of regressions with various levels of fail/change/pass > > > Thanks, > > Andreas Hansson > > _______________________________________________ gem5-dev mailing list [email protected] http://m5sim.org/mailman/listinfo/gem5-dev
