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

Add NVC simulator support #44

Closed
kraigher opened this issue May 9, 2015 · 11 comments
Closed

Add NVC simulator support #44

kraigher opened this issue May 9, 2015 · 11 comments

Comments

@kraigher
Copy link
Collaborator

kraigher commented May 9, 2015

I am in progress of adding support for the NVC simulator. There are still some bugs and features to be fixed be before we can add support for it but the author is very cooperative and is making great progress.

@kraigher kraigher self-assigned this May 9, 2015
@kraigher
Copy link
Collaborator Author

I have pushed the work in progress to the nvc branch. The currently blocking bug is nickg/nvc#185 which is triggered by test_runner_cleanup. Otherwise it is able to compile and run the user_guide example up until test_runner_cleanup.

@peteut
Copy link

peteut commented Apr 2, 2016

I just grepped the nvc branch but couldn't find any reference to nvc. Is there any progress on this?

@kraigher
Copy link
Collaborator Author

kraigher commented Apr 2, 2016

@peteut nvc support has not been quite finished yet but the basic user guide example works. There are still some problems to be solved like that it does not like some of our custom resolved types in the VHDL runner but @LarsAsplund might refactor those away. It is in a usable state though unless you use special stuff like test_runner_watchdog which will require the custom resolved type.

@peteut
Copy link

peteut commented Apr 2, 2016

Great, I'm testing my VHDL backend for Migen currently and aiming for unit test support, vunit looks like the perfect fit. I'm not going to use test_runner_watchdog anyway and wrote a basic SimulatorInterface for nvc for now.

@kraigher
Copy link
Collaborator Author

kraigher commented Apr 2, 2016

@peteut There is already a simulator interface for nvc on the nvc branch.

@peteut
Copy link

peteut commented Apr 2, 2016

@kraigher: got it, thanks! I somehow created a local branch, instead of checking out your nvc branch. Sorry for the confusion.

@nickg
Copy link
Contributor

nickg commented Feb 5, 2018

@kraigher what issues are stopping this working now? I have some free time to work on nvc at the moment so could look into any problems you run into...

@kraigher
Copy link
Collaborator Author

kraigher commented Feb 6, 2018

@nickg I will check with your latest fixes after I rebase the NVC branch. At least this issue is still a blocker nickg/nvc#202 if we do not work arround it in VUnit itself.

@nickg
Copy link
Contributor

nickg commented Feb 6, 2018

@kraigher nickg/nvc#202 is fixed now but I only tested it it a few simple cases so it may still have bugs

@kraigher
Copy link
Collaborator Author

kraigher commented Feb 6, 2018

@nickg I rebased nvc branch on vunit3 and found various problems. I will create reprocers for them. Some assets failed and some unresolved symbol.

@LarsAsplund
Copy link
Collaborator

Closed by #904

@eine eine added the Tool: NVC label Feb 27, 2023
@eine eine added this to the v4.7.0 milestone Feb 27, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

5 participants