-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
http://reviews.m5sim.org/r/649/#review1140
-----------------------------------------------------------


I'm going to also send you my code for building.  I'd like to incorporate some 
of the ideas. So I can import regress (which would mean creating build.py or 
something like that) and use it directly in my environment.  (We can do that as 
a separate diff though).


util/regress
<http://reviews.m5sim.org/r/649/#comment1556>

    While you're at it you could add the following to the help:
    
    (0 to use total number of cores)



util/regress
<http://reviews.m5sim.org/r/649/#comment1557>

    Another common option is --update-ref, can we add that here and pass it 
through?



util/regress
<http://reviews.m5sim.org/r/649/#comment1559>

    I have nicer code for this sort of thing.  You can see it in 
src/python/m5/options.py, examples of usage in main.py
    
    Look for action='append' (and notice the necessity of split)



util/regress
<http://reviews.m5sim.org/r/649/#comment1558>

    Seems like we should fix this on the SCons side.  Is that difficult?  Also, 
does NOISA have any tests?  Seems like we could have a memory tester for NOISA.


- Nathan


On 2011-04-20 22:42:16, Steve Reinhardt wrote:
> 
> -----------------------------------------------------------
> This is an automatically generated e-mail. To reply, visit:
> http://reviews.m5sim.org/r/649/
> -----------------------------------------------------------
> 
> (Updated 2011-04-20 22:42:16)
> 
> 
> Review request for Default, Ali Saidi, Gabe Black, Steve Reinhardt, and 
> Nathan Binkert.
> 
> 
> Summary
> -------
> 
> util/regress: make default action a more thorough regression
> 
> Changed the --variants option to --test-variants and added a new
> --compile-variants option for variants that are only compiled
> (not tested).  The former still defaults to 'opt' and the latter
> defaults to 'debug,fast'.
> 
> Also changed the behavior when no tests are specified from just
> compiling to running the 'quick' tests.
> 
> As a result, a plain 'util/regress' invocation will now compile
> (but not test) the debug and fast builds, and compile and run the
> quick regressions on the opt build.  This should be the default
> set of tests that are run before committing.  Since the nightly
> regressions use this same script, this will also be the new
> nightly regression behavior.
> 
> Test-only regressions can still be done by setting --compile=''.
> Compile-only regressions can be done by setting --test=''.
> 
> 
> Diffs
> -----
> 
>   util/regress a9d06c894afe 
> 
> Diff: http://reviews.m5sim.org/r/649/diff
> 
> 
> Testing
> -------
> 
> 
> Thanks,
> 
> Steve
> 
>

_______________________________________________
m5-dev mailing list
m5-dev@m5sim.org
http://m5sim.org/mailman/listinfo/m5-dev

Reply via email to