Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Ongoing cleanup of SymbiFlow sv-tests (help wanted) #1628

Open
veripoolbot opened this issue Dec 8, 2019 · 2 comments
Open

Ongoing cleanup of SymbiFlow sv-tests (help wanted) #1628

veripoolbot opened this issue Dec 8, 2019 · 2 comments

Comments

@veripoolbot
Copy link

@veripoolbot veripoolbot commented Dec 8, 2019


Author Name: Wilson Snyder (@wsnyder)
Original Redmine Issue: 1628 from https://www.veripool.org


SymbiFlow has a large set of System Verilog tests: https://github.com/SymbiFlow/sv-tests
With current runtime status here: https://symbiflow.github.io/sv-tests/

Ongoing project is to go through each of these, understand the failure, and either fix the tests or Verilator to get as green as reasonable.

One change not yet pushed to sv-tests (waiting for Verilator release) is using "--Wpedantic -Wno-context" which should clean up at least the following:

 chapter-20/20.5--shortreal-bits-conv.sv
 chapter-22/22.12--*
@veripoolbot

This comment has been minimized.

Copy link
Author

@veripoolbot veripoolbot commented Dec 15, 2019


Original Redmine Comment
Author Name: Stefan Wallentowitz (@wallento)
Original Date: 2019-12-15T22:26:46Z


Hi,

I saw that there are a couple of missing things around queues. Do you want to get them added, in particular bounded queues? Also, currently we don't support indexed accesses and deletes, right? Do you mind to add them, the user will probably (actually probably not..) understand they are slow.

I can set aside some time to get many of the errors fixed while already touching this part.

Cheers,
Stefan

@veripoolbot

This comment has been minimized.

Copy link
Author

@veripoolbot veripoolbot commented Dec 15, 2019


Original Redmine Comment
Author Name: Wilson Snyder (@wsnyder)
Original Date: 2019-12-15T22:41:58Z


Bounded queues are supported in git master as of yesterday ;), but that's where I was stopping. Anything else you'd like to add is appreciated, probably assignment patterns are the main part missing to provide what I see UVM using for queues.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
1 participant
You can’t perform that action at this time.