ibex
ibex copied to clipboard
lowRISC
Rip out all the non-cosim stuff from dv/uvm/core-ibex scripts
Depends on #1565 but, once we've got working "cosim tooling", we should get rid of the old way of doing things.
This doesn't include the work to rip out the old sequences for e.g. exceptions: we'll track that separately.
estimate 2
The dependency #1565 has been closed out by Greg on 10 May. As a tldr; for Cosim tooling: Spike with cosim support. Main thing discussed in #1565 was to get Spike running with Cosim and then deb package it and have it run on the CI.
Main files that reference this are under: https://github.com/lowRISC/ibex/tree/master/dv/uvm/core_ibex Namely; Makefile and compare.py. The tests will also use a post_compare flag that will need to be deprecated/removed.
In the documentation side of things the files https://ibex-core.readthedocs.io/en/latest/03_reference/verification.html etc will need to have post_compare removed.
Estimate of time needed is half a day for documentation removal of any post_compare and non-cosim tooling and a couple of hours for modifying the code to remove references. Total est time: 1 day.
Adding this related issue that is touched upon by cosim changes. https://github.com/lowRISC/ibex/issues/1669
Per the Thursday HW Ibex DV meeting. Splitting this issue into two parts. This issue has been re-labeled with the tools and scripts elements. New issue related to documentation to be created. This issue should then be able to be closed off by @hcallahan-lowrisc fairly soon. Believe that it is waiting on a PR that @hcallahan-lowrisc can link here.
@johngt Will be closed by #1705, which addresses this among other refactoring changes.
@hcallahan-lowrisc - believe that https://github.com/lowRISC/ibex/pull/1705 is now merged so that this can now be closed out. Not tagging is completed just yet.
@johngt I've created one final small PR to remove the last few items. I've setup that PR to close this issue.