On Sat, Jun 2, 2012 at 5:00 PM, R. Diez <[email protected]> wrote:
> Hallo Julius:
>
>> I have expanded the ORPSoC overflow flag testsuite and got a better fix.
>> [...]
>> If I submit an RTL patch, are you happy to test it against your suite
>> before I check this in?
>
> I wonder if it makes sense to maintain 3 different test suites: one in 
> or1ksim, one in ORPSoC (which is rather small) and the merged one I ported to 
> orbuild.

No of course not, that's why what you're doing is very good. But at
present I'm doing my development in ORPSoC, and from the sounds of
things your testsuite already surpasses what's in ORPSoC to test the
overflow stuff.

>
> The latter has the advantage that you can run it easily against other 
> targets. As you have seen, orbuild's test suite is the most helpful one when 
> it comes to finding bugs in ORPSoC and will also help debug other upcoming 
> CPU core implementations, like Peter Gavin's CGEN-based simulator.
>
> Wouldn't it be better if you ran orbuild's test suite on your PC before you 
> checked in your changes?
>

Probably, yes. Is exactly what you're testing with available in a repo
somewhere?

Cheers

Julius
_______________________________________________
OpenRISC mailing list
[email protected]
http://lists.openrisc.net/listinfo/openrisc

Reply via email to